fix(select): select placeholder no longer blocks context menu interactions #584

[ldalkee]

fix(select): select placeholder no longer blocks context menu interactions #584

Summary by CodeRabbit

• Style
  • Fixed placeholder interaction handling in select form components to prevent unintended pointer event interception, ensuring proper click behavior across the interface.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 654383d8-fcfd-4df9-8f72-0966f6569869

📥 Commits

Reviewing files that changed from the base of the PR and between a35d686 and 8adf0b7.

📒 Files selected for processing (2)

• src/community/components/form/select/select.module.scss
• src/tedi/components/form/select/select.module.scss

---

📝 Walkthrough

Walkthrough

Two select placeholder styling updates disable pointer events by adding pointer-events: none; to prevent placeholders from capturing pointer/tap interactions in both community and tedi component libraries.

Changes

Cohort / File(s)	Summary
Select Placeholder Styling src/community/components/form/select/select.module.scss, src/tedi/components/form/select/select.module.scss	Added pointer-events: none; CSS property to select placeholder styling in both modules to prevent pointer/tap event capture on the placeholder element.

Possibly related issues

• Right clicking into a select component, which has a placeholder, does not trigger the correct context menu #584: Directly addresses the same objective of making select placeholders non-interactable via pointer-events: none to restore correct context-menu/right-click behavior.

Poem

🐰 A placeholder so meek, now rests from the fray,
No pointer shall bother it, keeping events at bay,
Two select components, now matching in grace,
With pointer-events: none; in their rightful place! ✨

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding pointer-events: none to select placeholders to prevent them from blocking context menu interactions, directly addressing issue #584.
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

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Stylelint (17.5.0)

src/community/components/form/select/select.module.scss

ConfigurationError: Could not find "stylelint-config-recess-order". Do you need to install the package or use the "configBasedir" option?
at getModulePath (file:///usr/local/lib/node_modules/stylelint/lib/utils/getModulePath.mjs:29:9)
at loadExtendedConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:241:21)
at extendConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:208:25)
at async augmentConfigBasic (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:73:20)
at async augmentConfigFull (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:126:24)
at async getConfigForFile (file:///usr/local/lib/node_modules/stylelint/lib/getConfigForFile.mjs:102:26)
at async resolveOptionValue (file:///usr/local/lib/node_modules/stylelint/lib/utils/resolveOptionValue.mjs:27:24)
at async standalone (file:///usr/local/lib/node_modules/stylelint/lib/standalone.mjs:204:19)

src/tedi/components/form/select/select.module.scss

ConfigurationError: Could not find "stylelint-config-recess-order". Do you need to install the package or use the "configBasedir" option?
at getModulePath (file:///usr/local/lib/node_modules/stylelint/lib/utils/getModulePath.mjs:29:9)
at loadExtendedConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:241:21)
at extendConfig (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:208:25)
at async augmentConfigBasic (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:73:20)
at async augmentConfigFull (file:///usr/local/lib/node_modules/stylelint/lib/augmentConfig.mjs:126:24)
at async getConfigForFile (file:///usr/local/lib/node_modules/stylelint/lib/getConfigForFile.mjs:102:26)
at async resolveOptionValue (file:///usr/local/lib/node_modules/stylelint/lib/utils/resolveOptionValue.mjs:27:24)
at async standalone (file:///usr/local/lib/node_modules/stylelint/lib/standalone.mjs:204:19)

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!