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

pref-dialog: add style-class frame #173

Closed
wants to merge 1 commit into from
Closed

pref-dialog: add style-class frame #173

wants to merge 1 commit into from

Conversation

raveit65
Copy link

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:
pnmixer-bad

with the PR:
pnmixer-good

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.

The GtkNotebook can now address with
notebook.frame {}
by a gtk+-3 theme
@raveit65
Copy link
Author

You should really check your travis CI config,
Pnmixer builds fine at fedora build server with this patch ;-)
https://kojipkgs.fedoraproject.org//packages/pnmixer/0.7.2/1.fc28/data/logs/x86_64/build.log

@hasufell
Copy link
Collaborator

Did you add that thing with the glade editor or manually?

@raveit65
Copy link
Author

with
[rave@mother ~]$ rpm -qa glade*
glade-3.20.0-3.fc26.x86_64

@hasufell
Copy link
Collaborator

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.

@hasufell
Copy link
Collaborator

@elboulangero seems to work for me, do you still have the ability to test pnmixer?

@raveit65 raveit65 closed this Feb 15, 2018
@hasufell
Copy link
Collaborator

Why did you close this PR?

@elboulangero
Copy link
Collaborator

Btw. i updated pnmixer in fedora to latest version and switched to gtk+-3.

Great to hear that, thanks !

Pnmixer will become the default mixer-applet for the MATE desktop for fedora 28 ;-)

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.

@elboulangero
Copy link
Collaborator

Seeing #70, I understand why this is closed.

@elboulangero
Copy link
Collaborator

@raveit65

A lot of themes are using notebook.frame {} for styling a notebook inside a dialog window.

Any example of 'lot of themes' ?

Example BlueSubmarine from MATE desktop.

I see that you're a developer of this theme. So should I understand that you personally use notebook.frame to theme gtk notebooks ?

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.

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

Successfully merging this pull request may close these issues.

3 participants