Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

content: 10 modern layouts in 1 line of CSS [2020-06-30] #2871

Closed
kaycebasques opened this issue May 14, 2020 · 5 comments · Fixed by #3263
Closed

content: 10 modern layouts in 1 line of CSS [2020-06-30] #2871

kaycebasques opened this issue May 14, 2020 · 5 comments · Fixed by #3263
Assignees
Labels
frontend new content for new content requests
Milestone

Comments

@kaycebasques
Copy link
Contributor

@kaycebasques kaycebasques added frontend new content for new content requests labels May 14, 2020
@kaycebasques kaycebasques added this to the 2020 Jun 30 milestone May 14, 2020
@stale
Copy link

stale bot commented Jun 13, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. To prevent his from happening, leave a comment.

@stale stale bot added the stale label Jun 13, 2020
@una
Copy link
Contributor

una commented Jun 15, 2020

@stale stale bot removed the stale label Jun 15, 2020
@rachelandrew
Copy link
Collaborator

@una looked at this today in UK time, I noted on the doc the following:

  • I made some suggested edits to make this match the style guide in terms of use of second person and the imperative.
  • It needs links to the MDN info for each property, plus links to browser compat info for the later stuff.
  • There are a few tech corrections / queries.
  • I think your declarations would be much clearer in the context of an entire ruleset with the relevant line highlighted. I changed one to look like that so you can see what I mean.

@una
Copy link
Contributor

una commented Jun 17, 2020

See PR: #3263

  • Adds links to MDN
  • Updates grammar and structure based on suggestions from @rachelandrew
  • Adds ruleset context

@una
Copy link
Contributor

una commented Jul 7, 2020

Just a reminder this is going live tomorrow

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frontend new content for new content requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants