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.

Sign up for GitHub

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

Jump to bottom

[Tooltip] - Тултип заезжает на границы TextArea #721

Closed
StarLordJ opened this issue Sep 18, 2018 · 3 comments
Closed

[Tooltip] - Тултип заезжает на границы TextArea #721

StarLordJ opened this issue Sep 18, 2018 · 3 comments
Assignees
@wKich
Labels
bug help wanted

Comments

@StarLordJ
Copy link

tooltiptop
tooltipbottom

Привет! В retail-ui версии 0.20.1 (возможно и раньше, но в версии 0.16.3 всё ещё работало правильно) Tooltip стал заезжать на границы TextArea во всех позициях "top ..." и "bottom ...".
Поглядел в браузерах: Chrome, Edge, IE11 - везде воспроизводится.
@wKich wKich added the bug label Sep 18, 2018
@wKich
Copy link
Member

wKich commented Sep 18, 2018

Есть такое. Кажется можно полечить, если для span, в который оборачивается контент тултипа, проставить стиль display: inline-block

@21alexander21 21alexander21 self-assigned this Sep 18, 2018
@wKich wKich mentioned this issue Sep 27, 2018
Closed
@21alexander21 21alexander21 reopened this Sep 27, 2018
@21alexander21
Copy link
Contributor

Ситуация такова: display: inline-block лечит баг с позиционированием, но ломает относительную ширину у переданного потомка. А т.к. у очень многих пользователей контролы обернуты в ValidationWrapperV1 (который оборачивает потомка в тултип), это сломало много форм.

Пока фикс откатим, чтобы поискать лучшее решение

@21alexander21 21alexander21 mentioned this issue Sep 27, 2018
Closed
wKich added a commit that referenced this issue Nov 13, 2018
@wKich
feat(Tooltip): add useWrapper prop
9b53ba1 
Allow render children without wrapper. That fix tooltip behavior with inline-block elements

fix #721
@wKich wKich mentioned this issue Nov 14, 2018
Merged
5 tasks
wKich added a commit that referenced this issue Nov 14, 2018
@wKich
feat(Tooltip): add useWrapper prop
3a0f39a 
Allow render children without wrapper. That fix tooltip behavior with inline-block elements

fix #721
@wKich
Copy link
Member

wKich commented Nov 14, 2018

Пофиксится в #901 при использовании пропса useWrapper={false} для тултипа

@wKich wKich assigned wKich and unassigned 21alexander21 Nov 14, 2018
wKich added a commit that referenced this issue Nov 20, 2018
@wKich
feat(Tooltip): add useWrapper prop
71d803a 
Allow render children without wrapper. That fix tooltip behavior with inline-block elements

fix #721
@wKich wKich mentioned this issue Jan 18, 2019
Closed
@zhzz zhzz mentioned this issue Apr 6, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wKich wKich

Labels
bug help wanted
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wKich @21alexander21 @StarLordJ