bitcoin-core / gui-qml

Bitcoin GUI (experimental QML-based fork)
MIT License
107 stars 40 forks source link

Introduce Display settings #311

Closed jarolrod closed 1 year ago

jarolrod commented 1 year ago

This moves "Display" related settings into a new "Display" page. Also sets up QSettings wiring for options.

Display Settings Theme Block clock
Screen Shot 2023-05-09 at 9 48 38 PM Screen Shot 2023-04-23 at 2 13 16 AM Screen Shot 2023-04-23 at 2 13 19 AM

Windows Intel macOS Apple Silicon macOS ARM64 Android

GBKS commented 1 year ago

I have to say, this doesn't feel right. Having to go several screens into settings to adjust the size only having to go back to see what it looks like is just not right. And the big one is so huge on desktop that it looks a bit crazy.

Could we also toggle through the sizes via a simple double-tap on the clock? It does require that the UI can differentiate between single and double-taps, not sure if that's built-on or some custom functionality.

Here's a quick mock-up with some ideas (icons are placeholders). I reframed this as two display modes. One is for active use, the other is called "Showcase". Each has an explainer. For "Showcase", there might be other things we may want to tweak, like hiding the Settings button until you interact with the screen. It could also be cool if the big one renders with more detail, more animations or other unique treatments rather than just being bigger.

image

jarolrod commented 1 year ago

Updated from 7626d1f to fd00969, compare

Changes: addressed review feedback

@GBKS I've integrated your suggestion on the "Block Clock display mode" page. But, I've left out the icon until we get an official svg for it (see pictures in PR description). I've also left out the text that you can double tap on the clock; that will be introduced in the followup that implements double tap detection. I've also left out the last sentence in the Showcase option until that functionality is actually introduced.

The values for default and showtime could use review.

jarolrod commented 1 year ago

Updated from from fd00969 to 753d052

Changes: default is now 1/3 as declared by the based on PR, and display is now 1/2. We can fine tune these values later

johnny9 commented 1 year ago

ACK 753d052b4c055559c06023dd7ce781909e3bb51e

GBKS commented 1 year ago

Tested and looks good. I slightly tweaked the icons and made them export ready, find them in Figma here.

image

What do you think of changing "Default" to "Compact"?

Could you also please put the page title in the title bar rather than in the page itself?

And another question about casing. I always use sentence case in design, but I see a good amount of title case in the application. Let's decide on something (obviously I'd go for sentence case). What's your take?

1/2 still feels a tad OP, but let's run with it and see if we want to tweak it after a bit of usage.

jarolrod commented 1 year ago

Updated from from 753d052 to b59654c, compare

Changes:

jarolrod commented 1 year ago

@GBKS as shown in the new PR description images, should the Display settings page itself have a title bar?

GBKS commented 1 year ago

Yes sir, just "Settings".

jarolrod commented 1 year ago

Updated b59654c to 410b135

changes: rebased over main

jarolrod commented 1 year ago

Updated from 410b135 to b0f7265

changes: added "Settings" title to the display settings page

jarolrod commented 1 year ago

cc @GBKS @mouxdesign CI artifacts are built and this can be tested now

jarolrod commented 1 year ago

Updated from b0f7265 to af0ddf6, rebased over main