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

Update main element example to use cc0 image #1092

Merged
merged 2 commits into from
Aug 9, 2018

Conversation

a2sheppy
Copy link
Contributor

@a2sheppy a2sheppy commented Aug 3, 2018

Replaced the lizard image used here to a known CC0
one.

Replaced the lizard image used here to a known CC0
one.
@schalkneethling schalkneethling self-requested a review August 7, 2018 13:33
@schalkneethling schalkneethling self-assigned this Aug 7, 2018
@schalkneethling schalkneethling added the enhancement Improves an existing feature. label Aug 7, 2018
Copy link
Contributor

@schalkneethling schalkneethling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One suggestion, other than that. This works! Thanks @a2sheppy

@@ -1,5 +1,5 @@
<header>
<img class="logo" src="/media/examples/lizard.png" />
<img class="logo" src="/media/cc0-images/Gecko--320x213.jpg" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This image is sized really small. I know this is not introduced by this PR but, perhaps you can also go ahead and change the size from:

header > .logo {
    height: 2.5rem;
...

to

header > .logo {
    height: 4.5rem;
...

Copy link
Contributor

@schalkneethling schalkneethling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

@schalkneethling schalkneethling merged commit 2c1b2c0 into mdn:master Aug 9, 2018
wbamberg pushed a commit to wbamberg/interactive-examples that referenced this pull request Aug 31, 2018
* 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants