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

Missing header buttons in tray window with Ubuntu package #1746

Closed
ShevT opened this issue Jan 21, 2020 · 9 comments
Closed

Missing header buttons in tray window with Ubuntu package #1746

ShevT opened this issue Jan 21, 2020 · 9 comments
Milestone

Comments

@ShevT
Copy link

ShevT commented Jan 21, 2020

nextcloud-client 2.7.0-20200121.032623~eoan1

qt_version

nexcloud

@DominiqueFuchs
Copy link
Contributor

Thanks for reporting, @ShevT !

I'm not fully understanding this issue though:

  1. Why is the window so transparent? Hard to see anything
  2. You mean the trayheader buttons lice local folder, apps, right?
  3. Could you retry with the latest version? 2.7.0-20200121.162920~eoan1
  4. Are the buttons functional or completely missing? (E.g. only missing icons but working on click or completely nonexistent)

@ShevT
Copy link
Author

ShevT commented Jan 22, 2020

  1. While taking a screenshot, the window closes. Therefore, I caught a transparent. In the original, it is not transparent.
  2. Yes
    Screenshot_20200122_111745
  3. Same (see screenshot)
  4. All buttons work. Only missing icons.

@DominiqueFuchs DominiqueFuchs changed the title QT 5.12.4 - Missing buttons Missing header buttons in tray window Jan 22, 2020
@DominiqueFuchs
Copy link
Contributor

Thanks again for clarifying! As I can't reproduce this with the app being built directly from source (or the AppImage), I'm assuming this is some problem with the ubuntu beta packages - maybe with the QT::Svg module or asset paths. I'll look into this (and maybe ask @ivaradi for some more info) as soon as we approach a more stable master state, which is not yet the case right now because there are first some other issues that have to be finished (including our package build-chain itself).

Until then, if you want to play around with the current state and test it, you may want to try the current AppImages (see #1744 for more info on that) :-)

@DominiqueFuchs DominiqueFuchs changed the title Missing header buttons in tray window Missing header buttons in tray window with Ubuntu package Jan 22, 2020
@ShevT
Copy link
Author

ShevT commented Jan 22, 2020

Until then, if you want to play around with the current state and test it, you may want to try the current AppImages (see #1744 for more info on that) :-)

Icons are displayed with this version
Screenshot_20200122_151047

@ShevT
Copy link
Author

ShevT commented Feb 6, 2020

Fixed in #1744

@ShevT ShevT closed this as completed Feb 6, 2020
@ShevT ShevT reopened this Mar 3, 2020
@ShevT
Copy link
Author

ShevT commented Mar 3, 2020

After merge # 1744 the problem remained

@misch7 misch7 added this to the 2.7 🌟 UI improvements milestone Mar 3, 2020
@nicolasfella
Copy link
Contributor

I can reproduce this with current master. The icons are only missing when the org.kde.desktop QQC2 style is used, forcing it to e.g. Material with QT_QUICK_CONTROLS_STYLE=Material nextcloud makes them appear

@nicolasfella
Copy link
Contributor

There seem to be a few things broken.

First the org.kde.desktop QQC2 style needs patching to support loading icons from qrc (https://phabricator.kde.org/D28194)

Then the icons are still not visible because the background seems to "cover" it. When I comment out the background assignment I see the icons. Not sure yet whether that's an issue in the KDE QQC2 style or Nextcloud

@er-vin
Copy link
Member

er-vin commented May 6, 2020

So it turns out I've hunting that one today and indeed as @nicolasfella several things are in the way. First there the patching needed to support loading icons from qrc URLs in the org.kde.desktop QQC2 style. But it's only half of the story...

The QQC2 style is done in a way that everything is rendered via a private StyleItem (which delegates to QStyle) set on the background property of the control... Yes, that means that overriding that property basically throws out all the rendering from the org.kde.desktop style...

Honestly, this makes for a very fragile style IMO. I wonder if it should be modified to not use background like that. I wonder if using the contentItem wouldn't make more sense. Anyway, I think this should be fixed on the style side.

In the meantime, my PR #1962 should work around that issue.

@er-vin er-vin closed this as completed May 7, 2020
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

5 participants