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

[Bug]: Компоненты на основе Clickable не интерактивны #6284

Closed
3 tasks done
BlackySoul opened this issue Dec 15, 2023 · 1 comment · Fixed by #6382
Closed
3 tasks done

[Bug]: Компоненты на основе Clickable не интерактивны #6284

BlackySoul opened this issue Dec 15, 2023 · 1 comment · Fixed by #6382
Assignees
@BlackySoul
Labels
cmp:avatar cmp:button cmp:form cmp:rich-cell type:bug
Milestone
v6.0.0

Comments

@BlackySoul
Copy link
Contributor

BlackySoul commented Dec 15, 2023 •
edited
Loading

Описание

  • Button в форме больше не реагирует на взаимодействие
    У нас сейчас контролы без обработчиков неактивные, и если мы раньше могли просто расположить <Button type="submit">Submit</Button> внутри <form>, то оно бы просто вызывало сабмит формы, сейчас тоже вызывает, но active/hover состояний на кнопке нет. (resolved by fix: button add onClick = noop #6381)

  • Отсутствует hover-состояние на Avatar c overlay=onHover

  • RichCell показывает неверный курсор

2023-12-21.19.36.00.mov

Версия

v6

В каких браузерах воспроизводится проблема?

No response

Шаги воспроизведения

No response

Ожидаемое поведение

No response

Скриншоты

No response

Пример с воспроизведением

No response
@BlackySoul BlackySoul added this to the v6 milestone Dec 15, 2023
@BlackySoul BlackySoul changed the title [Bug]: Button в форме больше не реагирует на взаимодействие [Bug]: Компоненты на основе Clickable не интерактивны Dec 16, 2023
@BlackySoul BlackySoul mentioned this issue Jan 15, 2024
Merged
@BlackySoul BlackySoul linked a pull request Jan 15, 2024 that will close this issue
fix(RichCell, ImageBaseOverlay): add noop default + remove disabled prop #6382
Merged
@vkcom-publisher
Copy link
Contributor

✅ v6.0.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BlackySoul BlackySoul

Labels
cmp:avatar cmp:button cmp:form cmp:rich-cell type:bug
Projects
VKUI
Archived in project
Milestone
v6.0.0
Development

Successfully merging a pull request may close this issue.

fix(RichCell, ImageBaseOverlay): add noop default + remove disabled prop VKCOM/VKUI
2 participants
@BlackySoul @vkcom-publisher