Skip to content

fix: disable text selection on sidebar#37

Merged
dastarruer merged 1 commit intomasterfrom
fix/disable-highlighting-on-sidebar
Apr 1, 2026
Merged

fix: disable text selection on sidebar#37
dastarruer merged 1 commit intomasterfrom
fix/disable-highlighting-on-sidebar

Conversation

@dastarruer
Copy link
Copy Markdown
Owner

@dastarruer dastarruer commented Mar 31, 2026

Closes #35

Summary by CodeRabbit

  • Style
    • Enhanced the data layers sidebar to prevent unintended text selection during user interactions.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4562687d-f381-4671-99e8-0805a4277bfa

📥 Commits

Reviewing files that changed from the base of the PR and between 8374585 and 4c1a7ee.

📒 Files selected for processing (1)
  • app/src/lib/components/data-layers/DataLayersSidebar.svelte

📝 Walkthrough

Walkthrough

This PR adds CSS styling to the DataLayersSidebar component to disable text selection on the sidebar element. The change applies user-select: none with appropriate browser prefixes to prevent users from highlighting sidebar text.

Changes

Cohort / File(s) Summary
Sidebar styling
app/src/lib/components/data-layers/DataLayersSidebar.svelte
Added CSS rules to disable text selection on the sidebar aside element using user-select property and browser-specific prefixes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 No more selecting our sidebar's words so fine,
With user-select: none, the layout will align,
The text stays put, no highlighting in sight,
The clicks and drags won't cause a fright,
Our sidebar shines, pristine and tight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: disabling text selection on the sidebar.
Linked Issues check ✅ Passed The pull request directly addresses issue #35 by adding CSS rules to disable text selection on the sidebar element.
Out of Scope Changes check ✅ Passed All changes in the pull request are directly related to the stated objective of disabling text selection on the sidebar.
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/disable-highlighting-on-sidebar

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@dastarruer
Copy link
Copy Markdown
Owner Author

@CodeRabbit review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 1, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@dastarruer dastarruer merged commit 6294f83 into master Apr 1, 2026
5 checks passed
@dastarruer dastarruer deleted the fix/disable-highlighting-on-sidebar branch April 1, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable highlighting on sidebar

1 participant