D27440: autofill input fields failing for some sites

David Rosca noreply at phabricator.kde.org
Wed Feb 19 12:08:10 GMT 2020


drosca requested changes to this revision.
drosca added a comment.
This revision now requires changes to proceed.


  I don't really like, there is ton of duplicated code. Also coding style is wrong.
  
  Can you change it so that you only touch what you need to touch? As far as I can see the only change you made is that instead of `input.type` it looks for `input.placeholder`.

REPOSITORY
  R875 Falkon

REVISION DETAIL
  https://phabricator.kde.org/D27440

To: puneethchanda, SGOrava, drosca
Cc: falkon, #falkon, grune, akilgus, siraj_qazi, saishm, anmolgautam, SGOrava, iodelay, spoorun, ptabis, navarromorales, clivej, mparillo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/falkon/attachments/20200219/5d49320c/attachment.html>


More information about the Falkon mailing list