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

feat: Chip Nvigation #1315

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

feat: Chip Nvigation #1315

wants to merge 7 commits into from

Conversation

wendellguimaraes-tlf
Copy link
Contributor

@wendellguimaraes-tlf wendellguimaraes-tlf commented Jan 30, 2025

OBVIVO-2618 (Chip)

Added the href property for navigation and changed the style when the Chip component has this property and is a navigation component.

https://www.figma.com/design/512a5Ke4EldL0GYffCXGCJ/%F0%9F%94%B8-Chips-Specs?node-id=2073-1672&t=KAssWsb3jE5QNxP3-0

@wendellguimaraes-tlf wendellguimaraes-tlf changed the title Chip Nvigation Feat: Chip Nvigation Jan 30, 2025
Copy link

github-actions bot commented Jan 30, 2025

Size stats

master this branch diff
Total JS 12.3 MB 12.3 MB +395 B
JS without icons 2.06 MB 2.07 MB +395 B
Lib overhead 73.5 kB 73.5 kB 0 B
Lib overhead (gzip) 17.1 kB 17.1 kB 0 B

Copy link

github-actions bot commented Jan 30, 2025

Accessibility report
✔️ No issues found

ℹ️ You can run this locally by executing yarn audit-accessibility.

@wendellguimaraes-tlf wendellguimaraes-tlf changed the title Feat: Chip Nvigation feat: Chip Nvigation Jan 30, 2025
Copy link

github-actions bot commented Jan 30, 2025

Deploy preview for mistica-web ready!

✅ Preview
https://mistica-fwvcta2fk-flows-projects-65bb050e.vercel.app

Built with commit 578b6dc.
This pull request is being automatically deployed with vercel-action

Copy link

github-actions bot commented Feb 4, 2025

Screenshot tests report

✔️ All passing

navigationActiveInverse: [
containerBase,
sprinkles({
color: vars.colors.controlActivated,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
color: vars.colors.controlActivated,
color: vars.colors.textActivated,

I found a mistake in specs, please, fix this color

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants