Murmele / Gittyup

Understand your Git history!
https://murmele.github.io/Gittyup
MIT License
1.49k stars 110 forks source link

Font too small on HiDPI monitors #311

Open alensiljak opened 1 year ago

alensiljak commented 1 year ago

Thanks again for the great tool. While using it on a laptop, I have noticed that the small font size used in the interface is causing too much eye strain. I've compared, for example, the menu size to a few other apps, i.e.:

image

or

image

Could this be increased to a comfortable size, please? I see in the screenshots that in Linux the size seems quite fine so it might be that the font in Windows is a bit weird.

exactly-one-kas commented 1 year ago

image

I can't reproduce this with Windows 10 and Windows 11, all menu bars and font sizes are correct

You can however create a new theme from the current one and set a custom font size image image

Does this work for you?

alensiljak commented 1 year ago

Thanks for the feedback! I see what's going on now. Since the laptop has an HD display, it is set to the recommended scaling of 125% (System > Display > Scale). The other apps resize the fonts appropriately but Gittyup stays at the default value.

alensiljak commented 1 year ago

Does this work for you?

As a workaround, yes, this also works. Thank you!

kaushalyap commented 1 year ago

I am experiencing the same but on Fedora.

gittyup-smallfont

RokeJulianLockhart commented 1 year ago

https://github.com/Murmele/Gittyup/issues/311#issuecomment-1575484261

@kaushalyap, that looks correct. For comparison, what size should the text be?

kaushalyap commented 1 year ago

@kaushalyap, that looks correct. For comparison, what size should the text be?

Nope file menus and side bar fonts are small, font on those sections should be bit bigger at least to the size of font in center section.

RokeJulianLockhart commented 1 year ago

https://github.com/Murmele/Gittyup/issues/311#issuecomment-1646838177

@kaushalyap, you genuinely think that they should be as big as the center text...?

kaushalyap commented 1 year ago

@kaushalyap, you genuinely think that they should be as big as the center text...?

Yes and then the center section should become bigger than that to make things correctly proportional.

DKBurns commented 1 year ago

Just to echo that on a 27" 4k monitor the text is really tiny in the commit history, recents, top panels too (at 125% scaling in W11).

In Rstudio IDE, zooming anywhere zooms the editor and the side panels/buttons/etc, but not the menu bar at the top. This seems to work pretty well, and I wonder if similar behaviour might strike a nice balance here?

For now, I will make a custom theme which meets my requirements. I copied the dark theme and then added the below to the bottom of the file, then restarted. this seems to work (thank you for the instructions above - I just picked a random font for now to make it work):

-- font size 
theme["font"] = {
    family = "Arial",
    size = "18"
}
JamesMowery commented 8 months ago

Just gave this app a try. I like the concept, but the interface is way too small. Borderline unusable. Tested on Arch and Ubuntu. Please add a global UI scaling option for those of us who are using HiDPI monitors. Thanks!

HubKing commented 4 months ago

It would probably better to open the new theme.lua file with the default text editor... or at least the add "Locate the file with the file manager" to the editor. Since the theme only applies when restarting the app, I have to keep re-open the options and switch to the Window tab and select "Edit current theme" again and again.

HubKing commented 4 months ago

Just to echo that on a 27" 4k monitor the text is really tiny in the commit history, recents, top panels too (at 125% scaling in W11).

-- font size 
theme["font"] = {
  family = "Arial",
  size = "18"
}

Is there any way to increase the item's height, too? I have set "20" for font, but it seems that item height doesn't grow enough. Lines look crammed.

image

HubKing commented 4 months ago

It seems that the font size is incorrect (too small) if installed from Flatpak, but correct if installed from AUR (Arch User Repository).

RokeJulianLockhart commented 4 months ago

https://github.com/Murmele/Gittyup/issues/311#issuecomment-2099379295

@HubKing, might be best to file that at https://github.com/flathub/com.github.Murmele.Gittyup/issues/new then.

RokeJulianLockhart commented 4 months ago

https://github.com/Murmele/Gittyup/issues/311#issuecomment-2099323937

@HubKing, indeed, they should adapt to the text size. https://forum.qt.io/post/270367 might be relevant here.