Skip to content

[DevTools] fix: hide scrollbar for suspense breadcrumbs#167

Closed
everettbu wants to merge 1 commit into
mainfrom
hide-scrollbar-for-suspense-breadcrumbs
Closed

[DevTools] fix: hide scrollbar for suspense breadcrumbs#167
everettbu wants to merge 1 commit into
mainfrom
hide-scrollbar-for-suspense-breadcrumbs

Conversation

@everettbu

@everettbu everettbu commented Dec 12, 2025

Copy link
Copy Markdown

Mirror of facebook/react#34914
Original author: hoxyq


In Chrome, the scrollbar almost takes the whole container size. This is before:
Screenshot 2025-10-19 at 22 49 02

@greptile-apps

greptile-apps Bot commented Dec 12, 2025

Copy link
Copy Markdown

Greptile Overview

Greptile Summary

adds CSS rules to hide scrollbar on .SuspenseBreadcrumbs container in DevTools

  • uses -ms-overflow-style: none for IE/Edge and scrollbar-width: none for Firefox
  • missing ::-webkit-scrollbar rule for Chrome/Safari/WebKit browsers

Confidence Score: 3/5

  • safe to merge after adding WebKit scrollbar hiding rule
  • the change correctly hides scrollbars for IE/Edge and Firefox but incomplete for Chrome/Safari, needs ::-webkit-scrollbar pseudo-element
  • SuspenseTab.css needs WebKit vendor prefix added

Important Files Changed

File Analysis

Filename Score Overview
packages/react-devtools-shared/src/devtools/views/SuspenseTab/SuspenseTab.css 3/5 adds scrollbar hiding for suspense breadcrumbs, but missing WebKit vendor prefix

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment on lines +130 to +131
-ms-overflow-style: none;
scrollbar-width: none;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

syntax: missing ::-webkit-scrollbar { display: none; } for Chrome/Safari

Suggested change
-ms-overflow-style: none;
scrollbar-width: none;
-ms-overflow-style: none;
scrollbar-width: none;
}
.SuspenseBreadcrumbs::-webkit-scrollbar {
display: none;
Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/react-devtools-shared/src/devtools/views/SuspenseTab/SuspenseTab.css
Line: 130:131

Comment:
**syntax:** missing `::-webkit-scrollbar { display: none; }` for Chrome/Safari

```suggestion
  -ms-overflow-style: none;
  scrollbar-width: none;
}

.SuspenseBreadcrumbs::-webkit-scrollbar {
  display: none;
```

How can I resolve this? If you propose a fix, please make it concise.

@everettbu everettbu added the Resolution: Stale Automatically closed due to inactivity label Jan 18, 2026
@everettbu

Copy link
Copy Markdown
Author

Upstream PR was closed or merged. Code is synced via branch mirror.

@everettbu everettbu closed this Jan 25, 2026
@everettbu
everettbu deleted the hide-scrollbar-for-suspense-breadcrumbs branch January 25, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Resolution: Stale Automatically closed due to inactivity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants