TT-39 fix: comments

[scastillo-jp]

[juanultimate]

I am not a UI/UX expert, but IMO the year/month picker is spanning a lot of space in the screen. It would be great if you can fit all the months in a single row, and reduce/remove the padding of the card-body element,

[codecov]

Codecov Report

Merging #588 (881c520) into master (13de42e) will increase coverage by 0.80%.
The diff coverage is 98.54%.

@@            Coverage Diff             @@
##           master     #588      +/-   ##
==========================================
+ Coverage   93.09%   93.89%   +0.80%     
==========================================
  Files          85       95      +10     
  Lines        1564     1721     +157     
  Branches      107      113       +6     
==========================================
+ Hits         1456     1616     +160     
  Misses         67       67              
+ Partials       41       38       -3     

Impacted Files	Coverage Δ
src/app/app-routing.module.ts	100.00% <ø> (ø)
...nents/time-range-form/time-range-form.component.ts	100.00% <ø> (ø)
...project-list-hover/project-list-hover.component.ts	88.37% <ø> (ø)
src/app/modules/users/store/user.reducers.ts	87.50% <87.50%> (ø)
...dules/time-entries/pages/time-entries.component.ts	84.28% <92.85%> (+1.47%)	⬆️
...ponents/details-fields/details-fields.component.ts	90.90% <96.66%> (+3.72%)	⬆️
src/app/guards/login-guard/login.guard.ts	100.00% <100.00%> (ø)
...nologies-report-guard/technologies-report.guard.ts	100.00% <100.00%> (ø)
src/app/modules/login/login.component.ts	100.00% <100.00%> (+16.66%)	⬆️
...app/modules/login/services/azure.ad.b2c.service.ts	86.95% <100.00%> (+4.60%)	⬆️
... and 33 more

---

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 d2a4d5f...881c520. Read the comment docs.

[josepato87]

LGTM

[josepato87]

LGTM

It's resolved and he is on vacation

[josepato87]

LGTM