Skip to content

Add loading attribute for HTMLIframeElement #1221

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
Mar 29, 2025

Conversation

1000ch
Copy link
Contributor

@1000ch 1000ch commented Dec 3, 2021

@github-actions
Copy link
Contributor

github-actions bot commented Dec 3, 2021

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

@1000ch 1000ch force-pushed the loading-attribute-for-iframe branch from 8e699d7 to f86ca91 Compare December 3, 2021 06:01
@1000ch 1000ch marked this pull request as ready for review December 22, 2021 04:24
@1000ch 1000ch force-pushed the loading-attribute-for-iframe branch from 5406a71 to 160700d Compare December 23, 2021 06:48
@1000ch 1000ch force-pushed the loading-attribute-for-iframe branch from 160700d to 1333865 Compare December 23, 2021 06:52
Copy link
Contributor

@Bashamega Bashamega left a comment

Choose a reason for hiding this comment

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

Looks good but please regenerate the baselines.

@1000ch 1000ch requested a review from Bashamega March 29, 2025 10:17
Copy link
Contributor

@Bashamega Bashamega left a comment

Choose a reason for hiding this comment

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

Please run npm i && npm run build && npm run baseline-accept

Copy link
Contributor

@Bashamega Bashamega left a comment

Choose a reason for hiding this comment

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

LGTM :)

@1000ch
Copy link
Contributor Author

1000ch commented Mar 29, 2025

Needs to be reviewed by @saschanaz ?

@Bashamega
Copy link
Contributor

Needs to be reviewed by @saschanaz ?

Yes

@saschanaz
Copy link
Contributor

LGTM, thanks!

@github-actions github-actions bot merged commit a276556 into microsoft:main Mar 29, 2025
5 of 6 checks passed
Copy link
Contributor

Merging because @saschanaz is a code-owner of all the changes - thanks!

@saschanaz
Copy link
Contributor

And thanks @Bashamega for picking up missed PRs!

@Bashamega
Copy link
Contributor

And thanks @Bashamega for picking up missed PRs!

You are welcome

@1000ch 1000ch deleted the loading-attribute-for-iframe branch March 29, 2025 12:13
@1000ch 1000ch restored the loading-attribute-for-iframe branch March 29, 2025 12:13
@1000ch 1000ch deleted the loading-attribute-for-iframe branch March 29, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants