Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use line-height in badge #244

Merged
merged 1 commit into from
Aug 28, 2023
Merged

fix: use line-height in badge #244

merged 1 commit into from
Aug 28, 2023

Conversation

gabrielduete
Copy link
Contributor

Infos

Task

What is being delivered?

Adding propriety line-height: 0 in Host of badge component, because the component needs to conform to the context of the container it's embedded in.

What impacts?

Component badge

Reversal plan

Reset commit to the previous version.

Evidences

Before
image

After
image

@gabrielduete gabrielduete requested a review from a team August 25, 2023 20:27
@gabrielduete gabrielduete self-assigned this Aug 25, 2023
@gabrielduete gabrielduete requested review from bomffas, IngridCavalcantiGti, jowjow22 and deverebor and removed request for a team August 25, 2023 20:27
@sonarcloud
Copy link

sonarcloud bot commented Aug 25, 2023

[juntossomosmais_atomium_core] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@github-actions
Copy link
Contributor

github-actions bot commented Aug 25, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-08-28 13:07 UTC

@gabrielduete gabrielduete changed the title fix: use lineheight fix: use line-height in badge Aug 25, 2023
@gabrielduete gabrielduete merged commit d073c2b into main Aug 28, 2023
4 checks passed
@gabrielduete gabrielduete deleted the fix/use-lineheight branch August 28, 2023 13:03
@github-actions github-actions bot mentioned this pull request Aug 28, 2023
@github-actions github-actions bot mentioned this pull request Oct 17, 2023
This was referenced Feb 15, 2024
@github-actions github-actions bot mentioned this pull request Jun 5, 2024
@github-actions github-actions bot mentioned this pull request Jun 19, 2024
@github-actions github-actions bot mentioned this pull request Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants