Closed Bug 1599778 Opened 5 years ago Closed 5 years ago

Info message text visible on top of main panel content while it fades out

Categories

(Firefox :: Protections UI, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 73
Tracking Status
firefox73 --- verified

People

(Reporter: nhnt11, Assigned: ewright)

References

Details

(Whiteboard: [bugday-20191211])

Attachments

(2 files)

STR:

  1. Open the info message in the protections panel
  2. Click the "i" button to close it and observe the text closely.

As the rest of the panel content slides up, it doesn't cover up the text, which stays on top in the layout. The issue is masked due to the opacity decreasing to zero while the panel moves up.

Removing the opacity: 0 rule shows that the text is actually on top of the rest of the panel content. This is solved by setting overflow: hidden on the header container (protections-popup-mainView-panel-header-section).

Erica, do you have time to make a patch for this? I tested the fix I mentioned above using the inspector and it seems straightforward. If not, I can look into it tomorrow. Maybe we can get it into 71, but probably not a big deal if we don't.

Flags: needinfo?(ewright)
Attached image Screenshot

Here's what it looks like mid-animation.

This is way more visible with the dark theme, since in the light theme the panel background and the text are both white.

Assignee: nobody → ewright
Status: NEW → ASSIGNED
Flags: needinfo?(ewright)
Priority: -- → P1
Pushed by ewright@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/67de5253f91c Info message fades below the main content panel. r=nhnt11
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 73
Regressions: 1601051

I have reproduced this bug with Nightly 72.0a1 (2019-11-27) on Windows 10, 64 Bit. The fix of this bug is now verified on latest Nightly 73.0a1!

Build ID : 20191216214733
User Agent : Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:73.0) Gecko/20100101 Firefox/73.0

Status: RESOLVED → VERIFIED
Has STR: --- → yes
Whiteboard: [bugday-20191211]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: