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

Create an attribute and property for toast closebutton and attach close functionality #17869

Merged
merged 1 commit into from Jul 18, 2019

Commits on Jul 18, 2019

  1. Create an attribute and property for toast closebutton and attach clo…

    …se functionality
    
    This change sets the groundwork for the toast closebutton, by
    adding an attribute to set the closebutton content,
    creating a closeButton property to reflect that attribute,
    and adding an event listener to close the toast when the closebutton is pressed.
    
    This change implements the close button API described in
    (https://github.com/jackbsteinberg/std-toast/blob/eec7728f7082a897d777181ac07b0448062ffca5/README.md),
    and is the subject of debate on issue #48 on the explainer here
    jackbsteinberg/std-toast#48.
    
    Future CLs will add the closeButton option for showToast
    (https://github.com/jackbsteinberg/std-toast/tree/master#showtoastmessage-options)
    Link: https://chromium-review.googlesource.com/c/chromium/src/+/1706453
    
    Bug: 972945
    Change-Id: I5d6c3055791a9f93ef414e575e128c61e2a944ed
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699269
    Reviewed-by: Fergal Daly <fergal@chromium.org>
    Reviewed-by: Kent Tamura <tkent@chromium.org>
    Commit-Queue: Jack Steinberg <jacksteinberg@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#678530}
    Jack Steinberg authored and chromium-wpt-export-bot committed Jul 18, 2019
    Copy the full SHA
    6043e72 View commit details
    Browse the repository at this point in the history