From de9fc463d64bc444177c966032ba33e07ca9b635 Mon Sep 17 00:00:00 2001 From: Eldar <61377022+EldarMuhamethanov@users.noreply.github.com> Date: Fri, 13 Dec 2024 14:14:30 +0300 Subject: [PATCH] fix(UsersStack): make counter width adaptive (#8066) * fix(UsersStack): make counter width adaptive * fix: add screenshots * test: update screenshots * fix: remove unused selectors --- .../UsersStack/UsersStack.e2e-playground.tsx | 6 +++ .../UsersStack/UsersStack.module.css | 44 ++++++++++++------- .../src/components/UsersStack/UsersStack.tsx | 9 +++- ...sersstack-android-chromium-dark-1-snap.png | 4 +- ...ersstack-android-chromium-light-1-snap.png | 4 +- .../usersstack-ios-webkit-dark-1-snap.png | 4 +- .../usersstack-ios-webkit-light-1-snap.png | 4 +- .../usersstack-vkcom-chromium-dark-1-snap.png | 4 +- ...usersstack-vkcom-chromium-light-1-snap.png | 4 +- .../usersstack-vkcom-firefox-dark-1-snap.png | 4 +- .../usersstack-vkcom-firefox-light-1-snap.png | 4 +- .../usersstack-vkcom-webkit-dark-1-snap.png | 4 +- .../usersstack-vkcom-webkit-light-1-snap.png | 4 +- 13 files changed, 60 insertions(+), 39 deletions(-) diff --git a/packages/vkui/src/components/UsersStack/UsersStack.e2e-playground.tsx b/packages/vkui/src/components/UsersStack/UsersStack.e2e-playground.tsx index 697b7faf1e..101abefb58 100644 --- a/packages/vkui/src/components/UsersStack/UsersStack.e2e-playground.tsx +++ b/packages/vkui/src/components/UsersStack/UsersStack.e2e-playground.tsx @@ -39,6 +39,12 @@ export const UsersStackPlayground = (props: ComponentPlaygroundProps) => { [base64Image, base64Image, base64Image, base64Image], ], }, + { + children: [undefined], + size: ['m', 'l'], + photos: [[base64Image, base64Image, base64Image]], + count: [99], + }, ]} > {(props: UsersStackProps) => } diff --git a/packages/vkui/src/components/UsersStack/UsersStack.module.css b/packages/vkui/src/components/UsersStack/UsersStack.module.css index 5cd8f86a12..c14515daba 100644 --- a/packages/vkui/src/components/UsersStack/UsersStack.module.css +++ b/packages/vkui/src/components/UsersStack/UsersStack.module.css @@ -2,6 +2,20 @@ display: flex; align-items: center; color: var(--vkui--color_text_secondary); + + --vkui_internal--UsersStack_item_size: 16px; +} + +.sizeS { + --vkui_internal--UsersStack_item_size: 16px; +} + +.sizeM { + --vkui_internal--UsersStack_item_size: 24px; +} + +.sizeL { + --vkui_internal--UsersStack_item_size: 32px; } .avatarsPositionInlineEnd { @@ -21,28 +35,24 @@ fill: var(--vkui--color_image_placeholder_alpha); } -.photoOthers { +.item { + block-size: var(--vkui_internal--UsersStack_item_size); + min-inline-size: var(--vkui_internal--UsersStack_item_size); +} + +.photo { + max-inline-size: var(--vkui_internal--UsersStack_item_size); +} + +.counter { display: flex; align-items: center; justify-content: center; + box-sizing: border-box; + padding-inline: var(--vkui--spacing_size_xs); color: var(--vkui--color_text_contrast_themed); background: var(--vkui--color_icon_secondary); - border-radius: 50%; -} - -.sizeS .photo { - inline-size: 16px; - block-size: 16px; -} - -.sizeM .photo { - inline-size: 24px; - block-size: 24px; -} - -.sizeL .photo { - inline-size: 32px; - block-size: 32px; + border-radius: calc(var(--vkui_internal--UsersStack_item_size) / 2); } .photoWrapper { diff --git a/packages/vkui/src/components/UsersStack/UsersStack.tsx b/packages/vkui/src/components/UsersStack/UsersStack.tsx index 15c753e0f9..87074e99c8 100644 --- a/packages/vkui/src/components/UsersStack/UsersStack.tsx +++ b/packages/vkui/src/components/UsersStack/UsersStack.tsx @@ -168,7 +168,12 @@ export const UsersStack = ({ const photoSrc = isPhotoType ? photo.src : photo; let photoElement = ( - + @@ -198,7 +203,7 @@ export const UsersStack = ({ const othersElement = canShowOthers ? (
- + +{count}
diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-dark-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-dark-1-snap.png index 25a86daedb..7019dd0ee0 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-dark-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-dark-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:ee69f96bdd2bf297035924ab840859ce389e0a7b4c3a331468000344a69982f5 -size 441796 +oid sha256:c1aca17d0f1686bfc693007fa7e211942f80b3c7e94f66c2da1f270916c80d41 +size 453390 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-light-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-light-1-snap.png index 101a5e1cfd..f08054cffc 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-light-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-android-chromium-light-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:73abf476aba0f366e94deeb0661774d1b8f8595b01b95af93a0e1dabc732d9ab -size 416675 +oid sha256:eb82ef4ce14cfa68f9b13b172924a5e6d103c2e518ab952ede31ee179e785d18 +size 427873 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-dark-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-dark-1-snap.png index 8e85c4eb28..4e93d2c49b 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-dark-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-dark-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:d4a27c17f9546df67f3d35becc5fe766ce5642c052d6879aa6561d29819f3e2f -size 745870 +oid sha256:9114b69732787b2efe29d14bf2935b9f6b4c73f119b523292cab7d66ad4ef93e +size 767884 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-light-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-light-1-snap.png index 0774fe3004..c30314a284 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-light-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-ios-webkit-light-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:522900ae25f267e5e6a8ae80e8220c20f13167671e33119305d18ccb49b03f06 -size 717701 +oid sha256:bc8810136d9c7ad324d3043dd3c1deba61bd8bd4347d1808e2138bbf4ec13947 +size 738806 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-dark-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-dark-1-snap.png index d4b7fbb34e..941048d652 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-dark-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-dark-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:2984b8e5ea6701ce1fcbebdd48537d917e4c933a15da0c460250f0a931a2a1c8 -size 400541 +oid sha256:ca235ea2111f6b219bcc330ac926d119b31e365203d8c94213c96a1b7cbd6512 +size 411979 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-light-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-light-1-snap.png index 3d53557478..d9f7d1b377 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-light-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-chromium-light-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:9de446e98ddff5f095b2ba642ba187f342b6c4c1172cda917f69049a93246005 -size 392965 +oid sha256:e999ce19da1bfd9f45de652028a879e2f7226927b8ef158902ba11e07871e7cd +size 404027 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-dark-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-dark-1-snap.png index da04c544eb..6f45970819 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-dark-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-dark-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:fd1c1436fc2491788d5344fd7e868b90d939bb490812c6e93ca0cde85f221fa5 -size 600186 +oid sha256:6141650c06f378d296110bb1ee89a26768e3b3781ffe012da6fc142aa9e7ca6c +size 619903 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-light-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-light-1-snap.png index 2d3276e395..c5b87c4655 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-light-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-firefox-light-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:94cb3d8e7427db4526f673ff2e747cadb44ae03be07b0f9f3ef126f1bafd0ba0 -size 620611 +oid sha256:bba818a7cd14ffef4312f2da8a078f363fecafdd1bdf1a9a371a50bffa23202a +size 640365 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-dark-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-dark-1-snap.png index 1c784553f9..92f6729b35 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-dark-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-dark-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:92e905134162cb0997fbc5f22cbb5875e979b685d5278a43ea9a41e58a1a51ff -size 698134 +oid sha256:f546cb1a8ef41b4ad1d56bb2fba058932a7b785c6ff4d87d4879ddf4f53b99d1 +size 720570 diff --git a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-light-1-snap.png b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-light-1-snap.png index 80ebfb4dc3..67fb4a3e20 100644 --- a/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-light-1-snap.png +++ b/packages/vkui/src/components/UsersStack/__image_snapshots__/usersstack-vkcom-webkit-light-1-snap.png @@ -1,3 +1,3 @@ version https://git-lfs.github.com/spec/v1 -oid sha256:ed0a76bf0a330ccb247e6d38b5bd33a64986463c80feecb2d9a6b20fcad64027 -size 690596 +oid sha256:a6233e8a07010e4cf7e55a7a7d60a944a5db05d052ff51de2dd2aeca5e2a4f60 +size 712494