Input clear button

[anandsinghparihar]

Add All search

Input with a clear button #610

Right now all search input doesn't have a clear button, we need to clear that input by backspace. so I hope we should add clear button.

here is a preview of a clear button :

[coveralls]

Coverage remained the same at 46.432% when pulling 0b2e396 on input_clear_button into 7816aa9 on dev.

[AllienWorks]

Styled.

[coveralls]

Coverage remained the same at 46.432% when pulling f6e13c4 on input_clear_button into 7816aa9 on dev.

[pciavald]

@anandsinghparihar @AllienWorks I edited the thread conversations, these matters are better discussed on Matrix 👍 cheers :)

[pciavald]

• @AllienWorks cross style is inconsistent
  • red cross
    • address book search
    • history clear all
  • white on red cross
    • closing modals
    • address book delete
    • send address clear
  • white on black cross
    • receive search
    • history search
• @anandsinghparihar can you please activate escape to clear & enter like in history search

[anandsinghparihar]

@pciavald

can you please activate escape to clear & enter like in history search
Enter is not required because of all search fire on input changes. I've added escape for clear search field.

@AllienWorks now you can work on the design part.

[coveralls]

Coverage remained the same at 46.432% when pulling cc5b16b on input_clear_button into 425000a on dev.

fixed

[AllienWorks]

Restyled.

This is how icons are organized now:

• part-circle-remove as "close/delete" (for closing modals, deleting entries)
• part-clear-all as "clear" (e.g. here for clearing text from inputs or whole forms)
• part-cross as "cancel"

I don't think we should unify all those 3 into 1; after all, there are differences between them. Those icons that aren't fixed according to this ^ are fixed already in #657 (FYI).

[coveralls]

Coverage remained the same at 46.432% when pulling 3829090 on input_clear_button into 425000a on dev.