feat(frontend-report-form): Add image validation and zoom functionality#287
feat(frontend-report-form): Add image validation and zoom functionality#287durante-stephanie merged 3 commits intodevelopfrom
Conversation
…ing for photo uploads
…ages in detail modal
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
|
||
| <div class="zoom-content"> | ||
| <img [src]="currentImageUrl()" | ||
| class="zoomed-image" |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line element attributes should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
class="zoomed-image" | <div class="zoom-content"> | ||
| <img [src]="currentImageUrl()" | ||
| class="zoomed-image" | ||
| alt="Zoomed Item" |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line element attributes should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
alt="Zoomed Item"| <img [src]="currentImageUrl()" | ||
| class="zoomed-image" | ||
| alt="Zoomed Item" | ||
| (click)="$event.stopPropagation()"> |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line element attributes should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
(click)="$event.stopPropagation()">| <label>Upload Photo: | ||
| <small>(Upload if available, max 5 photos)</small></label> | ||
| <label>Upload Photo <span class="required-asterisk">*</span>: | ||
| <small>(Required, max 5 photos)</small> |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line elements should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
<small>(Required, max 5 photos)</small>| [disabled]="isSubmitting || reportForm.invalid" | ||
| [title]="reportForm.invalid | ||
| [disabled]="isSubmitting" | ||
| [title]="reportForm.invalid || imageError |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line element attributes should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
[title]="reportForm.invalid || imageError|
|
||
| onSubmit(): void { | ||
| const totalImages = this.selectedFiles.length | ||
| + this.photoUrlsFormArray.length; |
There was a problem hiding this comment.
🤖 AI Review: Indentation for multi-line variable declarations should be at least a double tab to indicate continuation, as per AppDev Coding Guidelines.
+ this.photoUrlsFormArray.length;…ges in claim form modal
2 participants
Enhance the item report form with image validation to ensure at least one photo is uploaded. Implement zoom functionality for images in the detail modal, improving user experience when viewing images.