Add image carousel component#31
Conversation
| [ngClass]="{ 'es-image-carousel__arrow_enabled': canScrollLeft }" | ||
| > | ||
| <mat-icon>arrow_back</mat-icon> | ||
| </button> |
There was a problem hiding this comment.
Эти кнопки надо мпоменять местами, иначе порядок обхода неправильный
| > | ||
| <mat-icon *ngIf="viewSvgIcon" [svgIcon]="viewSvgIcon"></mat-icon> | ||
| <mat-icon *ngIf="!viewSvgIcon">visibility</mat-icon> | ||
| </button> |
There was a problem hiding this comment.
Надо что-то придумать с a11y, не могу без мышки нажимать на эти кнопки
| public set imageTypes(value: string) { | ||
| this._imageTypes = value ?? this.defaultOptions?.imageTypes ?? 'image/*'; | ||
| } | ||
| private _imageTypes: string; |
There was a problem hiding this comment.
Нужны ли такие сложности с files и imageTypes? Может было бы проще сделать один инпут images: string[], а все остальное на совесть пользователя? А то еще этот заточенный под нас интерфейс ESImageCarouselFile используется.
| return this._viewSvgIcon; | ||
| } | ||
| public set viewSvgIcon(value: string) { | ||
| this._viewSvgIcon = value ?? this.defaultOptions?.viewSvgIcon; |
There was a problem hiding this comment.
Нет в документации про defaultOptions
| * @ignore | ||
| */ | ||
| public get gridColumns(): string { | ||
| return `repeat(auto-fit, ${this.imageWidth}px)`; |
There was a problem hiding this comment.
Просто grid-auto-flow: column (или row, путяю их всегда) в стилях сделать?
| * Object with `ESImageCarouselAction` type is emitted. | ||
| */ | ||
| @Output() | ||
| public remove: EventEmitter<ESImageCarouselAction> = new EventEmitter(); |
There was a problem hiding this comment.
Как насчет removeSvgIcon? Может мусорную корзину захотят :)
klimenkosergei
changed the title
2 participants
No description provided.