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

Menus in Plasma 6 are too big #945

Closed
evgeniy-harchenko opened this issue Feb 29, 2024 · 19 comments
Closed

Menus in Plasma 6 are too big #945

evgeniy-harchenko opened this issue Feb 29, 2024 · 19 comments
Labels

Comments

@evgeniy-harchenko
Copy link

evgeniy-harchenko commented Feb 29, 2024

The menus are too big when they appear for the first time. When they are opened the next time, their size is normal. After restart app menus are big again. Tested on KDE Neon 6 (X11).

Example (video)
menu.mp4
@tsujan
Copy link
Owner

tsujan commented Feb 29, 2024

It should be about a bug in that app (Dolphin?). Have you seen it in a non-KDE Qt6 app — or in any other app?

@evgeniy-harchenko
Copy link
Author

I checked a lot of KDE apps, they all have this issue, even plasmashell (desktop context menu). There is no issue in other non qt6 apps e.g. doublecmd-qt, transmission-qt. A little bit later I'll check non-KDE Qt6 apps.

@tsujan
Copy link
Owner

tsujan commented Feb 29, 2024

To see the problem firsthand, I have to wait until Plasma6 comes to my distro — it will soon.

A little bit later I'll check non-KDE Qt6 apps.

No need to that. I already have many Qt6 apps, and none of them shows this symptom.

@tsujan
Copy link
Owner

tsujan commented Feb 29, 2024

Oh, another question: Do you have the latest git Kvantum?

@evgeniy-harchenko
Copy link
Author

Yes, I have the latest git version. But I decided to reinstall it, for the first time without any success. Then I manually removed all files and installed again. Now it works as expected. So most likely the problem is on my side. Closing.

@tsujan
Copy link
Owner

tsujan commented Mar 1, 2024

Qt 6.6 had a regression, for which I had included a workaround in git Kvantum. That's why I asked about git Kvantum.

Anyway, whenever Plasma6 comes here, I'll check its menus.

Thanks for reporting.

@tsujan
Copy link
Owner

tsujan commented Mar 1, 2024

Let's reopen this until Plasma6 comes here....

@FGD-Garuda
Copy link

FGD-Garuda commented Mar 10, 2024

What is the workaround?

I am running git 1.0.10.r52.ge1c4d44a-1

There are lot of changes (and bugs) in the new Plasma 6, I know that. 😄

Thanks for still maintaining kvantum!

@tsujan
Copy link
Owner

tsujan commented Mar 10, 2024

Thanks for still maintaining kvantum!

You're welcome! Kvantum will be developed indefinitely.

@FGD-Garuda
Copy link

@tsujan
Copy link
Owner

tsujan commented Mar 10, 2024

As for Dolphin's regression in its status-bar (#951 and #955), see #951 (comment)

@evgeniy-harchenko
Copy link
Author

@FGD-Garuda as a temporary fix you can edit *.kvconfig file of your theme. Reduce value of layout_margin in [%General] section. E.g. set it to 5.

P.S. Many thanks @tsujan for Kvantum!

@FGD-Garuda
Copy link

edit *.kvconfig file of your theme. Reduce value of layout_margin in [%General] section. E.g. set it to 5.

Hi!
Reducing that value to 4 on the Sweet theme specifically made it work perfect. I did not see a downside effect to that, meaning I didn't notice anything breaking somewhere else.

Other themes will require a smaller increment reduction because the area was already less cut off.

Tnx!

@tsujan
Copy link
Owner

tsujan commented Mar 11, 2024

This new piece of info might be useful to Dolphin's devs for fixing the regression because layout_margin sets PM_LayoutLeftMargin, PM_LayoutRightMargin, PM_LayoutTopMargin and PM_LayoutBottomMargin.

@evgeniy-harchenko, thanks for finding it; @FGD-Garuda, thanks for confirming it!

However, I'm not sure that the problem will be completely fixed if they consider those QStyle parameters in their codes.

@specialworld83
Copy link

I report my experience on archlinux. Before, there was no way to glue correctly even on Dolphin, see video https://youtu.be/tQ98eETr_2Y

Installed via yay -S kvantum-qt6-git and the problem was resolved immediately.

In fact, with dual monitors it reported that the drop-down menu was not displayed.

@tsujan
Copy link
Owner

tsujan commented Mar 13, 2024

@specialworld83
Yes, that's a multi-screen bug in Qt 6.6, for which I included a workaround in git several months ago:

#if (QT_VERSION >= QT_VERSION_CHECK(6,6,0))
/* WARNING: Because of a bug in Qt 6.6, translucent menus -- especially context menus
-- may be drawn with their minimum sizes and without contents after being
moved on a non-primary screen. As a workaround, the menu is resized here. */
w->resize(g.size());
#endif

The workaround will be in the next release.

@luxl
Copy link

luxl commented Mar 15, 2024

Had the same problem, but it seems that the problem was gone when I switch to using kvantum-git as suggested, at lease not so easy to repeat the problem, tried many times and it is ok. And thanks for maintaining this project, it is awsome!

@tsujan
Copy link
Owner

tsujan commented Mar 15, 2024

@luxl
Thanks for confirming that the menu problem is fixed in git!

@tsujan
Copy link
Owner

tsujan commented Mar 17, 2024

I tested menus of Plasma 6.0.2 with Kvantum 1.1.0 (released an hour ago) and couldn't find a problem. Closing....

@tsujan tsujan closed this as completed Mar 17, 2024
@tsujan tsujan added the Qt bug label Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants