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

server/apps/theming – standard nc css gets loaded before theming app css #9246

Closed
digsig opened this issue Apr 19, 2018 · 4 comments · Fixed by #31751
Closed

server/apps/theming – standard nc css gets loaded before theming app css #9246

digsig opened this issue Apr 19, 2018 · 4 comments · Fixed by #31751
Labels
1. to develop Accepted and waiting to be taken care of feature: theming papercut Annoying recurring issue with possibly simple fix.

Comments

@digsig
Copy link

digsig commented Apr 19, 2018

Steps to reproduce

  1. load nc without cached browser files
  2. theming app loads a ~1,8Mb background picture
  3. internet connection with 1,5Mb/s download

Expected behaviour

  • theming app's css should be loaded FIRST.
  • standard nc css (nc logo, blue background colour) should be loaded IN NO CASE!

Actual behaviour

  1. standard nc css (nc logo, blue background colour) gets loaded (more or less completely)
  2. after 0.3-1.5 sec it gets replaced with the desired theming app's css (black background, background picture, theming app logo)

Server configuration

Operating system:
debian 9

Web server:
apache2

Database:
mysql (MariaDB)

PHP version:
7.0.27

Nextcloud version:
13.0.1

@MorrisJobke
Copy link
Member

cc @nextcloud/theming

@juliusknorr
Copy link
Member

We are actually still using a static css file "guest.css" on the login page, we'll probably need to check if we can just drop that for a scss based one, since if I remember correctly, we just need this file on the maintenance/install page since we don't have the SCSS cacher ready then. With that we could make sure that we actually don't load the default styles.

Besides that, this seems to be handled differently by different browsers. Firefox shows the default styles, while chromium waits for the theming css to be loaded before rendering the page.

@juliusknorr juliusknorr added 1. to develop Accepted and waiting to be taken care of feature: theming papercut Annoying recurring issue with possibly simple fix. labels Apr 19, 2018
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@MorrisJobke MorrisJobke removed the stale Ticket or PR with no recent activity label Mar 25, 2019
@szaimen

This comment has been minimized.

@juliusknorr

This comment has been minimized.

@skjnldsv skjnldsv linked a pull request Apr 21, 2022 that will close this issue
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of feature: theming papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants