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

GNOME decorations hover should not use a gradient #72

Open
codic12 opened this issue Apr 3, 2020 · 19 comments
Open

GNOME decorations hover should not use a gradient #72

codic12 opened this issue Apr 3, 2020 · 19 comments
Labels
community enhancement New feature or request

Comments

@codic12
Copy link
Contributor

codic12 commented Apr 3, 2020

The latest Adwaita version does not use a gradient when you hover over a button.

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

I heard about that, it seems this #21 does not apply anymore :(

I think there is some screenshot in some other issue, I'll try to find it

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Oh, it's actually in #65

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

Ah ok ;)

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

Yup
image
Adwaita was updated for a more flat look :D

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

I guess it makes sense to change it if Gnome/Adwaita style users prefer it this way. Would you like to give it a try? It shouldn't be too difficult to change it

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

Sure! Where would I find the relevant files?

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

It's all in drawIconGnome function in breezebutton.cpp

@kupiqu kupiqu added community enhancement New feature or request labels Apr 3, 2020
@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

Hm, I don't know C++. I'll take a look and see what I can do though

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Hm, I don't know C++. I'll take a look and see what I can do though

I don't think you need much C++ knowledge for this purpose, give it a try :-)

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

I'm assuming you can just remove all the gradient references?
And then paint the standard color object instead of a gradient object?

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Yes, just that. I recommend that you make changes one by one and check it on the fly.

And then you can consider whether buttons should be surrounded by circles or squares according to Adwaita...

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Also, wondering whether we should keep previous Gnome style for those who like it, hmm, even though under a different name...

EDIT: but don't worry about it... it won't be difficult to have it back if someone requests it

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

Why not call this one adwaita and the old one 'old adwaita'?

Oh, ok

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Why not call this one adwaita and the old one 'old adwaita'?

Oh, ok

Sure, I am all for it, didn't want to give you too much trouble ;-)

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

:P
I have a school assignment to do;I'll give it a shot after!

@kupiqu
Copy link
Owner

kupiqu commented Apr 3, 2020

Sure, no hurries

@codic12
Copy link
Contributor Author

codic12 commented Apr 3, 2020

got sidetracked. I cannot figure it out :(

@kupiqu
Copy link
Owner

kupiqu commented Apr 4, 2020

Ok, no problem

@kupiqu
Copy link
Owner

kupiqu commented Nov 1, 2021

My aim is that SBE is maintained by the community. This shouldn't be hard to fix. I hope someone in the community cares about SBE and submits a pull request about it... I could help if somebody wants to try but is lost where to start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants