Skip to content

Fix: Allow sidebar to scroll on small screen heights. Issue #1643.#1728

Merged
MImran2002 merged 2 commits into
developmentfrom
AC_Side_Bar
Jun 17, 2026
Merged

Fix: Allow sidebar to scroll on small screen heights. Issue #1643.#1728
MImran2002 merged 2 commits into
developmentfrom
AC_Side_Bar

Conversation

@ACBerea

@ACBerea ACBerea commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Issue Description

Fixes for Issue #1643

  • Bottom of sidebar cut off on small screens.

Changes

  • Changed overflow-y to auto in sidebar.css (line 21). Allowed for scrolling on the sidebar.
  • Changed div <id="sidebar"> in sidebar.html to div <id="sidebar" class="bg-dark"> (line 5). Fixed the black background not extending vertically after auto was applied to overflow-y.

Testing

  • Visual testing was done to check if the changed were applied as need. Resizing the window frequently to check for changes.
  • DevTools were used to make live test changes for CSS elements without affecting the system.

…added class=bg-dark to line 5 in sidebar.html to fit the background color to the sidebar image.
Copilot AI review requested due to automatic review settings June 16, 2026 20:59

This comment was marked as spam.

@nahom70 nahom70 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Issue addressed and tested. Sidebar scroll is seamless and works on small screens. The dark background helps maintain consistency.

@DanielRukwasha DanielRukwasha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We approve, the sidebard seems to be working properly, and scrolling also meet the expectation

@github-actions

Copy link
Copy Markdown

View Code Coverage

@MImran2002 MImran2002 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good:
testing when the screen is reduced vertically, when the screen is reduced vertically and horizontally where the sidebar collapsed and when we clicked the hamburger ui and the sidebar pops up the side bar is responsive still and scorllable.

@MImran2002 MImran2002 merged commit d91a10e into development Jun 17, 2026
5 checks passed
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.

5 participants