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

Have a light/dark switch in the demo #160

Open
marco-piccolino opened this issue Apr 20, 2015 · 1 comment
Open

Have a light/dark switch in the demo #160

marco-piccolino opened this issue Apr 20, 2015 · 1 comment

Comments

@marco-piccolino
Copy link
Contributor

Instead of having dark and light controls side by side,
how about having a direct switch in the toolbar between light and dark?
This would better highlight any missing styling and improve the naturalness of the demo.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@iBelieve iBelieve changed the title have a light/dark switch in the demo [enhancement] Have a light/dark switch in the demo Apr 20, 2015
@iBelieve iBelieve added this to the Version 0.2 milestone Apr 20, 2015
@iBelieve iBelieve modified the milestones: Version 0.3, Version 0.4 Apr 2, 2016
@ronjouch
Copy link

Hi @marco-piccolino @ibeliever everybody. This demo bug proposes a boolean that style / darkBackground properties of all the demo's components would bind on.

I'm hijacking this bug to ask if, going further than the demo, you already discussed the introduction of an equivalent to qtquick2's "theme" dark/light enum?

I'm fiddling with qml-material after trying qtquick2 and realizing that qtquick2 is 1. young and 2. only focused on mobile for now. Qml-material seems more mature and also supports easily setting theme.{primaryColor, accentColor, backgroundColor} 👍 , but in qml-material there is no property letting me say like in qtquick2 "my app will be mostly dark, so please adjust all widgets to light text/widgets/icons on dark background, not the contrary".

Whereas in qtquick2 saying that is a matter of setting Material.theme: dark once, in qml-material I have to repeatedly specify style: "dark" and darkBackground: true properties everywhere. This results in:

  • Repetition in client code. Unless I'm missing a use case, most apps are either "mostly light" or "mostly dark", so having to always specify dark quickly becomes repetitive if you're unlucky enough to work on a "mostly dark" app (e.g. a photo editor or a visualization tool).
  • Bugs for components failing to provide a darkBackground property. For example, #371 ColorStyle of Textfield hard coded to Theme.light.hintColor

→ Have you already discussed whether a theme: dark/light switch would make sense?

oKcerG referenced this issue Jun 16, 2016

Verified

This commit was signed with the committer’s verified signature.
techmaharaj Atulpriya Sharma
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants