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

Calendar entry text may overflow with time #6582

Closed
4 tasks done
armhub opened this issue Feb 20, 2024 · 7 comments · Fixed by #6806
Closed
4 tasks done

Calendar entry text may overflow with time #6582

armhub opened this issue Feb 20, 2024 · 7 comments · Fixed by #6806
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
Milestone

Comments

@armhub
Copy link
Contributor

armhub commented Feb 20, 2024

Please see the screenshot:
screenshot-appointments

The calendar entry title ("Meeting 1 with much more text here") overflows the time indicator.

Test notes

  • The event title does not overflow:
    • at various viewport sizes
    • when multiple events are stacked with the same start times
    • when the event duration is short such as 10 minutes
@armhub armhub added the bug broken functionality, usability problems, unexpected errors label Feb 20, 2024
@marcosdiasalves
Copy link

I am able to reproduce this.
v218.240219.0

@rezbyte rezbyte self-assigned this Mar 26, 2024
rezbyte added a commit that referenced this issue Mar 28, 2024
I also experimented with a CSS flow based `CalendarEventBubble` to
simplify the code but that is still too difficult because of the
absolute positioning and sizing used in `CalendarDayEventsView`.

Closes #6582.
@charlag charlag added the state:done meets our definition of done label Apr 8, 2024
@wrdhub wrdhub added this to the mobile release milestone Apr 10, 2024
github-merge-queue bot pushed a commit that referenced this issue Apr 10, 2024
* Fix the name of events overflowing onto the time

I also experimented with a CSS flow based `CalendarEventBubble` to
simplify the code but that is still too difficult because of the
absolute positioning and sizing used in `CalendarDayEventsView`.

Closes #6582.

* Add a comment to explain `-webkit-line-clamp`

As requested during PR review.
@wrdhub wrdhub self-assigned this Apr 11, 2024
@wrdhub wrdhub added the state:tested We tested it and are about to release it label Apr 11, 2024
@wrdhub wrdhub removed their assignment Apr 11, 2024
@marcosdiasalves
Copy link

Not fixed for me.

WhatsApp Image 2024-04-12 at 17 14 00

Tuta V220.240411.0.
Samsung Galaxy A30 - Android 11 - One UI 3.1

@rezbyte
Copy link
Contributor

rezbyte commented Apr 15, 2024

Not fixed for me.

Hi @marcosdiasalves,
Sorry to hear that you are still experiencing issues.

I will mention this to our team and see what we can do.

murilopereirame pushed a commit that referenced this issue Apr 23, 2024
* Fix the name of events overflowing onto the time

I also experimented with a CSS flow based `CalendarEventBubble` to
simplify the code but that is still too difficult because of the
absolute positioning and sizing used in `CalendarDayEventsView`.

Closes #6582.

* Add a comment to explain `-webkit-line-clamp`

As requested during PR review.
@marcosdiasalves
Copy link

Not yet fixed @rodrigopereirame ! Do not bring this bug to the new standalone calendar app, please.

@wrdhub
Copy link
Contributor

wrdhub commented Sep 4, 2024

@marcosdiasalves I'm sorry that this still happens for you. I was hoping that the solution for other issue might help with it, but it seems like it has not. We have tested on multiple devices, and we simply cannot reproduce this. I am also not aware of other users still having this issue. I saw the reddit post, if more people say they are seeing this too maybe we can find the underlying issue. Updating your webview may help (https://tuta.com/support#web), an old webview has caused some issues in the past.

@marcosdiasalves
Copy link

marcosdiasalves commented Sep 4, 2024 via email

@marcosdiasalves
Copy link

Any news about this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants