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]: Increased space between apps and name when hovering the apps menu #46542

Closed
4 of 8 tasks
ChristophWurst opened this issue Jul 15, 2024 · 5 comments · Fixed by #46916
Closed
4 of 8 tasks

[Bug]: Increased space between apps and name when hovering the apps menu #46542

ChristophWurst opened this issue Jul 15, 2024 · 5 comments · Fixed by #46916
Assignees
Labels
4. to release Ready to be released and/or waiting for tests to finish 30-feedback bug design Design, UI, UX, etc. regression
Milestone

Comments

@ChristophWurst
Copy link
Member

⚠️ This issue respects the following points: ⚠️

Bug description

30:
Bildschirmfoto vom 2024-07-15 17-17-12
29:
Bildschirmfoto vom 2024-07-15 17-17-04

Steps to reproduce

  1. Log into Nextcloud daily
  2. Hover the app menu

Expected behavior

Good design

Installation method

None

Nextcloud Server version

master

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

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

Tested with FF on Linux

@ChristophWurst ChristophWurst added bug design Design, UI, UX, etc. 0. Needs triage Pending check for reproducibility or if it fits our roadmap regression 30-feedback labels Jul 15, 2024
@ChristophWurst
Copy link
Member Author

@marcoambrosini @susnux can you confirm or is this intentional?

@marcoambrosini
Copy link
Member

marcoambrosini commented Jul 17, 2024

@ChristophWurst not intentional as far as I know

@ChristophWurst ChristophWurst added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jul 17, 2024
@ChristophWurst ChristophWurst added this to the Nextcloud 30 milestone Jul 17, 2024
@susnux
Copy link
Contributor

susnux commented Jul 18, 2024

Well both there is a feature where apps can set a unread notification which is shown on the app menu then.
The space is needed to not overlay the text with the unread notification.

@ChristophWurst
Copy link
Member Author

Related: we lost the smooth transition

29 30
Bildschirmaufzeichnung vom 2024-07-31 11-27-09.webm Bildschirmaufzeichnung vom 2024-07-31 11-27-30.webm

@jancborchardt
Copy link
Member

Agree with @ChristophWurst, looks strange that both the space is too big and the animation is missing.

@susnux do you have a screenshot how it looks like with the unread notification? Couldn't it just be a circle/bubble (red or otherwise) on the top right of the app icon, like for the notifications icon?

Cc @sorbaugh as this is a high visibility design regression.

@jancborchardt jancborchardt moved this to 👓 Design review in 🖍 Design team Jul 31, 2024
@skjnldsv skjnldsv self-assigned this Jul 31, 2024
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 1. to develop Accepted and waiting to be taken care of labels Jul 31, 2024
@github-project-automation github-project-automation bot moved this from 👓 Design review to 🎉 Done in 🖍 Design team Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish 30-feedback bug design Design, UI, UX, etc. regression
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants