Skip to content

menu_story: add min_w to popup-menu to avoid resize on shortcut load#2320

Merged
madcodelife merged 1 commit intolongbridge:mainfrom
BeratHundurel:fix/popup-menu-resizing
Apr 30, 2026
Merged

menu_story: add min_w to popup-menu to avoid resize on shortcut load#2320
madcodelife merged 1 commit intolongbridge:mainfrom
BeratHundurel:fix/popup-menu-resizing

Conversation

@BeratHundurel
Copy link
Copy Markdown
Contributor

Description

Keyboard shortcuts in the popup menu caused the component to resize after opening. This change adds a minimum width to prevent the layout shift and keep the popup menu size stable.

Screenshot

Before

Ekran.Kaydi.2026-04-30.051436.mp4

After

Ekran.Kaydi.2026-04-30.051511.mp4

Break Changes

None

Checklist

  • I have read the CONTRIBUTING document and followed the guidelines.
  • Reviewed the changes in this PR and confirmed AI generated code (If any) is accurate.
  • Passed cargo run for story tests related to the changes.
  • [] Tested macOS, Windows and Linux platforms performance (if the change is platform-specific)

@madcodelife
Copy link
Copy Markdown
Member

Thanks.

@madcodelife madcodelife merged commit 63c7c77 into longbridge:main Apr 30, 2026
3 checks passed
AzureZee pushed a commit to AzureZee/gpui-component that referenced this pull request Apr 30, 2026
…ongbridge#2320)

## Description

Keyboard shortcuts in the popup menu caused the component to resize
after opening. This change adds a minimum width to prevent the layout
shift and keep the popup menu size stable.

## Screenshot

### Before


https://github.com/user-attachments/assets/b535fc43-c1eb-4b93-b108-cc7f01c16371

### After

https://github.com/user-attachments/assets/c5ee83b4-7ff6-4512-a0cb-9098b99cf78e

## Break Changes
None

## Checklist

- [x] I have read the [CONTRIBUTING](../CONTRIBUTING.md) document and
followed the guidelines.
- [x] Reviewed the changes in this PR and confirmed AI generated code
(If any) is accurate.
- [x] Passed `cargo run` for story tests related to the changes.
- [] Tested macOS, Windows and Linux platforms performance (if the
change is platform-specific)
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.

2 participants