fix(button): fix mouseover after disabled button #140
Conversation
Astero-lol
requested review from
GREENpoint,
Heymdall,
jangot,
ivliag,
iyromanov,
DNAlchemist,
gorelikov,
knovinskaya and
kara-u
src/button/button.css
Outdated
| @@ -24,6 +24,7 @@ | |||
| white-space: nowrap; | |||
|
|
|||
| transition: all 200ms ease; | |||
| transform: translateZ(0); | |||
There was a problem hiding this comment.
с этим свойством кнопки по оси z выше чем обычно становятся и событие стало срабатывать
There was a problem hiding this comment.
Можно поподробнее, почему так? Сейчас это решение выглядит не сильно дешёвым по ресурсам, Button в куче мест используется + это в разных местах может зааффектить. Есть другие способы?
There was a problem hiding this comment.
А в чем суть? Зачем им быть выше? Можешь объяснить суть бага...
There was a problem hiding this comment.
Суть бага если мы сначала наведем курсор мыши на кнопку в состоянии disabled, а потом переместим курсор на соседнюю кнопку, то событие mouseover не срабатывало, как будто над кнопкой есть какой то невидимый элемент, который это событие перехватывает, с чем этот баг может быть связан я так и не понял, все у кого я спрашивал считают что это баг реакта, в целом это решение проверенное и вероятность того что это может сломать очень мало, но чтобы совсем свести риск на нет можно добавить это только кнопкам в радиогруппах (так как этот баг только в кнопках которые вплотную друг к другу стоят)
There was a problem hiding this comment.
Завел issue прикрепил пример
There was a problem hiding this comment.
Вообще это дубликатом сделают скорее всего. Я сильно не вник в те 5 багов что там уже есть.
Если это решение действительно работает то можно его использовать.
tx44
requested review from
ziimir,
stepancar and
tx44
and removed request for
jangot,
DNAlchemist,
gorelikov,
knovinskaya,
kara-u and
iyromanov
|
@Astero-lol , вот такое решение не пробовал? |
Astero-lol
force-pushed
the
fix/mouseover-after-disabled-button
branch
from
0a619da to
85366d4
Compare
|
Сделал как Степа написал |
Fix mouseover after disabled button
Исправлен баг при котором не срабатывало событие mouseover после того как переводишь курсор мыши с disabled кнопки на любую другую
https://trello.com/c/GZTgThg4/309-checkboxgroup-проблемы-с-mouseover