1809491 - Long event location link will overlay dismiss button of Reminder dialog's event item - lost overflow handling

Status: RESOLVED FIXED

(Calendar :: Alarms, defect, P2)

Description

[Arthur K. (he/him)]

Attachment: dismiss-button-allows-link.jpg

STR:

1. Create a new event that fires off in a few minutes and in the Location field, enter a very long but valid URL to something
2. When the event alarm goes off, hover your mouse over the upper portion of the Dismiss button. You'll see the cursor change to a finger indicating it's pointing to a link.
3. Click the link and it will take you to the URL you entered into the Location field

Expected Results:

1. Hovering over and then clicking anywhere on the Dismiss button should dismiss the event

Actual Results:

1. Hovering over the upper portion of the Dismiss button (beneath which is the URL) bypasses the Dismiss button and allows the user to click the link beneath the Dismiss button

In the image, you can see there is transparency to the Dismiss button and the URL is visible beneath it.

Comment 1

[Arthur K. (he/him)]

You'll see this when someone sends you a Zoom link meeting and the meeting link is in the Location field

Comment 2

[Thomas D. (:thomas8)]

Thanks Arthur. Confirming for Beta 109.0b4 (64-bit), Win10m, and Daily 110.0a1 (2023-01-09) (64-bit). Not seen on Release 102.6.1 (64-bit) which still has overflow handling with ellipsis -> regression.

Redirecting Dismiss to the location link URL makes for a pretty bad reminder experience for everyday scenarios like Zoom invites, in addition to all the other reminder issues which we have... should fix this asap.

Alice, would you be in the mood for finding the regression window?

Comment 3

[Richard Marti (:Paenglab)]

Attachment: Bug 1809491 - Let the location description flex in the calendar reminder dialog. r=aleca

Comment 4

[Thomas D. (:thomas8)]

Awesome - as Richard has offered a patch following my triage, regression-window no longer needed.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/3e9b03c26236
Let the location description flex in the calendar reminder dialog. r=aleca

Comment 6

[Arthur K. (he/him)]

Attachment: date-time-overlayed.png

Was the fix for this supposed to have landed in 110.0b1? If os, it's not working. I noticed another related/unrelated issue here that may require another bug to be opened for it..

I'm also now seeing that whenever I experience bug 1786159, the event/alarm window's column that contains the "Snooze For" and "Dismiss" buttons will overlay the text (of an event that exists below it) of the event time over the title of the event. See the image for what I mean.

Shall I open a new bug or is this related to this bug as well?

Comment 7

[Arthur K. (he/him)]

Attachment: scroll-beyond-border.png

Sigh. If the alarm window is shrunk and the bottom scroll bar is revealed, I also appear to be able to scroll rightward beyond the right-hand border of the 2nd column and reveal white space. See image. That seems not right. Shall I open a separate bug for this as well?

Comment 8

[Arthur K. (he/him)]

Looks good on 110.0b2. Still seeing the stuff from comment 6 and comment 7. Open new bugs for each?

Comment 9

[Arthur K. (he/him)]

I created bug 1814203, bug 1814211 and bug 1814218 as they seem like they might be related to this one.