Merge lp:~elementary-apps/slingshot/no-relief-style into lp:~elementary-pantheon/slingshot/trunk

Proposed by Raphael Isemann on 2014-08-27
Status: Merged
Approved by: Daniel Fore on 2014-08-28
Approved revision: 444
Merged at revision: 444
Proposed branch: lp:~elementary-apps/slingshot/no-relief-style
Merge into: lp:~elementary-pantheon/slingshot/trunk
Diff against target: 11 lines (+1/-0)
1 file modified
src/Widgets/AppEntry.vala (+1/-0)
To merge this branch: bzr merge lp:~elementary-apps/slingshot/no-relief-style
Reviewer Review Type Date Requested Status
elementary Pantheon team 2014-08-27 Pending
Review via email:

Commit message

Set Button relief style to "NONE"

Description of the change

Other themes don't look good with the current slingshot as you can see in the screen. This patch fixes this in code.

It's up to discussion if we want to fix it in code or if the themes have to support slingshot directly (as i think it could also be fixed in the themes).

Not my patch, just turned the patch from Sergio Costas into a branch.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Widgets/AppEntry.vala'
2--- src/Widgets/AppEntry.vala 2014-05-26 06:11:39 +0000
3+++ src/Widgets/AppEntry.vala 2014-08-27 19:44:21 +0000
4@@ -36,6 +36,7 @@
5 private Backend.App application;
7 public AppEntry (Backend.App app) {
8+ this.relief = Gtk.ReliefStyle.NONE;
9 Gtk.TargetEntry dnd = {"text/uri-list", 0, 0};
10 Gtk.drag_source_set (this, Gdk.ModifierType.BUTTON1_MASK, {dnd},
11 Gdk.DragAction.COPY);


People subscribed via source and target branches