Skip to content

Conversation

@Eetwalt
Copy link
Collaborator

@Eetwalt Eetwalt commented Jan 27, 2026

Summary

Resolves #7003

How did you test this change?

dev tools

Desktop:
image

Mobile:
image

Copilot AI review requested due to automatic review settings January 27, 2026 13:43
@Eetwalt Eetwalt added stylesheets Changes to stylesheets javascript Changes to JavaScript files labels Jan 27, 2026
Copy link
Contributor

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 PR fixes bracket header option display by dynamically adjusting content based on available space, preventing overflow issues.

Changes:

  • Added a CSS class to hide overflowing bracket header options
  • Implemented JavaScript logic to dynamically show/hide header options based on container width
  • Added resize event listener to recalculate header option visibility when viewport changes

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
stylesheets/commons/Brackets.scss Added CSS class for hiding bracket header options
javascript/commons/Bracket.js Implemented dynamic header option display logic with resize handling

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

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

Labels

c: match2 javascript Changes to JavaScript files stylesheets Changes to stylesheets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bracket header options do not get dynamically updated based on column width

5 participants