Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Implement hide-autofill-popup #5287

Merged
merged 1 commit into from
Nov 7, 2016
Merged

Implement hide-autofill-popup #5287

merged 1 commit into from
Nov 7, 2016

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Oct 31, 2016

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

fix #3816
fix #4540
fix #5023

Auditors: @bbondy

Test Plan:

  1. Go to https://github.com/brave/browser-laptop/issues

  2. In the filters textbox, start typing characters until autocomplete pops up

  3. Push enter to submit

  4. Scroll the page should dismiss the popup

  5. Go to https://github.com/brave/browser-laptop/issues

  6. In the filters textbox, start typing characters until autocomplete pops up

  7. Push enter to submit

  8. Popup should be dismissed

fix brave#3816
fix brave#4540
fix brave#5023

Auditors: @bbondy

Test Plan:
1. Go to https://github.com/brave/browser-laptop/issues
2. In the filters textbox, start typing characters until autocomplete pops up
3. Push enter to submit
4. Scroll the page should dismiss the popup

1. Go to https://github.com/brave/browser-laptop/issues
2. In the filters textbox, start typing characters until autocomplete pops up
3. Push enter to submit
4. Popup should be dismissed
@darkdh darkdh added this to the 0.12.9dev milestone Oct 31, 2016
@bbondy
Copy link
Member

bbondy commented Nov 7, 2016

++

@bbondy bbondy merged commit 9a8c957 into brave:master Nov 7, 2016
@bsclifton
Copy link
Member

❤️

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