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

enhance(macros/CSSInfo): add "Animation type" link #10947

Merged
merged 16 commits into from
Apr 19, 2024

Conversation

mostafaroshdy1
Copy link
Contributor

Summary

This PR added the enchantment in this issue #8721

Problem

The ''Animation type" string was not a hyperlink, so i made it an anchor tag referring to the related page

Solution

  • I have created a file in macros named AnimationType and added the localString as i found them in the official page here (in every other language)

Screenshots

Before

image

After

image


How did you test this change?

Locally using yarn dev

@mostafaroshdy1 mostafaroshdy1 requested a review from a team as a code owner April 18, 2024 01:59
@github-actions github-actions bot added the macros tracking issues related to kumascript macros label Apr 18, 2024
kumascript/macros/CSSInfo.ejs Outdated Show resolved Hide resolved
@caugner caugner changed the title Added hyperlink to "Animation type" enhance(CSSInfo): add "Animation type" link Apr 18, 2024
kumascript/macros/CSSInfo.ejs Outdated Show resolved Hide resolved
@caugner caugner changed the title enhance(CSSInfo): add "Animation type" link enhance(macros/CSSInfo): add "Animation type" link Apr 19, 2024
mostafaroshdy1 and others added 13 commits April 19, 2024 20:47
…on document (mdn#10946)

fix(playground): dispatch readystatechange event
fix(playground): dispatch DOMContentLoaded event on document + bubble
Problem: We identified a drop in search engine impressions in April 2023, which coincides
with the Web/API retitle project (mdn/mdn#284), and we cannot
rule out that there is a causal relationship.

Solution: Run an experiment with 500 randomly sampled Web/API pages (of 2719 affected),
partially reverting  the change to their `<title>` (but not to the `<h1>`) by essentially
replacing `Foo: bar property` with `Foo.bar property`.
The previous test group was not sampled from
the full population of affected Web/API pages.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>
@github-actions github-actions bot added dependencies Pull requests that update a dependency file merge conflicts 🚧 Please rebase onto or merge the latest main. labels Apr 19, 2024
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the merge conflicts 🚧 Please rebase onto or merge the latest main. label Apr 19, 2024
@mostafaroshdy1
Copy link
Contributor Author

mostafaroshdy1 commented Apr 19, 2024

@caugner

I have updated the commits signature ,

i have followed these steps
https://github.com/mdn/yari/actions/runs/8756647461/job/24035231669?pr=10947

yarn.lock Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
@caugner caugner merged commit 40ce610 into mdn:main Apr 19, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file macros tracking issues related to kumascript macros
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants