Skip to content

Fix #2699: Prevent trashcan from being obscured by scrollbar#2868

Open
LuciferDono wants to merge 1 commit into
mozilla:mainfrom
LuciferDono:fix-2699-scrollbar-trashcan
Open

Fix #2699: Prevent trashcan from being obscured by scrollbar#2868
LuciferDono wants to merge 1 commit into
mozilla:mainfrom
LuciferDono:fix-2699-scrollbar-trashcan

Conversation

@LuciferDono

Copy link
Copy Markdown

Before submitting your pull request

  • I agree to license my code under the MPL 2.0 license.
  • I rebased my work on top of the main branch.
  • I ran npm test and all tests passed.
  • I added test coverages if relevant.

Description

When users have "Always show scrollbars" enabled in their OS settings, the trashcan (delete) and reset buttons on assigned site list rows are partly hidden behind the visible scrollbar.

Root cause: .truncate-text used inline-size: calc(100vw - 100px), which bases the width on the full viewport — including the area consumed by the scrollbar. This forced the text span to a fixed width that overflows the flex container when a scrollbar is present, pushing the action buttons behind it.

Fix: Replace the fixed calc() width with min-inline-size: 0, which is the standard CSS fix for allowing flex items to shrink below their content size. The parent <td> already uses display: flex and .menu-text already has flex: 1, so the text span now naturally fills only the remaining space after the favicon and buttons — regardless of whether a scrollbar is visible.

Type of change

  • Bug fix
  • New feature
  • Major change

Fixes #2699

@bakulf

bakulf commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Can you please attach a screenshot before and after? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trashcan partly obscured by scrollbar when managing site list of a container with "Always show scrollbars" option

2 participants