CIF-2291 - Cleanups in searchbar component

[LSantha]

• new CSS classes according to BEM methodology
• default minimal styling
• added v2 for searchber clientlib
• fixed search field reset button
• added text field for placeholder property in edit dialog
• updated readme

Related Issue

CIF-2291

How Has This Been Tested?

Manually, unit tests, UI tests.

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:

• I have signed the Adobe Open Source CLA.
• 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 and the overall coverage did not decrease.
• All unit tests pass on CircleCi.
• I ran all tests locally and they pass.

[codecov]

Codecov Report

Merging #655 (f3cd57b) into master (96b8b8d) will increase coverage by 0.05%.
The diff coverage is 96.15%.

@@             Coverage Diff              @@
##             master     #655      +/-   ##
============================================
+ Coverage     88.51%   88.57%   +0.05%     
  Complexity     1598     1598              
============================================
  Files           281      282       +1     
  Lines          7079     7131      +52     
  Branches       1047     1047              
============================================
+ Hits           6266     6316      +50     
- Misses          616      618       +2     
  Partials        197      197              

Flag	Coverage Δ
integration	59.56% <ø> (ø)
jest	85.61% <ø> (ø)
karma	90.01% <96.15%> (+0.40%)	⬆️
unittests	89.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
.../searchbar/v2/searchbar/clientlibs/js/searchbar.js	96.15% <96.15%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96b8b8d...f3cd57b. Read the comment docs.

[herzog31]

Looks good, but same problem like in the other PR. Why adding tests to Venia only and not in the components repo? The feature should be tested in this repo.