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

gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar display #1911

Merged

Conversation

jamescowens
Copy link
Member

This PR makes some stylesheet adjustments to make the menu bar consistent, fix truncation of the side toolbar icon labels, and also scale the side toolbar icons on HiDPI displays. Further work will need to be done on the stylesheets as they still are quite bizarre.

@jamescowens jamescowens self-assigned this Oct 5, 2020
@jamescowens jamescowens added this to the Gladys milestone Oct 5, 2020
@jamescowens
Copy link
Member Author

Before....
image

After... (this also includes the new fonts and changes in #1908.
image

@opsinphark
Copy link
Contributor

Glad you figured out where the stylesheet issues, it was driving me mad.
Looks good!

@jamescowens jamescowens merged commit 19725b5 into gridcoin-community:development Oct 5, 2020
@jamescowens
Copy link
Member Author

Apparently there is a bug of some sort in the border (or border-style): none. When you use that, it truncates the labels, like it is using a ridiculously thick invisible border. Unfortunately getting rid of the none creates a small visible border on the "light" stylesheet, but it is better than truncating the labels.

I also implemented a simple scaling rule for the icons. I did the same thing for the field widths or something similar in the peers tab when I did that a while back. Seems to work pretty well. This will get us by until you do some more thorough house cleaning, if you want, @opsinphark!

@opsinphark
Copy link
Contributor

I'm poking around a bit to see what I want to conquer next.

@jamescowens jamescowens changed the title Adjust stylesheets and scale icons to improve HiDPI side toolbar dispay gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar dispay Oct 8, 2020
@jamescowens jamescowens changed the title gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar dispay gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar display Oct 8, 2020
jamescowens added a commit that referenced this pull request Oct 8, 2020
…cyrossignol)

 - refactor: port chainparams #1878 (@div72)
 - gui: Update default font to Inter-Regular and console font to Inconsolata (@opsinphark, @jamescowens)
 - gui: Add "review beacon verification" button to wizard summary page #1912 (@cyrossignol)
 - rpc, wallet: Implement liststakes #1909 (@jamescowens)
 - rpc: Add "getlaststake" RPC function #1913 (@cyrossignol)
 - gui: Install bold variant of Inter font #1914 (@cyrossignol)

 - refactor: Consolidate Gridcoin-specific code #1894 (@cyrossignol)
 - script: Setup improvements #1895 (@nathanielcwm)
 - gui: Diagnostics refresh #1899 (@jamescowens)
 - superblock: Optimize superblock size calculation #1906 (@cyrossignol)
 - gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar display #1911 (@jamescowens)
 - doc: Tell user to disable win32 application support in WSL (for building) #1917 (@nathanielcwm)
 - rpc: Revise and expand help for beaconconvergence rpc call #1918 (@jamescowens)
 - scheduler: Increase default update check interval to 5 days #1920 (@cyrossignol)
 - gui: Prevent multiple dialogs from being open at the same time #1922 (@scribblemaniac)

 - refactor: Clean up remaining legacy timer code #1892 (@cyrossignol)

 - build: Add --without-brotli option to curl.mk #1902 (@G_UK)
 - test: Remove fs_tests... file after the fs test #1903 (@div72)
 - util, gui: Fix shutdown segfault and repair broken overview page staking status #1901 (@jamescowens)
 - scraper: Fix order of destruction for global scraper objects #1904 (@cyrossignol)
 - scraper: Fix global object destruction order for MacOS #1905 (@cyrossignol)
 - util: Decouple out-of-sync state from block acceptance #1921 (@cyrossignol)
@jamescowens jamescowens deleted the toolbar_fixes branch November 7, 2020 20:44
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.

3 participants