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

owncloud desktop client tray icon difficult to see #8244

Closed
thoja21 opened this issue Nov 20, 2020 · 15 comments
Closed

owncloud desktop client tray icon difficult to see #8244

thoja21 opened this issue Nov 20, 2020 · 15 comments
Assignees
Labels
Milestone

Comments

@thoja21
Copy link

thoja21 commented Nov 20, 2020

Possibly related to changes made for issue #7356

Expected behaviour

Icon should appear white, or coloured (as before in version 2.6.3).

Actual behaviour

Icon is now dark blue in colour (almost invisible in tray). Toggling monochrome icons only changes the sync status part of the icon. See attached screenshot of icon in Windows status icon tray:

owncloudTrayIcon

Steps to reproduce

  1. Open ownCloud desktop client on Windows 10.
  2. Check ownCloud icon.

Client configuration

Client version: 2.7.1

Operating system: Windows 10 version 2004

OS language: English (United Kingdom)

Installation path of client: default (Program files)

@pnwwebworks
Copy link

I am also experiencing this issue. Windows 10 Pro 20H2 Build 19042.630, OwnCloud client 2.7.1 Build 2560. Prior to the update I was using the colorized icons. Turning on monochrome icons just hides the entire icon as seen above. Changing to light mode for Windows leaves the dark icon (visible of course).

@thoja21 thoja21 changed the title owncloud desktop client difficult to see owncloud desktop client tray icon difficult to see Nov 20, 2020
@h4rp00n33r
Copy link

h4rp00n33r commented Nov 21, 2020

Same experience here (Windows 10 Pro 20H2 Build 19042.630, OwnCloud client 2.7.1 Build 2560):
image

@sagamusix
Copy link

Same here. In general I use the light Windows 10 theme, but with a dark taskbar (as it was default until some recent Windows 10 update, 1909 or 2004?). I suppose the client now simply checks if the light theme is used, instead of whether a light taskbar is used?

@mutipg
Copy link

mutipg commented Nov 23, 2020

I'm confirming this issue too. Dark theme Windows 10.

@jnweiger jnweiger mentioned this issue Nov 23, 2020
63 tasks
@TheOneRing
Copy link
Member

Yes we are only checking UIColorType_Background so if the dark theme has a light taskbar ... meh, will need more research.
Also be aware that the icon will not instantaneously be updated, clicking around on the ui might get you an updated icon.

@pnwwebworks
Copy link

My system is a dark theme with a dark taskbar, so that does not apply. I also have not changed my theme at all vs the software updating and instantly losing the proper icons.

@TheOneRing
Copy link
Member

image
image

@h4rp00n33r
Copy link

Yes, it works correct as long as you choose "pure" themes "dark" or "light". But in this mixed mode (dark windows mode, light app mode) it's broken:
image
image

@pnwwebworks
Copy link

image
image

That's how it should work. But not how it is working.
Screenshot 2020-11-23 111946
Screenshot 2020-11-23 112018

As above, seems to work when the same, but not mixed.

@TheOneRing
Copy link
Member

Ouch, so again we learn: don't trust stackoverflow 🙈

@sagamusix
Copy link

If it's impossible to query the correct value, I wonder if a little outline for the icon would help?

@TheOneRing
Copy link
Member

Looks like rather than using some complex winrt code we just need to check
HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Themes\Personalize\SystemUsesLightTheme

@sagamusix
Copy link

Keep in mind that having separate system and app themes is a relatively new feature in Windows 10, though (as said, I think it was introduced in 1909 or 2004). If the registry key doesn't exist, you should probably fall back to assuming that the taskbar is dark.

@TheOneRing
Copy link
Member

I see 2.7.2 on the horizon

@TheOneRing TheOneRing added this to the 2.7.2 milestone Nov 23, 2020
@TheOneRing TheOneRing self-assigned this Nov 23, 2020
TheOneRing added a commit that referenced this issue Nov 24, 2020
TheOneRing added a commit that referenced this issue Nov 24, 2020
TheOneRing added a commit that referenced this issue Nov 26, 2020
TheOneRing added a commit that referenced this issue Nov 26, 2020
TheOneRing added a commit that referenced this issue Nov 26, 2020
@h4rp00n33r
Copy link

Now everything works perfect again. Thanks, @TheOneRing

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

7 participants