-
Notifications
You must be signed in to change notification settings - Fork 32
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
pref-dialog: add style-class frame #173
Conversation
The GtkNotebook can now address with notebook.frame {} by a gtk+-3 theme
You should really check your travis CI config, |
Did you add that thing with the glade editor or manually? |
with |
Good, because we also use glade and try to avoid manual changes, because next time someone opens the files in glade, it will save them in a different order/format. |
@elboulangero seems to work for me, do you still have the ability to test pnmixer? |
Why did you close this PR? |
Great to hear that, thanks !
Hmm, well I can't say for sure, but I would assume fully featured desktops like MATE use PulseAudio. And this is an issue. PNMixer is an ALSA only mixer, and is known to misbehave when PulseAudio is around. |
Seeing #70, I understand why this is closed. |
Any example of 'lot of themes' ?
I see that you're a developer of this theme. So should I understand that you personally use The commit you propose is a bit weird. A gtk application developer is not supposed to add some random css classes to his widgets just in case some theme developer, somewhere, decided to implement this class. I don't think gtk css is supposed to be used this way. Of course you're free to prove me wrong by providing links on this topic. |
The GtkNotebook can now address with
notebook.frame {}
by a gtk+-3 theme
A lot of themes are using notebook.frame {} for styling a notebook inside a dialog window.
Example BlueSubmarine from MATE desktop.
w/o this PR:
with the PR:
This doesn't have an effect for themes wo doesn't use the frame style-class here, eg. Adwaita.
Btw. i updated pnmixer in fedora to latest version and switched to gtk+-3.
Pnmixer will become the default mixer-applet for the MATE desktop for fedora 28 ;-)
I am using pavucontrol as mixer for fedora.