Skip to content

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

Merged
merged 3 commits into from
Sep 28, 2021

Conversation

digi-booster
Copy link
Contributor

…e formats example

@digi-booster
Copy link
Contributor Author

@teoli2003 Can you review this?

Copy link
Collaborator

@rachelandrew rachelandrew left a 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.

@digi-booster
Copy link
Contributor Author

digi-booster commented Sep 13, 2021

You need to repeat any change inside the textarea or it will be overwritten onload.

Is this correct now?

These are very easy to test by opening them in a browser.

How can I do this can you please elaborate?

@digi-booster
Copy link
Contributor Author

Hi @rachelandrew, I waiting for your re-review of this pr.

@rachelandrew
Copy link
Collaborator

To open a file in a browser? Double click the file icon, it will open in the browser.

@@ -30,6 +33,9 @@

<textarea class="playable playable-css" style="height: 150px;">
Copy link
Collaborator

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.

@rachelandrew rachelandrew merged commit e39d727 into mdn:master Sep 28, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants