-
Notifications
You must be signed in to change notification settings - Fork 839
add cross browser support for image-set() css ref in alternative image formats example #50
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
…e formats example
@teoli2003 Can you review this? |
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.
You need to repeat any change inside the textarea or it will be overwritten onload.
These are very easy to test by opening them in a browser.
Is this correct now?
How can I do this can you please elaborate? |
Hi @rachelandrew, I waiting for your re-review of this pr. |
To open a file in a browser? Double click the file icon, it will open in the browser. |
images/image-set-type.html
Outdated
@@ -30,6 +33,9 @@ | |||
|
|||
<textarea class="playable playable-css" style="height: 150px;"> |
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.
Have you tested that this does not cause a scrollbar? I think height: 150px will not be tall enough for people to see the code.
…e formats example