Walkthrough
Adds a position option for Select content (popper default, item-aligned), documents the option with examples and notes, and updates contributor guidance/agent rules about a Soon badge. Runtime: resolves position from props โ appConfig โ default, exposes position to content props, derives isItemAligned, replaces inline value/placeholder markup with RSelectValue, conditionally uses SelectViewport for item-aligned viewports, and adjusts viewportRef exposure to return the underlying element. Theme: moves animation classes into a variants.position config. Playground: example using content.position: 'item-aligned'.
Estimated code review effort
๐ฏ 4 (Complex) | โฑ๏ธ ~45 minutes
๐ฅ Pre-merge checks | โ 3 โ Passed checks (3 passed)| Check name | Status | Explanation |
|---|---|---|
| Title check | โ Passed | The PR title clearly and specifically describes the main change: adding support for item-aligned position mode in the Select component. |
| Description check | โ Passed | The PR description provides relevant context about the bug fix, explaining what changes were made and why they were necessary for Reka UI's layout measurement requirements. |
| Docstring Coverage | โ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
โ๏ธ Tip: You can configure your own custom pre-merge checks in the settings.
โจ Finishing Touches ๐งช Generate unit tests (beta)- Create PR with unit tests
- Commit unit tests in branch
fix/select-item-aligned-position
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.
โค๏ธ ShareComment @coderabbitai help to get the list of available commands and usage tips.