VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

[Refactor][ChipsSelect][ChipsInput] Исправляем внутреннюю реализацию компонентов

Open eugpoloz opened this issue 3 years ago • 4 comments

resolves #2884, relates to #2845 (нужно, чтобы там сделать по красоте)

ChipsInput:

  • разделила на ChipsInput и ChipsInputBase,

ChipsSelect:

  • обернула в свой собственный FormField, заюзав внутри ChipsInputBase,
  • сделала кнопку-стрелочку, которая тогглит дропдаун (часть https://github.com/VKCOM/VKUI/issues/1673),
  • подбила стили,
  • подправила типы (чтобы не смущать людей возможностью передать свой after),

IconButton:

  • дала возможность переопределять свойства activeHover и activeEffectDelay, которые уходят ниже в Tappable (чтобы можно было убрать эффекты с кнопки в ChipsSelect).

eugpoloz avatar Jul 25 '22 16:07 eugpoloz

size-limit report 📦

Path Size
JS 256.5 KB (+0.04% 🔺)
JS (gzip) 76.98 KB (-0.04% 🔽)
JS (brotli) 64.98 KB (-0.1% 🔽)
JS, unstable 30.94 KB (+0.3% 🔺)
CSS 282.16 KB (+0.02% 🔺)
CSS (gzip) 38.26 KB (+0.04% 🔺)
CSS (brotli) 30.63 KB (+0.16% 🔺)
CSS, unstable 1.22 KB (+0.98% 🔺)

github-actions[bot] avatar Jul 25 '22 17:07 github-actions[bot]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit e848e1c3004223b435cf2afe660f4c2810be4a07:

Sandbox Source
VKUI - default example Configuration

codesandbox-ci[bot] avatar Jul 25 '22 17:07 codesandbox-ci[bot]

👀 Styleguide deployed

See the styleguide for this PR at https://vkcom.github.io/VKUI/pull/2890/

github-actions[bot] avatar Jul 25 '22 17:07 github-actions[bot]

Code coverage

lines3875 / 497977.82%
statements3945 / 507877.68%
functions839 / 106978.48%
branches3492 / 484672.05%
branchesTrue0 / 0100.00%

Generated by :no_entry_sign: dangerJS against e848e1c3004223b435cf2afe660f4c2810be4a07

github-actions[bot] avatar Jul 25 '22 17:07 github-actions[bot]