Skip to content

Comments

fix: Fixed the vertical scroll bar on the home page#1864

Open
Sankar110000 wants to merge 1 commit intoapache:masterfrom
Sankar110000:sankar
Open

fix: Fixed the vertical scroll bar on the home page#1864
Sankar110000 wants to merge 1 commit intoapache:masterfrom
Sankar110000:sankar

Conversation

@Sankar110000
Copy link

Fixed: #1826

Changes:

I have added properties in the customTheme css file to make the scroll-bar hidden

Screenshots of the change:
Screenshot 2025-02-28 125341

Respected Code:
Screenshot 2025-02-28 125318

Copy link
Member

Choose a reason for hiding this comment

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

Too many changes... Could you only adjust the CSS part?

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request attempts to fix issue #1826 by hiding the vertical scrollbar on the home page. The change adds CSS rules to hide scrollbars using WebKit-specific pseudo-elements.

Changes:

  • Added ::-webkit-scrollbar { display: none; } rule to hide scrollbars in WebKit-based browsers

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +108 to +110
::-webkit-scrollbar {
display: none;
}
Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

The current implementation only hides scrollbars in WebKit-based browsers (Chrome, Safari, Edge). Firefox and older browsers won't be affected by this change. For complete cross-browser support, you should also add the standard CSS property scrollbar-width: none; to hide scrollbars in Firefox. Additionally, consider adding -ms-overflow-style: none; for older versions of Internet Explorer and Edge.

Copilot uses AI. Check for mistakes.
Comment on lines +109 to +111
display: none;
}

Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

Hiding scrollbars completely can negatively impact accessibility and user experience. Users who rely on visual scrollbar indicators to understand their position on a page may be confused. Additionally, users with motor impairments who cannot use a mouse wheel may have difficulty scrolling without visible scrollbars. Consider whether this is truly necessary, or if the root cause (unwanted overflow) should be addressed instead by fixing the layout that's creating the scrollbar.

Suggested change
display: none;
}
width: 8px;
height: 8px;
}
::-webkit-scrollbar-thumb {
background-color: rgba(0, 0, 0, 0.3);
border-radius: 4px;
}
::-webkit-scrollbar-track {
background-color: transparent;
}

Copilot uses AI. Check for mistakes.
Comment on lines +108 to +111
::-webkit-scrollbar {
display: none;
}

Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

This solution hides the symptom (scrollbar visibility) rather than addressing the root cause (unwanted overflow). The issue description mentions a vertical scrollbar appearing when it shouldn't, which typically indicates that some element on the page is causing overflow. Hiding the scrollbar doesn't prevent scrolling or fix the underlying layout issue. Consider investigating which element is causing the overflow and fixing the layout instead. You can use browser dev tools to identify overflow-causing elements.

Suggested change
::-webkit-scrollbar {
display: none;
}

Copilot uses AI. Check for mistakes.
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.

[Fix]: Issue with Vertical Side Scroll Bar on Home Page

2 participants