Skip to content

[UIK-5581] filter trigger rebranding - #3066

Open
ilyabrower wants to merge 21 commits into
UIK-5351/add-more-variables-to-themefrom
UIK-5581/filter-trigger-rebranding
Open

[UIK-5581] filter trigger rebranding#3066
ilyabrower wants to merge 21 commits into
UIK-5351/add-more-variables-to-themefrom
UIK-5581/filter-trigger-rebranding

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.

…les-to-theme' into UIK-5581/filter-trigger-rebranding
@ilyabrower
ilyabrower marked this pull request as ready for review August 4, 2026 10:31
transform: translateY(-50%);
width: 1px;
background-color: var(--intergalactic-border-primary, oklch(0.137 0.026 175.7 / 0.161));
transition: height calc(var(--intergalactic-duration-extra-fast, 100) * 1ms) ease-out; /* TODO: Sync with animationin FilterTrigger */

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.

@sheila-semrush could you please take a look

Comment thread website/docs/style/design-tokens/design-tokens.json
Comment thread website/docs/style/design-tokens/design-tokens.json
@@ -1,22 +1,75 @@
SFilterTrigger[selected] {

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.

in the code we have selected prop but here it is removed
@sheila-semrush @ilyabrower is that expected?

Image Image

@ilyabrower
ilyabrower force-pushed the UIK-5581/filter-trigger-rebranding branch from 7875141 to aa00a22 Compare August 7, 2026 10:51
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.

3 participants