Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explicitly style ToolButton insensitive state, fixes #4901 #78

Merged
merged 1 commit into from Oct 10, 2015

Conversation

samdroid-apps
Copy link
Contributor

The insensitive state of ToolButtons was implicitly styled, however
newer Gtk+ versions have broken this behaviour.

Thanks to @i5o for discovering the correct opacity value.

Replaces #77

The insensitive state of ToolButtons was implicitly styled, however
newer Gtk+ versions have broken this behaviour.

Thanks to @i5o for discovering the correct opacity value.
@samdroid-apps samdroid-apps added this to the 0.108 milestone Oct 10, 2015
@i5o
Copy link
Contributor

i5o commented Oct 10, 2015

Testing / just in case 😆 /

@i5o
Copy link
Contributor

i5o commented Oct 10, 2015

Tested.
Looks like I forgot about the "92" in "#949294".. :-/
Its "#949494" with the patch.

Honestly I don't see a real difference between those colors.

So, should we merge it? Or wait to calculate the perfeeeeeeect opacity?
Or merge it for a temp solution before we geet the perfect one?

@samdroid-apps
Copy link
Contributor Author

@i5o, is it possible in gtk to change the blue opacity seperatly from the other opacity?

@i5o
Copy link
Contributor

i5o commented Oct 10, 2015

By @samdroid-apps
difference between #949494 and #949294

http://codepen.io/anon/pen/epEOdm/

Honestly as user I don't see the difference.
captura de pantalla_2015-10-10_17-43-42

i5o added a commit that referenced this pull request Oct 10, 2015
Explicitly style ToolButton insensitive state, fixes #4901
@i5o i5o merged commit f1cc110 into sugarlabs:master Oct 10, 2015
@i5o
Copy link
Contributor

i5o commented Oct 10, 2015

merged.

If we want to be really specifics (¿really needed?) we should find a perfect opacity that returns to #949294.

But this works for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants