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.

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

[Feature] [FileUpload] Добавление начальных файлов при инициализации компонента #1832

Closed
1 of 10 tasks
TheSpaceHuman opened this issue Jun 21, 2024 · 5 comments
Assignees
Labels
estimate:sp3 Story Point - 3 source:idp issue from IDP type:improvement It is not a initial goal
Milestone

Comments

@TheSpaceHuman
Copy link
Contributor

TheSpaceHuman commented Jun 21, 2024

Библиотека

  • @prizm-ui/core
  • @prizm-ui/components
  • @prizm-ui/install
  • @prizm-ui/icons
  • @prizm-ui/icons-loader
  • @prizm-ui/flag-icons
  • @prizm-ui/theme
  • @prizm-ui/charts
  • @prizm-ui/ast
  • @prizm-ui/nx-plugin

Компонент

File Upload

Предложение

Есть необходимость во взаимодействии с компонентом с начально установленными файлами.

Предлагаю добавить механизм для добавления файлов через публичный метод addFiles или через @input files. Это благополучно скажется на придерживании prizm дизайн системы, не прибегая к реализации кастомных компонентов, копирующие ваш дизайн.

Так же учесть новый статус для progress и игнор появления экшена retry для этих файлов.

Снимок экрана 2024-06-21 в 12 36 26

Хорошим бонусом бы было улучшить devExp в части удаления. Сейчас, чтобы понять что было удалено, необходимо сопоставлять предыдущий массив файлов с текущим из эвента (filesChange). Данное решение неудобно, так как необходимо совмещение, как удалений, так и добавлений в единственном (filesChange). Поэтому добавление новых эвентов (deletedFile) и (addedFiles) было бы удобным решением для разработки.

Появился запрос на кастомные action btn для загруженных файлов.

1723112928525

Реализации через ng-container с select-том в месте экшенов для кастомных действий очень сильно пригодилась.
Пример:

<ng-container select="actions">
      <button prizmIconButton name="icon1"></button>
      <button prizmIconButton name="icon2"></button>
      <button prizmIconButton name="icon3"></button>
 </ng-container>
@ickisIckis ickisIckis added type:improvement It is not a initial goal estimate:sp3 Story Point - 3 labels Jun 28, 2024
@ickisIckis ickisIckis self-assigned this Jun 28, 2024
@imonogarov imonogarov added this to the 4.7.0 milestone Aug 7, 2024
@imonogarov
Copy link
Collaborator

Нужно добавить ControlValueAccessor

@TheSpaceHuman
Copy link
Contributor Author

Внес дополнительные фичи в описание.

@PrizmDS
Copy link

PrizmDS commented Aug 19, 2024

Доработку компонента в дизайне сделаем.

@ickisIckis нужно запланировать обновление в разработке

@TheSpaceHuman
Copy link
Contributor Author

Взялся за реализацию, по завершению сделаю PR.

TheSpaceHuman added a commit to TheSpaceHuman/Prizm that referenced this issue Sep 16, 2024
TheSpaceHuman added a commit to TheSpaceHuman/Prizm that referenced this issue Sep 17, 2024
TheSpaceHuman added a commit to TheSpaceHuman/Prizm that referenced this issue Sep 17, 2024
TheSpaceHuman added a commit to TheSpaceHuman/Prizm that referenced this issue Sep 17, 2024
@ZurabDev ZurabDev modified the milestones: 4.7.0, 5.3.0 Oct 3, 2024
ZurabDev added a commit that referenced this issue Oct 4, 2024
feat(components/file-upload): added init files && custom actions #1832
@ickisIckis ickisIckis mentioned this issue Oct 8, 2024
6 tasks
@bumalai bumalai added the source:idp issue from IDP label Oct 16, 2024
@bumalai
Copy link
Contributor

bumalai commented Oct 16, 2024

@ickisIckis наверно стоит закрыть задачу

@ickisIckis ickisIckis modified the milestones: 5.3.0, 5.1.0 Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
estimate:sp3 Story Point - 3 source:idp issue from IDP type:improvement It is not a initial goal
Projects
None yet
Development

No branches or pull requests

6 participants