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

[Tooltip] Добавить возможность изменять размеры отступов в теле тултипа #1877

Closed
ando771 opened this issue Feb 6, 2020 · 3 comments
Labels
customization Issues related to library customization big task enhancement good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute.

Comments

@ando771
Copy link
Contributor

ando771 commented Feb 6, 2020

[email protected], Windows 10, Chrome 79

Сейчас для Tooltip нельзя изменить значение padding в теле тултипа. Они захардкожены в значении padding: 15px 20px. Хотелось бы иметь возможность каким-то образом изменять их.

Обойти проблему можно установкой отрицательных margin: -15px -20px у содержимого тултипа, переданного через пропс render.

Какие я вижу решения:

  1. Передавать размер отступа явно через пропсы Tooltip
  2. Через ThemeProvider, но не уверен, насколько это верно архитектурно
  3. Убрать полностью дефолтные, захардкоженные отступы из тултипа и считать, что пользователь библиотеки сам будет проставлять нужные ему через render.

Готов закинуть pr, если кто-то подскажет, какое решение лучше.

@wKich
Copy link
Member

wKich commented Feb 10, 2020

Tooltip отрисовывает отступы согласно гайдам, в 99% сценариев дает возможность не задумывать над отступами и просто выводить контент. Поэтому 3-й пункт мало того что ведет к ломающим изменениям, так обязывает всех кто сейчас использует тултипы начать явно задавать отступы по гайдам.

1-й пункт вполне возможное решение, но с учетом наличия механизма кастомизации внешнего вида через ThemeProvider, а так же то что некоторые компоненты уже позволяют кастомизировать отступы через него, например Button. Считаю что реализация через 2-й пункт наиболее правильная.

@wKich wKich added customization Issues related to library customization big task enhancement labels Feb 10, 2020
@ando771
Copy link
Contributor Author

ando771 commented Feb 11, 2020

@wKich я тебя понял, постараюсь на этой неделе кинуть pr со вторым вариантом.

@zhzz zhzz added the good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute. label Aug 30, 2020
@dzekh
Copy link

dzekh commented Jun 1, 2021

в 3.0 и 2.17 реализована возможность кастомизировать отступы тултипов, закрываю.

@dzekh dzekh closed this as completed Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customization Issues related to library customization big task enhancement good first issue Easy issue that you can do in several hours. Good for externalev delopers to start contribute.
Development

No branches or pull requests

4 participants