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!: change defaults, add prefix to css vars #99

Merged
merged 5 commits into from
Sep 8, 2023
Merged

Conversation

Lunory
Copy link
Collaborator

@Lunory Lunory commented Sep 7, 2023

  • change default color subheader, footer icons
  • change default icon size
  • add prefix gn to navigation css vars

@Lunory Lunory requested a review from DarkGenius as a code owner September 7, 2023 15:36
@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@Lunory Lunory changed the title feat!: change default color subheader, footer icons & default icon size feat!: change default color subheader, footer icons & default icon size & add prefix to navigation css vars Sep 7, 2023
@Lunory Lunory changed the title feat!: change default color subheader, footer icons & default icon size & add prefix to navigation css vars feat!: change defaults, add prefix to css vars Sep 7, 2023
@@ -1,6 +1,6 @@
import React from 'react';
import settingsIcon from '../../../../.storybook/assets/settings.svg';
import addIcon from '../../../../.storybook/assets/add.svg';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we remove old assets?

@DarkGenius
Copy link
Collaborator

Multiple tooltip should not trigger footer item highlighting

image

Add css vars to readme
Fix HighlightedNode in story
Rename css var --gn-aside-header-header-divider-line-color
@Lunory Lunory merged commit 08a812a into main Sep 8, 2023
3 checks passed
@Lunory Lunory deleted the lunory/mini-feats branch September 8, 2023 13:50
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.

3 participants