-
Notifications
You must be signed in to change notification settings - Fork 100
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
Make background white again #2403
Comments
I think it looks better with the light gray background. It also gives a bit more contrast to the tasks. |
@raimund-schluessler I don't think that that contrast is needed. The page is easily readable without a gray background, and I think it doesn't make much sense that we have this in only this one app. |
I also think that white background is cleaner. What about making this configurable if we cannot agree on it? |
I'd agree that the white background is better:
But of course it's your call as maintainer @raimund-schluessler :) Just a configuration option would be overkill here. We just need a good default. |
Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
I think we should go back to a white background. Looks better and it's consistent with other nextcloud apps
Now:
Proposed change:
Describe alternatives you've considered
No response
Additional context
No response
cc @raimund-schluessler @jakobroehrl @nextcloud/designers
The text was updated successfully, but these errors were encountered: