Skip to content

Conversation

@SevereCloud
Copy link
Contributor

@SevereCloud SevereCloud commented Nov 10, 2025

Описание

Изменяем кнопки раскрытия и копирования кода

2025-11-10.12.03.56.mov

Release notes

Документация

  • Улучшили кнопки раскрытия и копирования кода

@SevereCloud SevereCloud requested a review from a team as a code owner November 10, 2025 09:02
@SevereCloud SevereCloud self-assigned this Nov 10, 2025
@SevereCloud SevereCloud added this to VKUI Nov 10, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

✅ Изменений в собранных файлах нет

Commit da23d2f

@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

👀 Docs deployed

📦 Package ❌

yarn add @vkontakte/vkui@

Commit da23d2f

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.03%. Comparing base (2abc174) to head (da23d2f).
⚠️ Report is 26 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9152   +/-   ##
=======================================
  Coverage   95.02%   95.03%           
=======================================
  Files         420      419    -1     
  Lines       11184    11185    +1     
  Branches     4210     4211    +1     
=======================================
+ Hits        10628    10630    +2     
+ Misses        556      555    -1     
Flag Coverage Δ
unittests 95.03% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@SevereCloud SevereCloud added ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча ci:cherry-pick:v7 labels Nov 10, 2025
@SevereCloud SevereCloud moved this to 👀 In Review in VKUI Nov 10, 2025
@BlackySoul
Copy link
Contributor

image

Вот такая штука не очень хорошо выглядит теперь х)

По-моему, мы даже с дизайнерами как-то разбирали - они предлагали просто всегда справа держать зарезервированное место под иконку (по всей высоте) 🤔

Но мейби просто фон будет лучше

@SevereCloud SevereCloud force-pushed the SevereCloud/docs/Playground/expand-and-copy-buttons branch from f715dc1 to f4cc57d Compare November 12, 2025 08:11
@SevereCloud
Copy link
Contributor Author

Вот такая штука не очень хорошо выглядит теперь х)

Исправил

@inomdzhon inomdzhon added this to the v7.10.1 milestone Nov 12, 2025
@inomdzhon
Copy link
Contributor

inomdzhon commented Nov 12, 2025

image Вот такая штука не очень хорошо выглядит теперь х)

По-моему, мы даже с дизайнерами как-то разбирали - они предлагали просто всегда справа держать зарезервированное место под иконку (по всей высоте) 🤔

Но мейби просто фон будет лучше

А что если?

demo.mov

В тёмной теме

image

На светлой (поправил отступы, на видео не те отступы)

image

Сейчас много пространства иконка копирования занимает

2025-11-12.17.05.39.mov

@SevereCloud
Copy link
Contributor Author

Поправил глобально, теперь везде кнопка копирования показывается при наведении

@inomdzhon
Copy link
Contributor

Поправил глобально, теперь везде кнопка копирования показывается при наведении

image

Рис. 1. В Использование кнопка остается прозрачной при наведение, но физически есть

image

Рис. 2. Отступ справа у .prism-code .language-tsx в 44px нужно в любом случае добавить, а то кнопка будет перекрывать текст если первые строки будут длинные


Но точно ли хотим делать на наведение кнопку копирования?

Может возьмём опыт Stackblitz и будем показывать кнопку копирования на десктопе, а на мобилах скрывать (пример https://ark-ui.com/docs/components/accordion).

В будущем можем придти к решению когда код частично виден, пока не раскроешь и тогда вернуть кнопку копирования в тулбар (пример, https://base-ui.com/react/components/accordion).

@SevereCloud
Copy link
Contributor Author

Рис. 1. В Использование кнопка остается прозрачной при наведение, но физически есть

Исправлю

Рис. 2. Отступ справа у .prism-code .language-tsx в 44px нужно в любом случае добавить, а то кнопка будет перекрывать текст если первые строки будут длинные

Переделаю на Button с подложкой

Может возьмём опыт Stackblitz и будем показывать кнопку копирования на десктопе, а на мобилах скрывать (пример https://ark-ui.com/docs/components/accordion).

Проблему не решает

image

Но точно ли хотим делать на наведение кнопку копирования?

Она может мешать видимости текста. Много где видел такое поведение кнопки. А вообще есть мысль вовсе избавиться от кнопки копирования, оставив только при наличии заголовка

@SevereCloud SevereCloud modified the milestones: v7.10.1, v7.11.0 Nov 17, 2025
@SevereCloud SevereCloud force-pushed the SevereCloud/docs/Playground/expand-and-copy-buttons branch from 0217407 to ed5355c Compare November 19, 2025 15:36
@SevereCloud SevereCloud force-pushed the SevereCloud/docs/Playground/expand-and-copy-buttons branch from ed5355c to da23d2f Compare November 19, 2025 15:38
@SevereCloud
Copy link
Contributor Author

В итоге избавились от кнопки копирования и оставили ее только при наличии заголовка

@inomdzhon inomdzhon removed the ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 👀 In Review

Development

Successfully merging this pull request may close these issues.

5 participants