fix: remove whitespace from logo image - closes #374

[DanielRuf]

Type:
bugfix

The following has been addressed in the PR:

• Remove whitespace from image using font-size: 0.

See also https://css-tricks.com/fighting-the-space-between-inline-block-elements/

Description:

Resolves #374

[juanpicado]

LGTM 👏

[juanpicado]

Again snapshots :( are failing ...

[DanielRuf]

Some bot token issue: https://github.com/verdaccio/ui/pull/400/checks?check_run_id=363440405

[DanielRuf]

Failing snapshots, one test would need snapshot updates every year: https://circleci.com/gh/verdaccio/ui/6584?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-checks-link

We should improve this.

[juanpicado]

There is something planned https://github.com/verdaccio/ui/issues/199

[juanpicado]

@DanielRuf could you update snapshots?

[DanielRuf]

Which exact command is needed for that and is that currently documented?

[juanpicado]

At jest you need to add an argument -u. Perhaps worth to add a new script

"test:snapshopts": "npm run test -- -u",

to facilitate this process while the other ticket is being fixed.

[DanielRuf]

Might be good to document this in the contributing section of the readme and a CONTRIBUTING file.

I will update the PR later today.

[codecov]

Codecov Report

Merging #400 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #400     +/-   ##
=========================================
- Coverage   87.81%   87.72%   -0.1%     
=========================================
  Files         137      137             
  Lines         952      953      +1     
  Branches      211      198     -13     
=========================================
  Hits          836      836             
- Misses        102      104      +2     
+ Partials       14       13      -1

Impacted Files	Coverage Δ
src/components/Header/HeaderLogo.tsx	60% <0%> (-15%)	⬇️
src/components/Icon/styles.ts	88.88% <0%> (ø)	⬆️