Closed Bug 1803794 Opened 2 years ago Closed 2 years ago

Add max height to dismiss button in Recently Closed tabs in Firefox View

Categories

(Firefox :: Firefox View, task, P2)

task
Points:
1

Tracking

()

VERIFIED FIXED
109 Branch
Tracking Status
firefox109 --- verified

People

(Reporter: kcochrane, Assigned: kcochrane)

References

Details

(Whiteboard: [fidefe-2022-mr1-firefox-view] )

Attachments

(2 files)

Attached image dismiss-button-too-tall

When a tab title in Recently Closed tabs wraps to a second line, the new dismiss button matches the height of rest of the list item making it appear too tall in these cases. We should add a max-height to the button to prevent this.

Severity: -- → S3
Priority: -- → P2
Attachment #9306451 - Attachment description: Bug 1803794 - Add max-height to dismiss button of Recently Closed tabs in Fx View r=sfoster → Bug 1803794 - Align dismiss button of Recently Closed tabs in Fx View to the center to prevent varying height r=sfoster

Aligning items of the list item to the center resolved this issue without the need for max-height.

Pushed by kcochrane@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e7085edce038 Align dismiss button of Recently Closed tabs in Fx View to the center to prevent varying height r=sfoster
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
Flags: qe-verify+

Reproduced the issue with Firefox 109.0a1 (2022-12-02) on Windows 10x64.
The issue is verified fixed with Firefox 109.0b2 on Windows 10x64, macOS 11, and Ubuntu 20.04. The hover effect for Dismiss tab button is no longer having a bigger height when a tab URL wraps to multiple lines.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: