Skip to content
This repository has been archived by the owner on Jun 2, 2022. It is now read-only.

feat: Adds new Badge interactive variation #396

Merged
merged 7 commits into from
Mar 22, 2022

Conversation

lucasfp13
Copy link
Contributor

@lucasfp13 lucasfp13 commented Mar 17, 2022

What's the purpose of this pull request?

This PR intends to add a new Badge variant called interactive. This will be useful when the user needs an action to close/remove badges.

How does it work?

Default interactive badge:

Screen Shot 2022-03-17 at 16 48 04Screen Shot 2022-03-17 at 16 48 13

Small interactive badge:
Screen Shot 2022-03-17 at 16 46 34Screen Shot 2022-03-17 at 16 48 26

References

FSSS 204 - Creates Badge Interactive variation

Checklist

  • CHANGELOG entry added

@lucasfp13 lucasfp13 self-assigned this Mar 17, 2022
@netlify
Copy link

netlify bot commented Mar 17, 2022

✅ Deploy Preview for basestore ready!

🔨 Explore the source changes: d0184ba

🔍 Inspect the deploy log: https://app.netlify.com/sites/basestore/deploys/623a1d0c6cc60100080369d1

😎 Browse the preview: https://deploy-preview-396--basestore.netlify.app

@gatsby-cloud
Copy link

gatsby-cloud bot commented Mar 17, 2022

Gatsby Cloud Build Report

basestore

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 13m

Performance

Lighthouse report

Metric Score
Performance 💚 95
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 93

🔗 View full report

@lucasfp13 lucasfp13 marked this pull request as ready for review March 17, 2022 19:53
@vtex-sites
Copy link

vtex-sites bot commented Mar 17, 2022

Preview is ready

This pull request generated a Preview

👀   Preview: https://preview-396--base.preview.vtex.app
🔬   Go deeper by inspecting the Build Logs
📝   based on commit 1a9bf4e

Copy link
Contributor

@heitorado heitorado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with @eduardoformiga review and suggestion - Apart from that, everything looks good 👍🏼

Copy link
Contributor

@filipewl filipewl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking smooth!!! Left some points for discussion.

@filipewl filipewl requested a review from sergiofontes March 18, 2022 13:56
@lucasfp13 lucasfp13 force-pushed the feat/FSSS-204-create-badge-interactive-variation branch 3 times, most recently from 2451051 to 85eb18d Compare March 21, 2022 13:37
@lucasfp13 lucasfp13 force-pushed the feat/FSSS-204-create-badge-interactive-variation branch 2 times, most recently from b2380ae to 4ea8830 Compare March 22, 2022 18:36
@lucasfp13 lucasfp13 force-pushed the feat/FSSS-204-create-badge-interactive-variation branch from 4ea8830 to d0184ba Compare March 22, 2022 19:01
@lucasfp13 lucasfp13 merged commit 1a9bf4e into master Mar 22, 2022
@lucasfp13 lucasfp13 deleted the feat/FSSS-204-create-badge-interactive-variation branch March 22, 2022 20:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants