Skip to content
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 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

Fixes issue with openInNewTab for DataTable rows #3106

Open
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

olegam
Copy link
Contributor

@olegam olegam commented Feb 22, 2025

Fixes issue where clicking a table row link with openInNewTab=true did not actually open the link in a new tab

Description

There was an issue where setting openInNewTab for a DataTable row would still open the link in the same tab. This can be tested with the example project on the page /tables/new-table/ by clicking a row in the first table. It opens the link (a Google search) in the same tab even though openInNewTab=true is set. This happens because there is an on:click handler on the TableRow that doesn't take the target=_blank property into account.

@archiewood archiewood temporarily deployed to Approval required to run action on external PR February 24, 2025 14:20 — with GitHub Actions Inactive
@archiewood archiewood deployed to Approval required to run action on external PR February 24, 2025 14:20 — with GitHub Actions Active
@archiewood archiewood added the needs-ci Maintainer to review and start CI label Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-ci Maintainer to review and start CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants