Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Button] Еext overflow IF-572 #2932

Open
Eazymov opened this issue Jun 29, 2022 · 6 comments
Open

[Button] Еext overflow IF-572 #2932

Eazymov opened this issue Jun 29, 2022 · 6 comments

Comments

@Eazymov
Copy link
Contributor

Eazymov commented Jun 29, 2022

@skbkontur/react-ui@4.1.1
Chrome@103.0.5060.53
Ubuntu@22.04

(У кнопки указана ширина 150px)
image

Предполагаемое решение:
image
image

Юзкейс: в одном из контролов как текст кнопки задаём имя выбранного файла, если он есть. Не редко имя файла длинное.

@JackUait
Copy link
Contributor

Привет! Это поведение потенциально ломающее, так что мы не можем сделать его поведением по умолчанию, но в том, чтобы добавить проп, который будет включать такое поведение - ничего плохого не вижу

Егор (@zhzz) и Вова (@dzekh), что думаете по этому поводу?

@zhzz
Copy link
Member

zhzz commented Jun 30, 2022

Да, думаю можно добавить проп.

@JackUait
Copy link
Contributor

Создал задачу, будем заниматься

@dzekh
Copy link
Contributor

dzekh commented Jun 30, 2022

норм, но может какие-то ворнинги добавить? Ибо так нельзя по гайдам: https://guides.kontur.ru/components/button/#13

Обрезать конечно всё равно лучше чем чтобы вот так торчало, но это не должно быть «даром».

И еще можно подумать над опциями как обрезать — в середине может или в конце, или в начале названия. от того где находится наиболее ценная инфа.

@JackUait
Copy link
Contributor

JackUait commented Jul 1, 2022

Думаю, что ворнинг - это лишнее, так как пользователи библиотеки реализуют дизайн, следовательно, если такой кейс возникает - проблема кроется где-то в дизайне

И тут два решения:

  1. Добавить проп, который будет обрезать текст кнопки
  2. Выводить название файла в другом месте, например, в тултипе

@dzekh
Copy link
Contributor

dzekh commented Jul 4, 2022

@Eazymov Мы пока не берем в работу эту задачу, но будем рады принять пулл-реквест!

@dzekh dzekh changed the title [Button] text overflow [Button] Еext overflow IF-572 Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants