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(default-css): Fixing rules for links inside lists, fixing description list - FRONT-4676 #3711

Open
wants to merge 3 commits into
base: v4-dev
Choose a base branch
from

Conversation

planctus
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Nov 11, 2024

@github-actions github-actions bot temporarily deployed to pull request November 11, 2024 12:36 Inactive
@emeryro
Copy link
Contributor

emeryro commented Nov 13, 2024

Just to double check here: the description list has an underlined link. This is is the only case in our examples where the link is underlined in a list. Is it on purpose?
image

@@ -54,7 +54,7 @@ export const Default = () => `
<li>Ordered list</li>
<li>Ordered list
<ol>
<li>Nested ordered list</li>
<li><a class="ecl-link">Nested ordered list</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li><a class="ecl-link">Nested ordered list</a></li>
<li><a class="ecl-link" href="#">Nested ordered list</a></li>

@planctus
Copy link
Collaborator Author

mmh..better asking the designers, this was orginated by a request related to unordered and ordered lists, the description list was not touched, i only fixed some styles in the default css

@github-actions github-actions bot temporarily deployed to pull request November 13, 2024 07:59 Inactive
@planctus planctus removed the Question label Nov 13, 2024
@github-actions github-actions bot temporarily deployed to pull request November 13, 2024 13:02 Inactive
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.

2 participants