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

[Bug]: Show grid toggle is not fixed and causes style break #37022

Closed
5 of 9 tasks
HannesJo0139 opened this issue Mar 2, 2023 · 7 comments
Closed
5 of 9 tasks

[Bug]: Show grid toggle is not fixed and causes style break #37022

HannesJo0139 opened this issue Mar 2, 2023 · 7 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug needs info

Comments

@HannesJo0139
Copy link

HannesJo0139 commented Mar 2, 2023

⚠️ This issue respects the following points: ⚠️

  • This is a bug, not a question or a configuration/webserver/proxy issue.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • Nextcloud Server is running on 64bit capable CPU, PHP and OS.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

NC version 25.0.4. Fresh setup (linuxserver.io docker container) shows the following behavior. When I scroll down in files list, the button to activate grid view moves out of screen and where it was there is kind of a hole now so you can see part of file list details where it should be barely visible.. See screenshot right under the rounded light corner in the upper right.
I'm using MacOs if that is important info.

Bildschirm­foto 2023-03-02 um 23 14 58

Steps to reproduce

Expected behavior

..

Installation method

None

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

default apps

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@HannesJo0139 HannesJo0139 added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Mar 2, 2023
@HannesJo0139
Copy link
Author

Additional info: It happens on MacOs only. Using Windows the button pos is fixed and everything is fine.

@arnowelzel
Copy link
Contributor

arnowelzel commented Mar 10, 2023

I can confirm that this happens in Windows as well - just clear your browser cache and you will see the error there too (which means Nextcloud also does not make sure that the latest CSS is loaded which is also not ideal).

My fix with CustomCSS:

#view-toggle {
  position: fixed;
}

@HannesJo0139
Copy link
Author

Ok that is really weird. On my Windows 11 Google Chrome the btn is fixed even after deleting browser data..

@arnowelzel
Copy link
Contributor

Ok that is really weird. On my Windows 11 Google Chrome the btn is fixed even after deleting browser data..

Are you sure that you cleared the cache? If Chrome seems to work, try Edge or another browser which never has seen Nextcloud ever before. I can reproduce this at any time - disable the CSS fix, clear cache and the toggle button moves up again when scrolling since its position is clearly not fixed.

@HannesJo0139
Copy link
Author

Hmm, somehow deleting all browser data seems to not delete all browser data lol. You are right. Edge, Firefox, Chrome - It is broken everywhere :)

@arnowelzel
Copy link
Contributor

Hmm, somehow deleting all browser data seems to not delete all browser data lol. You are right. Edge, Firefox, Chrome - It is broken everywhere :)

Be aware that "delete browser data" may not include the relevant part of the cache! You have to double check:

  • the time range has to be set to "all" and not just "last hour"
  • be sure to check the chache option (it should be by default - but just to be sure)

@solracsf
Copy link
Member

solracsf commented May 8, 2023

This has been fixed by #37895
Report back if not (published on next version)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug needs info
Projects
None yet
Development

No branches or pull requests

4 participants