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

docs(docs-infra): change deprecated modules' headers' style in the hover state #54337

Closed
wants to merge 1 commit into from

Conversation

Ahmed-Hakeem
Copy link
Contributor

@Ahmed-Hakeem Ahmed-Hakeem commented Feb 8, 2024

Because of style specificity, deprecated modules' headers don't have their styles in runtime, especially when hovering over the header.

Before hovering, it has a line-through style.
image

After hovering, it returns back to the underline because of its inconvenient style specificity.
image

Copy link

github-actions bot commented Feb 8, 2024

Deployed aio for d7c29d9 to: https://ng-dev-previews-fw--pr-angular-angular-54337-aio-fc30djq7.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@JeanMeche
Copy link
Member

Hi, thank you for the improvement.

Could you take a look at the failing linting job. We have specific commit message guidelines. Could you reword you commit to docs(docs-infra) ... thank you.

@angular-robot angular-robot bot added the area: docs Related to the documentation label Feb 8, 2024
@ngbot ngbot bot added this to the Backlog milestone Feb 8, 2024
@Ahmed-Hakeem Ahmed-Hakeem changed the title fix: deprecated modules' headers' styles don't take effect while hovering docs(docs-infra): deprecated modules' headers' styles don't take effect while hovering Feb 8, 2024
@Ahmed-Hakeem
Copy link
Contributor Author

Hi, thank you for the improvement.

Could you take a look at the failing linting job. We have specific commit message guidelines. Could you reword you commit to docs(docs-infra) ... thank you.

My pleasure

Thank you for your help ❤️. Now everything is perfect.

@Ahmed-Hakeem Ahmed-Hakeem changed the title docs(docs-infra): deprecated modules' headers' styles don't take effect while hovering docs(docs-infra): change deprecated modules' headers' style in the hover state Feb 8, 2024
@AndrewKushnir AndrewKushnir added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Feb 9, 2024
jessicajaniuk pushed a commit that referenced this pull request Feb 9, 2024
jessicajaniuk pushed a commit that referenced this pull request Feb 9, 2024
@jessicajaniuk
Copy link
Contributor

This PR was merged into the repository by commit 5890f05.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker aio: preview area: docs Related to the documentation target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants