tab longpress fix

[simo6529]

Summary by CodeRabbit

• Refactor
  • Updated dropdown component's internal structure and styling for better layout consistency
  • Fine-tuned responsive design classes and minor spacing adjustments to align with updated design tokens

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request updates a dropdown component to use newer Headless UI composite components, replacing Transition.Root with Transition and Dialog.Panel with DialogPanel. Tailwind class names are adjusted to reflect responsive prefixes, and minor structural reordering is applied to accommodate the updated component hierarchy.

Changes

Cohort / File(s)	Summary
Headless UI Component Migration components/utils/select/dropdown/CommonDropdownItemsMobileWrapper.tsx	Updated Transition and Dialog imports to use composite components (Transition.Root → Transition, Transition.Child → TransitionChild, Dialog → DialogPanel/DialogTitle). Adjusted Tailwind classes including responsive prefixes (lg:desktop-hover utilities) and reordered layout properties (margins, padding, flex/grid). Minor structural changes to close button and content container positioning.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop, a skip, components align,
New composite partners in design,
Transitions smooth and dialogs bright,
This dropdown now feels just right! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'tab longpress fix' is vague and does not clearly describe the actual changes made. The PR primarily updates Headless UI component usage (Transition and Dialog) with new composite components, adjusts class names, and modifies layout structure—none of which directly relate to 'tab longpress' functionality.	Revise the title to accurately reflect the main change, such as 'Update CommonDropdownItemsMobileWrapper to use new Headless UI composite components' or 'Refactor dropdown mobile wrapper with Transition and Dialog updates'.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud