feat(timepicker): remove inline styling, #2496

[LimarenkoDenis]

#2496

PR Checklist

Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.

• read and followed the CONTRIBUTING.md guide.
• built and tested the changes locally.
• added/updated tests.
• added/updated API documentation.
• added/updated demos.

[codecov]

Codecov Report

Merging #2812 into development will increase coverage by 0.31%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development    #2812      +/-   ##
===============================================
+ Coverage        70.38%   70.69%   +0.31%     
===============================================
  Files              137      137              
  Lines             4092     4149      +57     
  Branches           711      731      +20     
===============================================
+ Hits              2880     2933      +53     
+ Misses             913      911       -2     
- Partials           299      305       +6

Impacted Files	Coverage Δ
src/timepicker/timepicker.component.ts	91.42% <ø> (ø)	⬆️
src/typeahead/typeahead.directive.ts	71.8% <0%> (-0.09%)	⬇️
src/typeahead/typeahead-container.component.ts	79.83% <0%> (+6.98%)	⬆️
src/utils/utils.class.ts	80% <0%> (+30%)	⬆️

---

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 eccb382...de87da6. Read the comment docs.

[IlyaSurmay]

Hi!
If you simply remove inline styles from timepicker, it will cause this:

So you should replace them with component-scoped styles. Please update your PR.
Thanks for PR!

[IlyaSurmay]

Inline styles must be replaced with component-scoped styles

[LimarenkoDenis]

Sorry for misunderstanding the issue, I updated pull request

[YevheniiaMazur]

Tested, looks good

[valorkin]

@LimarenkoDenis thanks for the contribution! :)