Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix(ui): resolved active link underline issue#951

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
Mayank2142 wants to merge2 commits intoprocessing:main
base:main
Choose a base branch
Loading
fromMayank2142:main

Conversation

@Mayank2142
Copy link

@Mayank2142Mayank2142 commentedAug 30, 2025
edited by ksen0
Loading

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-de1c6bb5df1ae1e7a619-52b5-4436-8f40-49330c6b0ed3issue:- [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;    }  }}

@ksen0ksen0 requested review fromcoseeian and removed request forcoseeianSeptember 18, 2025 09:11
@ksen0
Copy link
Member

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 freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Image] Excessively long alternative text

4 participants

@Mayank2142@yugalkaushik@coseeian@ksen0

[8]ページ先頭

©2009-2025 Movatter.jp