Skip to content

Conversation

Mayank2142
Copy link

@Mayank2142 Mayank2142 commented Aug 30, 2025

Resolves #873

@yugalkaushik
Copy link

@Mayank2142 Can you share the screenshot of screen related to this fix & mention what exactly the issues was.

@Mayank2142
Copy link
Author

bfa71628-7ea2-49b4-8e79-de1c6bb5df1a e1e7a619-52b5-4436-8f40-49330c6b0ed3 issue:- [Link Item] Underline styling is applied to a non-link container instead of the link itself #870

@coseeian
Copy link
Collaborator

With the following SCSS settings, the .current styles are not being applied, even though the a element in the HTML does have the current class. In the browser’s styles panel, there is no trace of the .current class being applied, although it does exist in the compiled CSS file.
 
.linklabel {  a {    color: inherit;    text-decoration: none;    &.current {      text-decoration: underline;      text-decoration-thickness: 2px;      text-underline-offset: 2px;    }  }}

@ksen0 ksen0 requested review from coseeian and removed request for coseeian September 18, 2025 09:11
@ksen0
Copy link
Member

ksen0 commented Sep 18, 2025

Hi @Mayank2142 , thanks for working on this! Just a quick note that I've added a "Resolves issue" comment to your PR - this way it can be linked to the original issue. Also @coseeian have left some comments above, please follow up and update your PR or ask for any help if needed!

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.

[Image] Excessively long alternative text
4 participants