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 modal's show method to accept relatedTarget as an argument #33300

Merged
merged 6 commits into from
Mar 16, 2021
Merged

Update modal's show method to accept relatedTarget as an argument #33300

merged 6 commits into from
Mar 16, 2021

Conversation

WinterSilence
Copy link
Contributor

Add new example for bootstrap.Modal.show(relatedTarget)

@XhmikosR
Copy link
Member

XhmikosR commented Mar 8, 2021

What exactly are you trying to improve here?

@WinterSilence
Copy link
Contributor Author

@XhmikosR explore missed parameter

@XhmikosR
Copy link
Member

XhmikosR commented Mar 8, 2021

@rohit2sharma95 please check this out if it's right or now (and feel free to rename the PR title/variables etc )

@rohit2sharma95 rohit2sharma95 changed the title Update modal.md bootstrap.Modal.show(relatedTarget) Add a modal example to accept relatedTarget as an argument Mar 8, 2021
@@ -892,6 +892,13 @@ Manually opens a modal. **Returns to the caller before the modal has actually be
myModal.show()
```

Also, you can pass related target:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, please mention that the DOM element passed as the first argument can be received in the modal events (as the relatedTarget property)

@rohit2sharma95 rohit2sharma95 changed the title Add a modal example to accept relatedTarget as an argument Update modal's show method to accept relatedTarget as an argument Mar 9, 2021
Copy link
Collaborator

@rohit2sharma95 rohit2sharma95 left a comment

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants