Skip to content
This repository has been archived by the owner on Mar 19, 2018. It is now read-only.

Having a tri-state requireCancelable is overly complex #9

Closed
RByers opened this issue Jul 3, 2015 · 1 comment
Closed

Having a tri-state requireCancelable is overly complex #9

RByers opened this issue Jul 3, 2015 · 1 comment
Labels
spec

Comments

@RByers
Copy link
Member

RByers commented Jul 3, 2015

As issue #7, #5 and #4 make clear, having 3 options for requireCancelable is pretty complicated. Perhaps we can simplify this into a simple boolean 'mayCancel' that indicates only whether or not the handler might call preventDefault.
@RByers RByers mentioned this issue Jul 3, 2015
Closed
RByers added a commit that referenced this issue Jul 3, 2015
@RByers
Replace requiresCancelable with simple boolean mayCancel
23a91c9 
Issue #9
@RByers RByers added the spec label Jul 3, 2015
@RByers
Copy link
Member Author

RByers commented Jul 3, 2015

This seems a lot simpler to me.

@RByers RByers closed this as completed Jul 3, 2015
@RByers RByers mentioned this issue Jul 3, 2015
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
spec
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@RByers