Skip to content

Conversation

@martinkranec
Copy link
Contributor

@martinkranec martinkranec commented Nov 20, 2025

Description

  • Disable copying content from side menu and enable from content part of the page

Implements NAE-2263

Dependencies

No new dependencies were introduced

Third party dependencies

No new dependencies were introduced

Blocking Pull requests

There are no dependencies on other PR

How Has Been This Tested?

Manually

Test Configuration

Name Tested on
OS Windows 11
Runtime Node 22.17.0
Dependency Manager Npm 10.9.2
Framework version
Run parameters
Other configuration

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @...
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • [] I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

Summary by CodeRabbit

  • Style
    • Modified selection behavior in the navigation drawer. The left rail is now selectable, while the right drawer content is now non-selectable.

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

@coderabbitai
Copy link

coderabbitai bot commented Nov 20, 2025

Walkthrough

The deny-select CSS class is repositioned from the left rail container to the right drawer content in the navigation-double-drawer component template, changing text selectability behavior between the two regions.

Changes

Cohort / File(s) Summary
Navigation Double Drawer Template
projects/netgrif-components/src/lib/navigation/navigation-double-drawer/navigation-double-drawer.component.html
Moved deny-select class from left rail container to right drawer content, making the left rail selectable and the right drawer non-selectable

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single file modification with CSS class reassignment
  • No logic, template structure, or functionality changes

Suggested labels

bugfix

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly summarizes the main change: moving copy/selection prevention from the entire application to just the side menu, which aligns with the actual modification of the deny-select class placement.

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.

@coderabbitai coderabbitai bot added the bugfix label Nov 20, 2025
@sonarqubecloud
Copy link

@martinkranec martinkranec changed the title [NAE-2263] V celej aplikacií je nastavené aby sa nedal kopírovať obsah [NAE-2263] Copy/selection prevention is set to to entire application not just for side menu Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants