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

Change wide-separators and color properties for new Gtk versions #79

Closed
wants to merge 1 commit into from

Conversation

i5o
Copy link
Contributor

@i5o i5o commented Oct 11, 2015

With new gtk version the Separator height has regressed
This patch check the new gtk version and if case its

= 3.16 the properties color, and -GtkWidget-wide-separators
changes to work values.

This is a regression.

Fixes #4902

@@ -3,7 +3,7 @@ import os
import math

gtk_major, gtk_minor, gtk_patch = map(int, gtk.split('.'))
treeview_pseudo_element = gtk_major >= 3 and gtk_minor >= 16
new_gtk = gtk_major >= 3 and gtk_minor >= 16
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One day, new_gtk will be very old. Please use a different variable for different flags

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

regression_gtk maybe?

With new gtk version the Separator height has regressed
This patch check the new gtk version and if case its
>= 3.16 the properties color, and -GtkWidget-wide-separators
changes to new values.
If version < 3.16 the values aren't changed,
this is due to #4175

This is a regression.

Fixes #4902
@samdroid-apps
Copy link
Contributor

Hum, disabling the wide separators isn't helping:

That is wayyy thinner than in the designs.

Plus, I should probs say that IMO, this bug should be WONTFIX. Having the separators go full height is far more like the palette separators. Keeping things the same is good :)

@i5o
Copy link
Contributor Author

i5o commented Oct 11, 2015

Yes, you're right.

Maybe we can keep it as 'WONTFIX' until we found a right solution.

Ignacio Rodríguez
SugarLabs at Facebook
https://www.facebook.com/pages/SugarLabs/187845102582

2015-10-11 0:32 GMT-03:00 Sam notifications@github.com:

Hum, disabling the wide separators isn't helping:

https://camo.githubusercontent.com/fce626a4e7fc71fbb2d1b139cea786393092b3be/68747470733a2f2f692e6779617a6f2e636f6d2f61353966663534613463386634653262336364313634383238663465326636332e706e67

That is wayyy thinner than in the designs.

Plus, I should probs say that IMO, this bug should be WONTFIX. Having the
separators go full height is far more like the palette separators. Keeping
things the same is good :)


Reply to this email directly or view it on GitHub
#79 (comment)
.

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

Successfully merging this pull request may close these issues.

None yet

2 participants