Skip to content

Update onreadystatecomplete.md #3230

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ericregnier
Copy link
Contributor

Added more details on addOnReadyStateComplete and removeOnReadyStateComplete events and added a sample

Added more details on addOnReadyStateComplete and removeOnReadyStateComplete events and added a sample
@JimDaly
Copy link
Contributor

JimDaly commented Mar 30, 2022

@ericregnier
I don't see that we support the addOnReadyStateComplete and removeOnReadyStateComplete methods for the control.
I didn't find them in the documentation. There are methods that exist, but if we don't document them, they aren't supported.

This shows the methods supported for an IFrame Control: iframe control type methods

Am I missing something?

@ericregnier
Copy link
Contributor Author

Hi @JimDaly,
For iFrames, shouldn't the event be supported, as we can set a handler in the classic UI?
For web resources, the documentation states that "A web resource control has the same set of methods available as the iframe control. See iframe control type" and so I assume the same event is supported for web resources. I can confirm that it works (I know it doesn't mean that it's supported thought).

@JimDaly JimDaly assigned sriharibs-msft and unassigned adrianorth Apr 21, 2022
@JimDaly
Copy link
Contributor

JimDaly commented Apr 21, 2022

@sriharibs-msft
Can you comment on whether we can add Eric's change to the docs?

@sriharibs-msft
Copy link
Contributor

This is being driven by @adrianorth and he is best placed to comment. cc @JimDaly

@MicrosoftDocs MicrosoftDocs locked as resolved and limited conversation to collaborators Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants