8000 fix: Navigation enabled when selecting rows by tatianainama · Pull Request #14817 · lightdash/lightdash · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: Navigation enabled when selecting rows #14817

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By 8000 clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 15, 2025

Conversation

tatianainama
Copy link
Contributor
@tatianainama tatianainama commented May 14, 2025

Closes: #14784

Description:

  • InfiniteResourceTable: Disable item navigation when selecting rows in table
    • Navigation is still enabled in the item title

Reviewer actions

  • I have manually tested the changes in the preview environment
  • I have reviewed the code
  • I understand that "request changes" will block this PR from merging

@owlas owlas requested a deployment to fix/14784-navigation-enabled-when-selecting - jaffle_db_pg_13 PR #14817 May 14, 2025 16:16 — with Render Abandoned
@owlas owlas deployed to fix/14784-navigation-enabled-when-selecting - headless-browser PR #14817 May 14, 2025 16:16 — with Render Active
Copy link
Contributor

UX suggestion: If we are disabling navigation when selected, the click on item should unselect it probably. Check similar flow Gmail app for example

Copy link
Contributor
@notgiorgi notgiorgi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, LGTM, one UX comment

@tatianainama
Copy link
Contributor Author

UX suggestion: If we are disabling navigation when selected, the click on item should unselect it probably. Check similar flow Gmail app for example

Love it! Will update with the proposal

@owlas owlas temporarily deployed to fix/14784-navigation-enabled-when-selecting - lightdash PR #14817 May 15, 2025 10:24 — with Render Destroyed
@owlas owlas temporarily deployed to fix/14784-navigation-enabled-when-selecting - headless-browser PR #14817 May 15, 2025 10:24 — with Render Destroyed
@tatianainama tatianainama merged commit d4f0c53 into main May 15, 2025
45 of 48 checks passed
@tatianainama tatianainama deleted the fix/14784-navigation-enabled-when-selecting branch May 15, 2025 12:19
lightdash-bot pushed a commit that referenced this pull request May 15, 2025
## [0.1628.1](0.1628.0...0.1628.1) (2025-05-15)

### Bug Fixes

* Navigation enabled when selecting rows ([#14817](#14817)) ([d4f0c53](d4f0c53))
@lightdash-bot
Copy link
Collaborator

🎉 This PR is included in version 0.1628.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Navigation is enabled when selecting items
4 participants
0