fix(core): dialog has padding for close button event when not closeab…

…le (#10012)
taiga-family · Dec 18, 2024 · 97d24ad · 97d24ad
1 parent b5d328c
commit 97d24ad
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/projects/core/components/dialog/dialog.interfaces.ts b/projects/core/components/dialog/dialog.interfaces.ts
@@ -19,6 +19,9 @@ export type TuiDialogSize = TuiSizeL | TuiSizeS | 'auto' | 'fullscreen' | 'page'
  */
 export interface TuiDialogOptions<I> {
     readonly appearance: string;
+    /**
+     * TODO: rename to closable in v5.0
+     */
     readonly closeable: Observable<boolean> | boolean;
     readonly data: I extends void ? undefined : I & {button?: string};
     readonly dismissible: Observable<boolean> | boolean;

diff --git a/projects/core/components/dialog/dialog.style.less b/projects/core/components/dialog/dialog.style.less
@@ -97,10 +97,13 @@
 
 .t-heading {
     margin: 0 0 0.5rem;
-    padding-inline-end: 2rem;
     overflow-wrap: break-word;
     font: var(--tui-font-heading-4);
 
+    &_closable {
+        padding-inline-end: 2rem;
+    }
+
     &:empty {
         display: none;
     }

diff --git a/projects/core/components/dialog/dialog.template.html b/projects/core/components/dialog/dialog.template.html
@@ -9,6 +9,7 @@
 <div class="t-content">
     <h2
         class="t-heading"
+        [class.t-heading_closable]="context.closeable"
         [id]="context.id"
         [textContent]="context.label"
     ></h2>