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

Fix visual glitches #896

Merged
merged 2 commits into from
Sep 25, 2023
Merged

Fix visual glitches #896

merged 2 commits into from
Sep 25, 2023

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Sep 21, 2023

📝 Summary

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

@max-nextcloud
Copy link
Collaborator

max-nextcloud commented Sep 21, 2023

Now the background is primary for me in the sidebar but the font color is black (testing with Nextcloud 27):
grafik

Looking at how other apps (talk, contacts) handle this the selected element in the navigation has a slighly brighter background and black font ( in bright mode).

grafik

Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

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

Please fix the App Navigation colors. I think they should match what other apps do or at least combine primary with white color.

@mejo- mejo- force-pushed the fix/visual_glitches branch from fb8f15a to 3c54b73 Compare September 25, 2023 10:38
@mejo- mejo- requested a review from max-nextcloud September 25, 2023 10:38
@mejo-
Copy link
Member Author

mejo- commented Sep 25, 2023

Please fix the App Navigation colors. I think they should match what other apps do or at least combine primary with white color.

Agreed. Actually the issue reported here got already fixed in @nextcloud/vue, so no need for further custom fixes on our side. Removed the corresponding change from this PR.

Signed-off-by: Jonas <jonas@freesources.org>
* Prevent cut off edges in emoji button
* Prevent cut off lower letter parts in title with some fonts

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/visual_glitches branch from 3c54b73 to 38d22c7 Compare September 25, 2023 10:40
@mejo- mejo- merged commit bcc8ffb into main Sep 25, 2023
30 of 34 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/visual_glitches branch September 25, 2023 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Visual glitches in UI
2 participants