Refactor on 162d199

primefaces · Oct 5, 2023 · bf9d72a · bf9d72a · vercel · Oct 5, 2023
1 parent bdecfee
commit bf9d72a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/app/components/image/image.css b/src/app/components/image/image.css
@@ -8,6 +8,7 @@
     .p-image-preview-container {
         position: relative;
         display: inline-block;
+        line-height: 0;
     }
 
     .p-image-preview-indicator {
@@ -18,6 +19,7 @@
         height: 100%;
         outline: none;
         border: none;
+        padding: 0;
         display: flex;
         align-items: center;
         justify-content: center;

diff --git a/src/app/components/image/image.ts b/src/app/components/image/image.ts
@@ -23,7 +23,7 @@ import { FocusTrapModule } from 'primeng/focustrap';
     template: `
         <span [ngClass]="containerClass()" [class]="styleClass" [ngStyle]="style">
             <img [attr.src]="src" [attr.srcset]="srcSet" [attr.sizes]="sizes" [attr.alt]="alt" [attr.width]="width" [attr.height]="height" [ngStyle]="imageStyle" [class]="imageClass" (error)="imageError($event)" />
-            <button type="button" class="p-image-preview-indicator" (click)="onImageClick()" #previewButton [ngStyle]="{'height': imgHeight, 'width': width + 'px'}" style="border: 'none';">
+            <button type="button" class="p-image-preview-indicator" (click)="onImageClick()" #previewButton [ngStyle]="{'height': height + 'px', 'width': width + 'px'}" style="border: 'none';">
                 <ng-container *ngIf="indicatorTemplate; else defaultTemplate">
                     <ng-container *ngTemplateOutlet="indicatorTemplate"></ng-container>
                 </ng-container>
@@ -230,10 +230,6 @@ export class Image implements AfterContentInit {
         min: 0.5
     };
 
-    get imgHeight() {
-        return this.document && this.document.defaultView ? getComputedStyle(DomHandler.findSingle(this.el.nativeElement, 'img'))['height'] : '100%';
-    }
-
     constructor(@Inject(DOCUMENT) private document: Document, private config: PrimeNGConfig, private cd: ChangeDetectorRef, public el: ElementRef) {}
 
     ngAfterContentInit() {