-
Notifications
You must be signed in to change notification settings - Fork 5
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
Please consider reducing widget heights #15
Comments
Thanks for the thorough issue. I'm a bit busy but I'll implement and release the changes within a few days. However, your Geany doesn't have vanilla menu bar and it seems to adjust to the icons. Below is a picture of vanilla Geany (I've shrunk it a little since taking the screenie). After playing with the buttons and menu bar a bit, I think they're going to look something like the following. |
Thanks for taking the feedback. The merged menu bar in Geany had slipped my mind; sorry about that. Anyway, the new design looks perfect - much more consistent and cohesive, and in keeping with the Adwaita Tweaks style. It allows better screen space usage and matches the title bar, apart from giving Adwaita Tweaks a distinctive identity. PS: You should update your screenshot of GNOME Tweak Tool on the home page, to show the new button sizes. |
I tried the latest version, and I find just two things amiss:-
Please see the attached screenshots. |
Glad to hear this. I was experimenting with it and I personally enjoyed the merged look as well. I'll push the changes tonight as I already have them on my computer. I'm usually a little unsure of larger changes like this one, so I really appreciate the feedback.
I completely overlooked the menu entries, I'll look into them tonight or tomorrow morning. Text boxes should be tinier than the defaults, it's the "+" and "-" buttons in your screenshot I completely forgot, I'll fix them as soon as possible. As for the switches, I'll deal with them too. They are the only element that couldn't be shrunk within the code, so I'll have to fire Gimp up soon ;) |
Could I make a suggestion? Is it possible to keep both styles? I am worried the change may impact users with HiDPI-displays negatively. I will of course have to test this to be sure. At the moment the theme works wonderfully on a 13" display with 1080P. Not sure how it will affect higher resolutions as they may be able to scale differently. |
@nooryani84, if it becomes an issue, I'll keep both versions available. I highly doubt it will as I'm merely adjusting the padding of the elements (the empty space around the text). The text will remain the same and the elements will adjust to the text. |
You're probably right, I just have the mindset "don't fix it if it isn't broken" As you say, will probably be fine. :) |
By the way, circular buttons, such as the one for accessing the settings of a connected network in the GNOME 3.24 Wi-Fi settings panel, are (now) squashed. I don't mind the squashed look, and I don't know if it's the result of the new button height reduction, but I thought I should mention it. |
@ocludo, I've pushed the "merged" look now and shrunk down the menus. Spinbuttons and switches are still large because I couldn't find a way to easily shrink them, I'll have to fire up Gimp when I have some free time. |
Thanks. It looks beautiful now. I'll wait for you to update the graphical elements (spinners and switches). |
Will it be possible for you to either reintroduce a slight amount of padding in the text boxes (less than Adwaita's), or thin down their borders? With the thick Adwaita borders the text boxes look crowded when there is no padding. For instance, look at the Properties of a file or folder in Nautilus and notice the text box containing the name. |
@ocludo, yes. Sorry I couldn't fix it today. I have a few hours tomorrow to fix all the issues and make the buttons look all nice and smooth. |
@nooryani84, sure. I wish there was a way to disable the border for some programs (f.ex. Chrome, Geany, Terminal) and enable it for some. Sadly, these things are up to one's taste and choosing the way to go with the theme is quite difficult. Do you want the old, darker gradients back too? If you know how to edit the gtk.css keep reading, if not, just ignore the rest of the text and reply here and I'll add a downloadable variant for you and others who prefer the older look ;) You want to edit the color code for the borders. F.ex. to revert the changes change the two If you want the stronger gradients back. Lower the gradients from Just for clarification, it's these two elements you'd be editing:
|
Padded text boxes any time soon? ;) |
@ocludo textboxes and buttons should be the same as regular Adwaita now, only difference being that they shrink if you make your font smaller (a huge flaw in regular Adwaita, in my opinion). I'll have to completely redo the GTK2 and GTK3 widgets at some point as shrinking them down didn't really work out. At least the menus ended up okay. Unfortunately, I have a ton of work plus a bachelor's thesis due soon and I'll have to get them out of the way first. |
My font is smaller than default, but text boxes and buttons are smaller than Adwaita's (i. e., Adwaita widgets are larger at the same small font size; haven't checked using default/large font size). Anyway, I like the buttons smaller in Adwaita Tweaks, and I've set the text box padding myself in the CSS - it's fine now.
I'll take the current version as final for the time being, then. As I said, I like the widgets small in this theme, since it matches the title and header bars and keeps things consistent. I hope you'll keep the small-widget version available. |
@ocludo, I'll definitely shrink them down when I have the time. Problem was, I was trying to shrink them down in a hurry and with minimal code changes (I named the mod "Tweaks" because it started with ~30 lines of code, most GTK themes are thousands of lines long). Going with general and global changes like |
@Jazqa Thank you for taking time to explain how to revert the changes. I don't want to be a difficult person and this theme should be for as many people to enjoy as possible, so I suppose I will just have to get used to your changes. If I were to manually change the theme back as you explained it will work just fine, but I would be missing out on all your other changes as well. I just have to say thank you for the theme, and perhaps I should fork your theme and make the changes as you suggested. I don't want to create extra work for you :) Edit: If it is possible, can you leave the https://github.com/Jazqa/adwaita-tweaks/releases/download/v1.5/adwaita-tweaks.zip as a separate branch? I have used this now and it works perfect! |
Please consider reducing the size of generic buttons and the height of menu bars, combo (drop-down) boxes, and tab headers. The Adwaita heights don't go well with the Adwaita Tweaks title bar height, especially since CSD header bar buttons in Adwaita Tweaks (such as the path bar and top-right buttons in Nautilus) are already shorter than the former. Please see the attached screenshots
The text was updated successfully, but these errors were encountered: