-
Notifications
You must be signed in to change notification settings - Fork 501
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
Update cc0 figcaption #1091
Update cc0 figcaption #1091
Conversation
Replaced the hamster with an elephant at sunset. Updated the CSS to work with the new image, and replaced the figure's caption text.
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.
One minor change required, other than that, this looks great! Thanks @a2sheppy
@@ -4,10 +4,10 @@ figure { | |||
flex-flow: column; | |||
margin: 0; | |||
padding: 5px; | |||
width: 22em; |
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.
I would suggest making this width: 19em;
as 22
causes the output area to scroll vertically.
While it doesn't require scrolling for me, it's close, so I presume it's due to local font configuration differences. I will submit an update momentarily. |
where font sizes are just a little bigger than for me.
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.
r+
* upstream/master: Add window.ieConfig.origin for the target origin (mdn#1112) Improve example code (mdn#1111) Adding an example for the String.substring() method (mdn#1074) Adding an example for String.slice() (mdn#1081) Adding an example of String.split() (mdn#1071) Adding an example for the String.substr() method (mdn#1073) chore(deps): update dependency jest to v23.5.0 (mdn#1109) chore(deps): update dependency puppeteer to v1.7.0 (mdn#1108) Estelle/morestrings (mdn#1095) Added method for String[@@iterator]() (mdn#1069) Replace header image with a CC0 one (mdn#1100) chore(deps): update dependency prettier to v1.14.2 (mdn#1107) chore(deps): update dependency clean-css to v4.2.1 (mdn#1096) Update main element example to use cc0 image (mdn#1092) Update cc0 figcaption (mdn#1091) Fix meta.json for contenteditabl (mdn#1106)
Replaced the hamster image (which was free but not expressly labeled cc0) with a silhouetted elephant. Wrote a new caption, and updated the CSS to work with the shape of the image better.