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

Dashboard polishing #22392

Merged
merged 5 commits into from
Aug 26, 2020
Merged

Dashboard polishing #22392

merged 5 commits into from
Aug 26, 2020

Conversation

juliushaertl
Copy link
Member

  • Add loading indicator until widgets are mounted
  • Add class to handle styling when body has been scrolled

@rullzer rullzer mentioned this pull request Aug 24, 2020
@jancborchardt jancborchardt added 3. to review Waiting for reviews design Design, UI, UX, etc. feature: dashboard labels Aug 24, 2020
@jancborchardt jancborchardt added this to the Nextcloud 20 milestone Aug 24, 2020
@jancborchardt jancborchardt force-pushed the bugfix/noid/dashboard-loading-til-moutned branch from 1d39a3b to 4dbdbd0 Compare August 24, 2020 19:27
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the fade below navigation when scrolled, rebased → good to go from my view. :)

@jancborchardt
Copy link
Member

Added another fix for Safari background blur since the -webkit prefix is needed as per https://caniuse.com/#search=backdrop-filter
Also removed one level of indent from the CSS so it’s proper/easier to write.

@nickvergessen
Copy link
Member

Node / build (12.x) (pull_request)

Forgot to build after that @jancborchardt ?

@jancborchardt
Copy link
Member

Node / build (12.x) (pull_request)

Forgot to build after that @jancborchardt ?

Yes, I thought we would use the compile bot for that once it’s all reviewed. Fine anyway :)

@rullzer rullzer mentioned this pull request Aug 25, 2020
19 tasks
juliushaertl and others added 4 commits August 25, 2020 20:30
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
@rullzer rullzer force-pushed the bugfix/noid/dashboard-loading-til-moutned branch from 4a1b01e to d0297d5 Compare August 25, 2020 18:31
@rullzer
Copy link
Member

rullzer commented Aug 25, 2020

/compile amend /

Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@npmbuildbot-nextcloud npmbuildbot-nextcloud bot force-pushed the bugfix/noid/dashboard-loading-til-moutned branch from d0297d5 to e4df8b6 Compare August 25, 2020 18:42
@rullzer rullzer merged commit 4298442 into master Aug 26, 2020
@rullzer rullzer deleted the bugfix/noid/dashboard-loading-til-moutned branch August 26, 2020 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. feature: dashboard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants