-
Notifications
You must be signed in to change notification settings - Fork 44
FileUploadItem: onDownload или onDownloadClick #926
Comments
Я не могу вспомнить, почему там внутри https:/alfa-laboratory/core-components/blob/master/packages/file-upload-item/src/Component.tsx#L135 @SiebenSieben @dmitrsavk @EGNKupava есть мысли? |
Кажется что preventDefault нужен только если downloadLink пустой и обработчик загрузки какой-то внешний, а в остальных случаях надо просто переходить по ссылке. @dmitrsavk ты же вроде разбирался с загрузкой файлов? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Описание
Кейс: надо повесить метрику на скачивание.
для этого надо передать downloadLink и onDownload, при этом в самом onDownload нужно повесить вызов метрики и накрутить скачивание, так как onDownload внутри использует preventDefault и если просто повесить метрику - скачивания не будет.
выглядит это не очень.
предложение: добавить отдельно onDownloadClick или что-то вроде этого, или чтобы была возможность отключать preventDefault, чтобы можно было и свою реализацию сделать и оставить дефолтную. как пример у вас есть Link с onClick, там ничего не блокируется
Stage - 0
The text was updated successfully, but these errors were encountered: