Add monochrome tray icon

[SakiiCode]

trafficstars

This PR adds a white tray icon that would be important on Linux to better fit the other icons. It can be enabled/disabled with a checkbox item in the context menu. Uses the electron store to save user preference.

Resolves #757

Signed-off-by: Sakii <[email protected]>

Checklist

• [x] Ensure your code works with manual testing.
• [x] New or updated public/exported symbols have accurate TSDoc documentation.
• [x] Linter and other CI checks pass.
• [x] Sign-off given on the changes (see CONTRIBUTING.md).

[t3chguy]

This needs design review and I think a UI preference flag rather than a context menu option

[americanrefugee]

This needs design review and I think a UI preference flag rather than a context menu option

I need more info / context in order to review this... Are we just talking about showing a white/monochrome Element logo in the MacOS system bar at the top of the screen when in dark mode?

As for the logo itself, I think maybe @JordanConnor would be more knowledgeable about the intended appearance of the logo?

[t3chguy]

@americanrefugee I believe it is predominantly for Linux where it seems to be quite common for apps to have monochrome logos

[JordanConnor]

Is the icon displayed within a specific shape? We prefer to include the roundel in the icon were possible (svg example)

[t3chguy]

That'll depend on exactly what flavour of Linux you are running

[SakiiCode]

I cannot attach screenshots to posts for some reason but you can find them here: https://imgur.com/a/monochrome-element-screenshots-rkxoIZc

I think a UI preference flag rather than a context menu option

That may be the most fancy solution, however this is purely electron-related and involving element-web would make the implementation unnecessarily complex if it's even possible

[t3chguy]

@SakiiCode we already have other Electron-only prefs exposed in the UI preferences

[JordanConnor]

@SakiiCode, thanks for the screenshots, yes at that scale it does feel somewhat bulky. But for consistency we may still want the roundel included, like Steam does for example

[SakiiCode]

Thanks for the feedback, I'll rework this PR

[CLAassistant]

All committers have signed the CLA.

[SakiiCode]

@t3chguy This PR should be ready ~~but I can't seem to be able to create a PR for matrix-react-sdk. The UI preference option is in SakiiCode/matrix-react-sdk/trayicons~~ (Somehow I forked the wrong repo sorry)

[florianduros]

@JordanConnor Can you take a look at the new icons ? thanks!

[JordanConnor]

Icon LGTM!

[SakiiCode]

@florianduros There was one thing I could not test: after getting pinged the red dot might(?) change back the icon to the colored one. Please definitely try this before merging

[florianduros]

@SakiiCode Noted, thanks

Can you please sign the CLA in both PR?