feat(ActionSheet, Alert): add stopPropagation ignore #8166
Conversation
github-actions
bot
added
the
patch
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
size-limit report 📦
|
e2e tests |
👀 Docs deployed
Commit 0bef3d2 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8166 +/- ##
=======================================
Coverage 95.50% 95.51%
=======================================
Files 401 401
Lines 11435 11453 +18
Branches 3774 3780 +6
=======================================
+ Hits 10921 10939 +18
Misses 514 514
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
BlackySoul
removed
the
patch
BlackySoul
changed the title
| const handleClick = (event: React.MouseEvent<HTMLElement>) => { | ||
| if (!allowClickPropagation) { | ||
| stopPropagation(event); | ||
| } | ||
| }; | ||
|
|
||
| const clickHandlers = mergeCalls({ onClick: handleClick }, { onClick }); |
There was a problem hiding this comment.
| const handleClick = (event: React.MouseEvent<HTMLElement>) => { | |
| if (!allowClickPropagation) { | |
| stopPropagation(event); | |
| } | |
| }; | |
| const clickHandlers = mergeCalls({ onClick: handleClick }, { onClick }); | |
| const handleClick = allowClickPropagation ? onClick : (event: React.MouseEvent<HTMLElement>) => { | |
| stopPropagation(event); | |
| onClick?.(event) | |
| }; |
не очень импонирует mergeCalls – внутри создается массив, потом внутри объект трасформируется в массив и потом итерируем по этому массиву
в данном случае, выглядит так, что проще не оборачивать в наш обработчик если передан allowClickPropagation
тоже самое в ActionSheet и его производных
Описание
На корневую обертку
ActionSheet/Alertмы навешиваем обработчик наonClickсstopPropagation. Возможно, для того, чтобы клики на всплывашках не аффектили основное приложение? Но вAlertдобавили обработчик, чтобы предотвратить ложные срабатыванияonClose- больше это не проблема, мы поменяли немного логику.Для чего клики нужны - глобальные обработчики клика для сбора аналитики.
Ещё у
ActionSheetDropdownMenuнавешивается обработчик наbody- он не особо нужен, потому что у насoverlayрастягивается на весь экран. Оставим наv8?Изменения
allowClickPropagationдля компонентовActionSheet/Alert+ есть возможность теперь прокинуть свои обработчикиonClick. Но, наверное, будет не совсем правильно убирать это в рамках минорной версии.Release notes
allowClickPropagationдля всплытияonClickи возможность передать кастомныйonClickallowClickPropagationдля всплытияonClickи возможность передать кастомныйonClick