Skip to content

Conversation

@brady-lamansky-gtt
Copy link
Contributor

Resolves #180
@cairocoder01 I updated the styling to be default multi-select text color unless the href in the element has a value. Let me know if you have any different suggestions!

@netlify
Copy link

netlify bot commented Jan 2, 2026

Deploy Preview for jade-chebakia-17493f ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 009dcf4
🔍 Latest deploy log https://app.netlify.com/projects/jade-chebakia-17493f/deploys/695e8e510a8ab70008b6ad47
😎 Deploy Preview https://deploy-preview-183--jade-chebakia-17493f.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 project configuration.

@cairocoder01
Copy link
Collaborator

@brady-lamansky-gtt The style looks better, but the link still causes it to show a pointer icon on hover and link to another page if clicked on. To fix that, we can use the CSS pointer-events: none; when we don't want it to act/show as a link. I think you could apply that when the href is "#". And maybe switch the selector for that so the default color is still the link color, but when the href is "#", set the color back to normal and apply that pointer-events style to reset it back to normal text.

text-decoration: none;
color: var(--primary-color, #3f729b);
}
.selected-option a[href="#"] {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since dt-connection inherits dt-tags, I checked that too. It is still showing a link style for an empty href. I think if we just update this CSS selector to .selected-option a[href="#"], .selected-option a[href=""], it should handle both blank hrefs and "#".

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.

dt-location: remove link style if no link

2 participants