trigger event on input autofill

[lordelix]

Description

Trigger a js event on choosing autofill in input fields

Related Issue

#27096

How Has This Been Tested?

Tested in FF 51 only (so far)

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[mention-bot]

@felixheidecke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975 and @vgezer to be potential reviewers.

[davitol]

• Firefox 51 for OSX
• Safari 9.1.2
• Chrome 56

@felixheidecke

👍 From my POV to merge this PR, but:

• With all browser, cleaning the textbox, the message says email saved ( I feel OK about it)
• Using Safari as browser, using one of the mails saved, the gauge does not work properly, but the mail gets saved.

[PVince81]

This will clash with #27277 where a Save button was added because of sending email confirmation.

[PVince81]

but as this is usable for other fields => 👍

[lordelix]

This will clash with #27277 where a Save button was added because of sending email confirmation.

@PVince81 What's the way to go?

[PVince81]

whoever finishes first gets the merge

[PVince81]

@felixheidecke did your bad luck break travis today ?

Try rebasing

[lordelix]

I tried no such thing today!?

[lordelix]

@davitol … well, this is as good as it gets right now. I could add the blur event also, but this gets fired too often in conjunction with the current event watchers. Checking if the content changed and save depending on the outcome should be a new issue.

@PVince81 fine with you?

[PVince81]

Fine with me if this works with IE11 also

[lordelix]

Fine with me if this works with IE11 also

Japp – works fine ;-)

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.