-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Make Test your skills pages consistent #15579
Conversation
files/en-us/learn/css/building_blocks/selectors/selectors_tasks/index.md
Outdated
Show resolved
Hide resolved
files/en-us/learn/css/building_blocks/test_your_skills_backgrounds_and_borders/index.md
Outdated
Show resolved
Hide resolved
files/en-us/learn/css/building_blocks/test_your_skills_backgrounds_and_borders/index.md
Show resolved
Hide resolved
files/en-us/learn/css/building_blocks/writing_modes_tasks/index.md
Outdated
Show resolved
Hide resolved
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.
Lots of comments, but they are all minor edits that you may or may not agree with. Thank you!!!
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.
Thank you @glmvc for your efforts to keep the content consistent! Just like @estelle, I have added multiple comments, all involving minor edits.
For simplicity, consistency, and maintainability, I've suggested renaming all headings in "Test your skills" to "Task 1", "Task 2", and so on. The title of the page tells us what the task is about.
files/en-us/learn/html/introduction_to_html/test_your_skills_colon__html_text_basics/index.md
Outdated
Show resolved
Hide resolved
...rn/html/multimedia_and_embedding/images_in_html/test_your_skills_colon__html_images/index.md
Outdated
Show resolved
Hide resolved
..._embedding/video_and_audio_content/test_your_skills_colon__multimedia_and_embedding/index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Estelle Weyl <[email protected]>
Co-authored-by: Estelle Weyl <[email protected]>
…s/index.md Co-authored-by: Estelle Weyl <[email protected]>
Co-authored-by: Estelle Weyl <[email protected]>
…unds_and_borders/index.md Co-authored-by: Estelle Weyl <[email protected]>
Co-authored-by: Dipika Bhattacharya <[email protected]>
…/test_your_skills_colon__html_images/index.md Co-authored-by: Dipika Bhattacharya <[email protected]>
…o_content/test_your_skills_colon__multimedia_and_embedding/index.md Co-authored-by: Dipika Bhattacharya <[email protected]>
Co-authored-by: Dipika Bhattacharya <[email protected]>
Co-authored-by: Dipika Bhattacharya <[email protected]>
@estelle, @dipikabh, I made a few more "consistent changes" based on the suggestions and because I went back through all the "test your skills" pages (to be exact in Learn/HTML and Learn/CSS, where the CSS layout modul is still missing, which I will make soon), and I have did the tests too to see if they are working as intended. ps: I'm sorry for the numerous commits, since I am quite new to git and github, I found out a bit later that you can add the suggestions to the batch. 🤯 |
Co-authored-by: Dipika Bhattacharya <[email protected]>
Hi @dipikabh, I also improved the contrast in some interactive examples like it's mentioned in issue #2054 I hope the images fit like this. I tried to make them the same as they were. But I did not know exactly how to make these screenshots, because I cannot get (even with compress) the same file size If the new screenshots don't suit, I will of course update them, but then someone would have to tell me how exactly to do it please... important: note that pr #74 and pr #75 have to be ideally merged at the same time I think? |
Hi @glmvc, not sure what's pending on this PR. Is this waiting on the other PRs you mentioned to land? |
Hey @dipikabh, I've resolved the failed checks. (1 image was not compressed) This pr should ideally be merged at the same time with the following prs from mdn/css-examples: because here in the pr there are changes for them... Sorry for the "crossover" with mdn/css-examples - afterwards this would have been something for a seperate pr 🤦♂️ I made also a seperate pr (#16032) for the discussed "Assessment or further help" sections (to make them also consistent). But the prs are independent from each other. Note pr Make mailto anchor work inside iframe which is also pending, but there is no change here in this pr yet (because I'm waiting for suggestions there). |
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.
Thank you so much @glmvc for your work on this!
@dipikabh you're welcome :) thank you for helping out! 🎉 |
Summary
made "Test your skills!" pages of the Learn/HTML and Learn/CSS modules more consistent:
Motivation
to keep things consistent - i have done this "consistency changes" now a few times in prs and it was always appreciated.
i made a few decisions based on how most test your skills pages (including the ones for Learn/JavaScript...) were structured and written (e.g. the boxes below the interactive editros were written most of the time in the "shorter way", but not for the Learn/CSS part)
Metadata