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

fix: OPTIC-885: Links not clickable from within Radio Button #6097

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

bmartel
Copy link
Contributor

@bmartel bmartel commented Jul 15, 2024

PR fulfills these requirements

  • Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
  • Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
  • Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

(check all that apply)

  • Product design
  • Backend (Database)
  • Backend (API)
  • Frontend

Describe the reason for change

The Radio Component allows an extended description to be provided as a label to the element, and when this option is utilized with links embedded in the description JSX the clickHandler would stop propagation of this event from the anchor tag which never allows for the links to be clickable. If the element which sourced the bubbled event is an anchor tag, leave the event alone to retain normal web behaviour and skip the Radio Component clickHandler.

@github-actions github-actions bot added the fix label Jul 15, 2024
Copy link

netlify bot commented Jul 15, 2024

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 62bf966
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/669538920e9a090008800bc0

Copy link

netlify bot commented Jul 15, 2024

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 62bf966
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/66953892273cba00088d5470

Copy link
Contributor

@yyassi-heartex yyassi-heartex left a comment

Choose a reason for hiding this comment

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

we have a tagName === "A"? i didnt realise that was a thing

@bmartel bmartel merged commit 564e6aa into develop Jul 16, 2024
31 checks passed
@bmartel bmartel deleted the optic-885 branch July 16, 2024 13:06
bmartel added a commit that referenced this pull request Jul 16, 2024
* fix: OPTIC-885: Links not clickable from within Radio Button

* add a comment stating this fix needs to be revisited
bmartel added a commit that referenced this pull request Jul 16, 2024
…6102)

* fix: OPTIC-885: Links not clickable from within Radio Button

* add a comment stating this fix needs to be revisited
juliosgarbi pushed a commit that referenced this pull request Jul 16, 2024
* fix: OPTIC-885: Links not clickable from within Radio Button

* add a comment stating this fix needs to be revisited
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants