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
Implement HTMLDialogElement#close #12352
Comments
Please make a comment here if you intend to work on this issue. Thank you! |
This is assigned to @tallowen, claimed in IRC. |
@tallowen Are you still working on this issue? |
Yes - I went unexpectedly off the grid for a week. Sorry about the delay! On Thu, Jul 21, 2016 at 9:41 AM, Keith Yeung notifications@github.com
Owen Coutts Email: owen@owencoutts.com |
@tallowen mind if I open a PR or are you willing to continue working? |
@KiChjang on it! |
Implements HTMLDialogElement#close <!-- Please describe your changes on the following line: --> Implements HTMLDialogElement#close according to [link](https://html.spec.whatwg.org/multipage/#the-dialog-element:dom-dialog-close) --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12352 - [X] There are tests for these change <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12847) <!-- Reviewable:end -->
Implements HTMLDialogElement#close <!-- Please describe your changes on the following line: --> Implements HTMLDialogElement#close according to [link](https://html.spec.whatwg.org/multipage/#the-dialog-element:dom-dialog-close) --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12352 - [X] There are tests for these change <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12847) <!-- Reviewable:end -->
Implements HTMLDialogElement#close <!-- Please describe your changes on the following line: --> Implements HTMLDialogElement#close according to [link](https://html.spec.whatwg.org/multipage/#the-dialog-element:dom-dialog-close) --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12352 - [X] There are tests for these change <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12847) <!-- Reviewable:end -->
Spec: https://html.spec.whatwg.org/multipage/forms.html#htmldialogelement
Code:
components/script/dom/htmldialogelement.rs
,components/script/dom/webidls/HTMLDialogElement.webidl
Test:
tests/wpt/web-platform-tests/html/semantics/interactive-elements/the-dialog-element/dialog-close.html
The text was updated successfully, but these errors were encountered: