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

Scrollbar appears in lxqt-config-appearence #1606

Closed
stefonarch opened this issue Nov 2, 2018 · 14 comments
Closed

Scrollbar appears in lxqt-config-appearence #1606

stefonarch opened this issue Nov 2, 2018 · 14 comments

Comments

@stefonarch
Copy link
Member

With recent changes (introducing gtk-styles settings) there is now a scrollbar present in the left sidebar menu. Not sure if this affects all languages/systems.

Possible Solution

Increase window height slightly. I didn't find the file...

schermata-02-09-54-42

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

Adjusting the height/width with respect to a special font/icon size or style is never a good practice. The scroll-bar is a valid part of every GUI -- you shouldn't be frightened by it ;)

screenshot

@tsujan tsujan closed this as completed Nov 2, 2018
@stefonarch
Copy link
Member Author

Well I'm not frightened that's not the problem ;)
The scrollbar here appears for all fonts bigger as 9px the same way, cutting of only a small part at the bottom.
If changing a default window size adding only 5px can avoid it... it's not even needed...

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

Generally, it's wrong to avoid scrollbars in the code -- it isn't really possible and any attempt can make the window too wide/tall for some.

If changing a default window size adding only 5px can avoid it

In your case. 5px, 4px,... all are wrong in the code. Qt designer sets the window width and height depending on who has used it first and with which font/style/icon sizes; even those dimensions may need to be decreased manually.

Last but not least, there's a reason that all toolkits have srollbars.

@Vladimir-csp
Copy link

Not about scrollbar, but I generally find the left pane in lxqt-config components always way too narrow. If its width is governed by some coefficient, could said coefficient be increased a bit?

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

The width is what's needed for showing the text and icon. But we may want a parameter to tell the code about the wrapping position, so that, for example, "Icons Theme" is drawn on a single line but "Icons Theme Is Beautiful" is drawn on 2 lines. It isn't urgent though.

@agaida
Copy link
Member

agaida commented Nov 2, 2018

We could make it wider without problems - would look better even with the short englisch texts - i remember an open bug by PcMan about.

EDIT: found it - #1510
Trying to use nice words: It looks horrible, we should do something about.

@Vladimir-csp
Copy link

Also keep in mind that other l10ns may have more lengthy labels.

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

We could make it wider without problems

Yes, I think so but I still don't know how easy it might be. A reasonable number of characters should be chosen. I think we could find the number in libfm-qt (although it isn't related to that dialog at all).

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

Oh, #1510 shows we're dealing with a real bug here (I didn't know that). A fix may be somehow urgent.

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

Found the cause; will try to fix it later.

@tsujan
Copy link
Member

tsujan commented Nov 2, 2018

OK, I set the max number of characters to ~13 (as in libfm-qt) and got this, which looks better:

13

I don't know how it will look with German or Chinese but using an empty QRect in liblxqtPageSelectWidget::updateMaxTextWidt() was the cause. Is 13 enough or should it be more?

EDIT: Corrected the height too.

@tsujan
Copy link
Member

tsujan commented Nov 3, 2018

Please test lxqt/liblxqt#185 with your language!

@stefonarch
Copy link
Member Author

Perfect now :)
schermata-03-18-08-03

@tsujan
Copy link
Member

tsujan commented Nov 3, 2018

@stefonarch Thanks! So, 13 is enough.

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

No branches or pull requests

4 participants