-
Notifications
You must be signed in to change notification settings - Fork 22.8k
Update index.md with restructuring of the assessment page #12696
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
Conversation
Missing table with Prerequisites & Objective added, Starting point, Hints and tips, Example sections added and text content adjusted accordingly
Preview URLsFlawsURL:
External URLsURL:
(this comment was updated 2022-04-11 00:28:58.693292) |
files/en-us/learn/css/css_layout/fundamental_layout_comprehension/index.md
Outdated
Show resolved
Hide resolved
…ion/index.md Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
There is a conflict to resolve here. |
@teoli2003 oh sorry, should be fine now! I explained to @hamishwillee at the code review why I would leave my changes this way - hasn't replied yet... So, I'm not sure if this is ready to merge? |
Let's wait for his feedback. |
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.
Thanks. Looks good. Sorry for the delay, I didn't see the notification(s).
EDITED. P.S. And thanks for this. I agree that consistency in these kinds of docs is really helpful.
Summary
Motivation
This page did not follow the usual structure of the assessment pages and many things were missing as well or did not make sense.
I hope these changes all fit - I've made an effort to have it set up like the other assessment pages. If I missed something, please let me know and I will adjust it.
Metadata