Nextcloud-vue: Add ability to tweak the modal backdrop color

Created on 26 Nov 2019  路  7Comments  路  Source: nextcloud/nextcloud-vue

Right now the modal casts this very dark backdrop on the whole page. I think that is good for photo viewing but very aggressive for anything else, as it almost completely hides the UI underneath.

Screenshot_20191126_104549

I think it would make sense to have at least 2 options: this very dark one, and a more clear one, as per material design specs: https://material.io/components/dialogs/#anatomy

What do you think @nextcloud/designers?

1. to develop design modal

Most helpful comment

Yes, I thought we agreed on only using this very dark one only for picture viewing? e.g. the First run wizard and all the normal modals should use a rather light backdrop.

This does not need to be configurable. Basically the default is a light backdrop, and Viewer simply uses a very dark backdrop.

All 7 comments

do it in css? ^^

at least 2 options: this very dark one, and a more clear one

ah right, that would be cleaner indeed :)
please do! :rocket: :heart:

Actually IMO we should make the dark option completely black, as the UI behind the current one is so dark that it's kinda useless and it seems almost an error/unwanted.

Yes, I thought we agreed on only using this very dark one only for picture viewing? e.g. the First run wizard and all the normal modals should use a rather light backdrop.

This does not need to be configurable. Basically the default is a light backdrop, and Viewer simply uses a very dark backdrop.

@jancborchardt so a toggle between the viewer mode and the first run wizard?

@jancborchardt so a toggle between the viewer mode and the first run wizard?

No, as said above:

This does not need to be configurable. Basically the default is a light backdrop, and Viewer simply uses a very dark backdrop.

So everyone uses the default, which is a lighter backdrop. Viewer force overrides it. If we see that any other app needs it, we can make the Viewer backdrop style available to others as well.

Nevermind, https://github.com/nextcloud/nextcloud-vue/pull/750 seems good :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

juliushaertl picture juliushaertl  路  10Comments

roadrunnerjb picture roadrunnerjb  路  5Comments

korelstar picture korelstar  路  7Comments

skjnldsv picture skjnldsv  路  9Comments

raimund-schluessler picture raimund-schluessler  路  9Comments