Skip to content

UI k 5553/links redesign - #3070

Open
ilyabrower wants to merge 54 commits into
UIK-5351/add-more-variables-to-themefrom
UIK-5553/links-redesign
Open

UI k 5553/links redesign#3070
ilyabrower wants to merge 54 commits into
UIK-5351/add-more-variables-to-themefrom
UIK-5553/links-redesign

Conversation

@ilyabrower

Copy link
Copy Markdown
Contributor

Motivation and Context

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Nice improve.

Checklist:

  • I have updated the documentation accordingly.
  • I have added new tests on added of fixed functionality.

"@semcore/checkbox": "^17.2.1",
"@semcore/spin": "^17.2.1",
"@semcore/tooltip": "^17.2.1",
"@semcore/link": "^17.2.1",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ilyabrower and maybe add typography too?

@@ -0,0 +1,35 @@
import { Flex } from '@semcore/base-components';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ilyabrower Is this import necessary?

<Link
size={textProps.size}
href={externalHref}
color='--gray-400'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe we should use semantic token? @ilyabrower @j-mnizhek

@j-mnizhek
j-mnizhek self-requested a review August 14, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment