Bug: Fix modules dropdown height inconsistency(#8635)

[Mananpatel08]

Description

Adjusted dropdown height styling for Modules field to match other fields like Due date and Cycle, ensuring consistent UI alignment.

Type of Change

• [ x] Bug fix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Improvement (change that would cause existing functionality to not work as expected)
• Code refactoring
• Performance improvements
• Documentation update

Screenshots and Media (if applicable)

Test Scenarios

• Tested locally in development environment
• Verified consistent height across all dropdown fields

References

Fixes #8635

Summary by CodeRabbit

• Style
  • Updated button container dimensions in the module select interface.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

Modified the module-select component to increase the button container height by adding the h-7.5 utility class and adjusted the onChange handler to wrap the module change callback with async void casting, addressing height inconsistency with other dropdown fields.

Changes

Cohort / File(s)	Summary
Module Select Component apps/web/core/components/issues/issue-detail/module-select.tsx	Increased button container height (added h-7.5 class) to match other dropdown fields. Refactored onChange handler to wrap handleIssueModuleChange callback with void cast for async handling. Simplified imports to include only useState.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A button grows just slightly taller,
No longer small, but standing squaller!
With h-7.5 in the mix so right,
The dropdowns dance in perfect height! ✨

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the bug fix for modules dropdown height inconsistency and references the issue number, accurately reflecting the main change.
Description check	✅ Passed	The PR description follows the template with all required sections completed: description, type of change (bug fix), screenshots, test scenarios, and references to issue #8635.
Linked Issues check	✅ Passed	The PR successfully addresses the styling change required by issue #8635: adjusting the Modules dropdown height to match other fields like Due date and Cycle.
Out of Scope Changes check	✅ Passed	The changes are focused on the module-select component styling and remain within scope of fixing the dropdown height inconsistency outlined in issue #8635.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into preview

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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 and usage tips.}

[Mananpatel08]

Hi maintainers, I’ve fixed the modules dropdown height inconsistency and verified it locally. Please let me know if any changes are needed. Thank you!