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

Proposed by Raphael Isemann
Status: Merged
Approved by: Danielle Foré
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 Pending
Review via email: mp+232468@code.launchpad.net

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
=== modified file 'src/Widgets/AppEntry.vala'
--- src/Widgets/AppEntry.vala 2014-05-26 06:11:39 +0000
+++ src/Widgets/AppEntry.vala 2014-08-27 19:44:21 +0000
@@ -36,6 +36,7 @@
36 private Backend.App application;36 private Backend.App application;
3737
38 public AppEntry (Backend.App app) {38 public AppEntry (Backend.App app) {
39 this.relief = Gtk.ReliefStyle.NONE;
39 Gtk.TargetEntry dnd = {"text/uri-list", 0, 0};40 Gtk.TargetEntry dnd = {"text/uri-list", 0, 0};
40 Gtk.drag_source_set (this, Gdk.ModifierType.BUTTON1_MASK, {dnd},41 Gtk.drag_source_set (this, Gdk.ModifierType.BUTTON1_MASK, {dnd},
41 Gdk.DragAction.COPY);42 Gdk.DragAction.COPY);

Subscribers

People subscribed via source and target branches