Merge lp:~l-admin-3/egtk/non-transparent-buttons into lp:~elementary-design/egtk/trunk

Proposed by Marcus Wichelmann
Status: Rejected
Rejected by: Cody Garver
Proposed branch: lp:~l-admin-3/egtk/non-transparent-buttons
Merge into: lp:~elementary-design/egtk/trunk
Diff against target: 37 lines (+21/-1)
1 file modified
gtk-3.0/gtk-widgets.css (+21/-1)
To merge this branch: bzr merge lp:~l-admin-3/egtk/non-transparent-buttons
Reviewer Review Type Date Requested Status
Cody Garver (community) Needs Resubmitting
Raphael Isemann (community) testing Approve
Danielle Foré Pending
Review via email: mp+229322@code.launchpad.net

Description of the change

I've added a new class to create non-transparent buttons to solve the indication problem in pantheon-greeter.

To post a comment you must log in.
Revision history for this message
Marcus Wichelmann (l-admin-3) wrote :

Sorry, i did not know, how to solve the problem with the blank line at the end of the file. I have tried some editors, but nothing works.

Revision history for this message
Raphael Isemann (teemperor) wrote :
Revision history for this message
Raphael Isemann (teemperor) wrote :

Same status as the other merge.

review: Needs Fixing
Revision history for this message
Raphael Isemann (teemperor) wrote :

About the blank like at the end of the file: Depends on the editor, but as long as you don't do languages like C/C++ that kind of just cat file together with #include you don't need to care about it too much. I think a \n at the end is the better way, but it really doesn't matter that much :)

Revision history for this message
Marcus Wichelmann (l-admin-3) wrote :

I would not break the code consistency of the elementary project. Im using scratch and the line at the end of the file will be removed automatically.

Revision history for this message
Raphael Isemann (teemperor) wrote :

It works in connection with the greeter-branch and fixes the issue in a clean way. Just commenting that Dan doesn't have to test both branches and we can merge this fast.

I approve, but Dan has the last word before this gets merged.

Revision history for this message
Raphael Isemann (teemperor) :
review: Approve (testing)
Revision history for this message
Cody Garver (codygarver) wrote :

Propose again if this is still intended for merge

review: Needs Resubmitting

Unmerged revisions

331. By Marcus Wichelmann

Added class to create non-transparent buttons

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gtk-3.0/gtk-widgets.css'
2--- gtk-3.0/gtk-widgets.css 2014-08-02 06:15:01 +0000
3+++ gtk-3.0/gtk-widgets.css 2014-08-02 12:27:52 +0000
4@@ -853,6 +853,26 @@
5 box-shadow: inset -1px 0 alpha (#000, 0.10);
6 }
7
8+/****************************
9+* Non-Transparent Button *
10+****************************/
11+
12+.non-transparent.button {
13+ background-image: linear-gradient(to bottom,
14+ shade (@base_color, 0.93),
15+ shade (@base_color, 0.97)
16+ );
17+}
18+
19+.non-transparent.button:active,
20+.non-transparent.button:hover:active,
21+.non-transparent.button:focus:active {
22+ background-image: linear-gradient(to bottom,
23+ shade (@base_color, 0.93),
24+ shade (@base_color, 0.97)
25+ );
26+}
27+
28 /*********
29 * Linked *
30 *********/
31@@ -2673,4 +2693,4 @@
32 .h3 { font: open sans 12; }
33
34 @import url("granite-widgets.css");
35-@import url("apps.css");
36\ No newline at end of file
37+@import url("apps.css");

Subscribers

People subscribed via source and target branches

to status/vote changes: