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(core): Checkbox link in label is not clickable #12636

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

khotcholava
Copy link
Contributor

fix(core): Checkbox link in label is not clickable

closes #12577

Description

Updated the checkbox component to ensure that links within custom labels are clickable and function as intended. This resolves the issue observed in the documentation example.

Before

Screencast.from.2024-10-23.16-34-31.webm

After

new.webm

closes [#12577](#12577)

## Description
Updated the checkbox component to ensure that links within custom labels are clickable and function as intended. This resolves the issue observed in the documentation example.
Copy link

netlify bot commented Oct 23, 2024

Deploy Preview for fundamental-ngx ready!

Name Link
🔨 Latest commit bad99fb
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-ngx/deploys/672e8e402c4a0700082d2c87
😎 Deploy Preview https://deploy-preview-12636--fundamental-ngx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@droshev droshev added this to the Sprint 139 - October 2024 milestone Oct 23, 2024
Comment on lines 11 to 13
.fd-checkbox__label-container {
pointer-events: auto;
}
Copy link
Contributor

@InnaAtanasova InnaAtanasova Oct 23, 2024

Choose a reason for hiding this comment

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

you can use:

&-container {
    pointer-events: auto;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, you need to bring this fix to fundamental-styles

closes [#11783](#11783)

- Refactored/simplified css selector for pointer event
Copy link

github-actions bot commented Oct 23, 2024

Visit the preview URL for this PR (updated for commit bad99fb):

https://fundamental-ngx-gh--pr12636-12577-core-fix-check-jw8dradi.web.app

(expires Mon, 11 Nov 2024 22:24:45 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 41b993ee8e451bd7c6770b342ce142dc886eacff

@droshev
Copy link
Contributor

droshev commented Oct 24, 2024

@khotcholava e2e of checkbox are failing. Can you check them?

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.

[BUG] Link in checkbox lable is not clickable
5 participants