Skip to content

Conversation

@j-mnizhek
Copy link
Member

@j-mnizhek j-mnizhek commented Jan 15, 2026

Changelog

@semcore/counter

Changed

  • value names for size property to match the whole size system in DS: 'm', 'l', 'xl' -> 's', 'm', 'l'.

Motivation and Context

We need to update value names for size property to match the whole size system in DS: 'm', 'l', 'xl' -> 's', 'm', 'l'. So that designers and devs will have more convenient list to choose the appropriate size for different components.

  • Updated values in the Counter component
  • Updated them in Playground as well
  • Default size was set from m to 's', so that teams won't have the unpredictable increasing of counters across all their interfaces

How has this been tested?

Manually.

Screenshots (if appropriate):

Before
image

After
image

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.

@j-mnizhek j-mnizhek marked this pull request as ready for review January 15, 2026 15:14
@j-mnizhek j-mnizhek force-pushed the UIK-2617/rename-values-for-size-prop branch from 028c6d0 to b7ac828 Compare January 15, 2026 15:25
@j-mnizhek j-mnizhek force-pushed the UIK-2617/rename-values-for-size-prop branch from d6f2cab to e9993cf Compare January 15, 2026 15:35
@j-mnizhek j-mnizhek requested a review from ilyabrower January 15, 2026 15:36
@ilyabrower ilyabrower changed the base branch from release/v16 to release/v17 January 16, 2026 09:32
@Valeria-Zimnitskaya Valeria-Zimnitskaya merged commit 4f031c6 into release/v17 Jan 22, 2026
13 checks passed
@Valeria-Zimnitskaya Valeria-Zimnitskaya deleted the UIK-2617/rename-values-for-size-prop branch January 22, 2026 21:21
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.

4 participants