fix:env configure remains visible

[adwait-bruno]

Description

JIRA

The dropdown was getting cut off by its container when the terminal panel was open and there were environments to make the list tall. This hid the Configure button at the bottom. Fixed by making the dropdown position itself relative to the whole window instead of its container

Contribution Checklist:

• I've used AI significantly to create this pull request
• The pull request only addresses one issue or adds one feature.
• The pull request does not introduce any breaking changes
• I have added screenshots or gifs to help explain the change if applicable.
• I have read the contribution guidelines.
• Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

Summary by CodeRabbit

• Bug Fixes
  • Improved the environment selector dropdown’s positioning behavior for more consistent placement on the screen.

[coderabbitai]

Walkthrough

The Dropdown component in EnvironmentSelector now passes a popperOptions prop configuring the positioning strategy as 'fixed', a single-line addition affecting popper placement behavior for the environment selector dropdown.

Changes

Environment Selector Dropdown Positioning

Layer / File(s)	Summary
Dropdown popperOptions update packages/bruno-app/src/components/Environments/EnvironmentSelector/index.js	Adds popperOptions={{ strategy: 'fixed' }} to the Dropdown call, changing the popper positioning strategy.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: helloanoop, lohit-bruno, naman-bruno, bijin-bruno

Poem

A tiny prop, a fixed embrace,
The dropdown now knows its place. 🐰
No more drifting, no more sway,
strategy: 'fixed' saves the day!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title matches the fix for keeping the environment Configure action visible in the dropdown.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}