This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

Fix for issue #73 disabled events (work in progress) #85

Merged

notlee merged 2 commits into Financial-Times:master from orangemug:fix/issue-73-disabled-events

Sep 18, 2019

Contributor

orangemug commented Dec 7, 2016

This is a work in progress pull request for issue #73, raised for early feedback.

Things to do pre-merge

Add a benchmark suite (in separate PR) to check performance
...

orangemug added 2 commits

September 29, 2016 22:59


          Prevent bubbling of events on disabled form elements (issue Financial…

0a805ea

…-Times#73)


          Perf improvement of new event handlig code.

74700f9

notlee self-requested a review

September 18, 2019 12:11

notlee approved these changes

View reviewed changes

Contributor

notlee left a comment

Hey 👋 I notice in the issue this pr is ready for review. It looks good to me; let's get this 1015 day old PR merged!

notlee merged commit 5053ff5 into Financial-Times:master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet