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

Quota link feature broken in 26 #457

Closed
nickvergessen opened this issue Apr 5, 2023 · 4 comments · Fixed by #541
Closed

Quota link feature broken in 26 #457

nickvergessen opened this issue Apr 5, 2023 · 4 comments · Fixed by #541

Comments

@nickvergessen
Copy link
Member

Steps to reproduce

  1. Create a link with position "User quota"
    grafik

Expected behaviour

Personal settings and files app left sidebar should link to the configured page
Bildschirmfoto vom 2023-04-05 19-39-31
Bildschirmfoto vom 2023-04-05 19-39-45

Actual behaviour

JS magic is in place

@skjnldsv
Copy link
Member

skjnldsv commented Apr 6, 2023

Ah, wasn't even aware something like that existed! :/

@provokateurin
Copy link
Member

The link in the personal settings is still there, it's just way down the page and looks very odd. I'm not sure how it was supposed to work before and why it is broken now. In my digging I couldn't find a way to put the links directly below the quota, I assume it only worked because there weren't so many entries in the personal settings before?

@provokateurin
Copy link
Member

The two options I see to fix this are:

  1. Completely remove the external sites from the personal info page because they can't be displayed directly below the quota
  2. Add a new proper section in the personal info, but accept that it's not near the quota.

@provokateurin
Copy link
Member

provokateurin commented Aug 21, 2023

image
in 25 it is definitely already broken and it is also at the bottom of the personal info page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants