-
Notifications
You must be signed in to change notification settings - Fork 22.7k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix some CSS ill cased URLs #38077
Fix some CSS ill cased URLs #38077
Conversation
Preview URLs (101 pages)
Flaws (809)URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
External URLs (7)URL:
URL:
URL:
URL:
URL:
URL: [ TRUNCATED! |
This pull request has merge conflicts that must be resolved before it can be merged. |
@@ -462,7 +462,7 @@ | |||
/en-US/docs/CSS/CSS_Reference/Mozilla_Extensions /en-US/docs/Web/CSS/Mozilla_Extensions | |||
/en-US/docs/CSS/CSS_Reference/Webkit_Extensions /en-US/docs/Web/CSS/Webkit_Extensions | |||
/en-US/docs/CSS/CSS_animated_properties /en-US/docs/Web/CSS/CSS_animated_properties | |||
/en-US/docs/CSS/CSS_animations /en-US/docs/Web/CSS/CSS_animations/Using_CSS_animations | |||
/en-US/docs/CSS/CSS_animations /en-US/docs/Web/CSS/CSS_Animations/Using_CSS_animations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure about these changes? See:
slug: Web/CSS/CSS_animations/Using_CSS_animations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These were reported as flaws in my PR: #38076 (comment)
So I thought a quick fix could be submitted. :(
@@ -156,11 +156,11 @@ A **CSS module landing page** provides an overview of the features that the modu | |||
The module landing page serves primarily as a _navigation_ page, but also functions as an at-a-glance _reference_ page for the module. | |||
|
|||
Some related properties and features that belong in other modules, but that are closely related to the functionality offered by the module you are documenting, can also be covered in a _Related concepts_ section. | |||
For example, the `<easing-function>` data type and the `prefers-reduced-motion` media query are not covered in the CSS animations module, but because they are closely related with CSS animations, it is a good idea to highlight them in the [Related concepts](/en-US/docs/Web/CSS/CSS_animations#related_concepts) section of the CSS animations module landing page. | |||
For example, the `<easing-function>` data type and the `prefers-reduced-motion` media query are not covered in the CSS animations module, but because they are closely related with CSS animations, it is a good idea to highlight them in the [Related concepts](/en-US/docs/Web/CSS/CSS_Animationsrelated_concepts) section of the CSS animations module landing page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just driving by and flagging this.
I converted this to a draft when I realized the URLs were getting reported in both cases. You can close this PR if it is no longer relevant. |
I think it's better to close for now, some fixes for this landing in mdn/yari#12603 |
The flaw checker has started reporting the following errors:
The PR fixes above errors.