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.

Sign up for GitHub

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

Jump to bottom

Notifications resize incorrectly #6430

Closed
1 task done
rezbyte opened this issue Jan 24, 2024 · 0 comments · Fixed by #6758
Closed
1 task done

Notifications resize incorrectly #6430

rezbyte opened this issue Jan 24, 2024 · 0 comments · Fixed by #6758
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done state:tested We tested it and are about to release it ui visual glitches, missing buttons, wrong text...
Milestone
220.240403.0

Comments

@rezbyte
Copy link
Contributor

rezbyte commented Jan 24, 2024 •
edited
Loading

Describe the bug
Making the notifications (the autoresponder notification for example) too small causes the background of the notification to disappear.

To Reproduce
Steps to reproduce the behavior:

  1. Cause a notification to appear
  2. Shrink the window horizontally until the issue occurs

Expected behavior
The notification repositions itself to remain visible.

Screenshots
Screencast from 24.01.2024 09:00:44.webm

Desktop (please complete the following information):

  • OS: Ubuntu
  • Browser Firefox
  • Version 122

Additional context
Video is on the test build but it occurs also on production.

Test Notes

  • When resizing the screen, the notification still looking good and visible
@rezbyte rezbyte added bug broken functionality, usability problems, unexpected errors ui visual glitches, missing buttons, wrong text... labels Jan 24, 2024
@murilopereirame murilopereirame self-assigned this Mar 19, 2024
murilopereirame added a commit that referenced this issue Mar 20, 2024
@murilopereirame
Fix notification resize incorrectly
548534a 
This commit sets the notification width to "fit-content" preventing the
notification from disappearing while resizing the screen.

fix #6430
@murilopereirame murilopereirame linked a pull request Mar 20, 2024 that will close this issue
Fix notification resize incorrectly #6758
Merged
@murilopereirame murilopereirame mentioned this issue Mar 20, 2024
Merged
@charlag charlag added the state:done meets our definition of done label Mar 25, 2024
@charlag charlag added this to the 220.240321.0+ milestone Mar 25, 2024
@murilopereirame murilopereirame removed their assignment Mar 27, 2024
github-merge-queue bot pushed a commit that referenced this issue Mar 27, 2024
@murilopereirame
Fix notification resize incorrectly (#6758)
bb5abba 
This commit sets the notification width to "fit-content" preventing the
notification from disappearing while resizing the screen.

fix #6430
@rezbyte rezbyte self-assigned this Mar 28, 2024
@rezbyte rezbyte added the state:tested We tested it and are about to release it label Mar 28, 2024
@rezbyte rezbyte removed their assignment Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done state:tested We tested it and are about to release it ui visual glitches, missing buttons, wrong text...
Projects
None yet
Milestone
220.240403.0
Development

Successfully merging a pull request may close this issue.

Fix notification resize incorrectly tutao/tutanota
3 participants
@charlag @murilopereirame @rezbyte