feat(content-management): adopt Design System primitives in Static Pages editor
Some checks failed
Architecture Governance / architecture (push) Has been cancelled
Some checks failed
Architecture Governance / architecture (push) Has been cancelled
Replace raw button/input elements with app-button, app-input, app-card, app-form-field, app-badge, app-empty-state across the static pages CMS editor. Checkboxes left native (no checkbox primitive built yet). Also fix pre-existing bug in ContentPageService.normalizePages: iterating Object.values(config) lost the record key, so legacy-shaped bootstrap entries without explicit id/slug fields (e.g. mock bootstrap.json's about-us, privacy-policy, terms-of-service) crashed normalizeSlug(undefined). Now falls back to the record key for id/slug/title. Verified in browser at /edit/static-pages: renders correctly, primitives styled per tenant CSS vars, no console errors after fix.
This commit is contained in:
@@ -1,45 +1,67 @@
|
|||||||
<section class="editor-section-card">
|
<section class="editor-section-card">
|
||||||
<div class="editor-actions">
|
<div class="editor-actions">
|
||||||
<h2>{{ 'builder.staticPages' | translate }}</h2>
|
<h2>{{ 'builder.staticPages' | translate }}</h2>
|
||||||
<button type="button" (click)="createPage()">{{ 'builder.createPage' | translate }}</button>
|
<app-button variant="primary" (click)="createPage()">{{ 'builder.createPage' | translate }}</app-button>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
@if (validation().duplicateSlugs.length > 0) {
|
@if (validation().duplicateSlugs.length > 0) {
|
||||||
<p class="editor-error">{{ 'staticPages.duplicateSlug' | translate }}</p>
|
<app-badge variant="danger">{{ 'staticPages.duplicateSlug' | translate }}</app-badge>
|
||||||
}
|
}
|
||||||
@if (validation().emptyTitles.length > 0) {
|
@if (validation().emptyTitles.length > 0) {
|
||||||
<p class="editor-error">{{ 'staticPages.emptyTitle' | translate }}</p>
|
<app-badge variant="danger">{{ 'staticPages.emptyTitle' | translate }}</app-badge>
|
||||||
|
}
|
||||||
|
|
||||||
|
@if (pages().length === 0) {
|
||||||
|
<app-empty-state [title]="'builder.staticPages' | translate" [description]="'builder.createPage' | translate">
|
||||||
|
<span slot="actions">
|
||||||
|
<app-button variant="primary" (click)="createPage()">{{ 'builder.createPage' | translate }}</app-button>
|
||||||
|
</span>
|
||||||
|
</app-empty-state>
|
||||||
}
|
}
|
||||||
|
|
||||||
<div class="stack-list">
|
<div class="stack-list">
|
||||||
@for (page of pages(); track page.id) {
|
@for (page of pages(); track page.id) {
|
||||||
<article class="sub-card">
|
<app-card padding="md">
|
||||||
|
<div class="page-card">
|
||||||
|
<div class="page-card__header">
|
||||||
|
<h3 class="page-card__title">{{ page.id }}</h3>
|
||||||
<div class="editor-actions">
|
<div class="editor-actions">
|
||||||
<h3>{{ page.id }}</h3>
|
<app-button variant="ghost" size="sm" (click)="move(page.id, -1)">↑</app-button>
|
||||||
<div class="editor-actions">
|
<app-button variant="ghost" size="sm" (click)="move(page.id, 1)">↓</app-button>
|
||||||
<button type="button" class="secondary" (click)="move(page.id, -1)">↑</button>
|
<app-button variant="danger" size="sm" (click)="deletePage(page.id)">{{ 'builder.deletePage' | translate }}</app-button>
|
||||||
<button type="button" class="secondary" (click)="move(page.id, 1)">↓</button>
|
|
||||||
<button type="button" class="secondary" (click)="deletePage(page.id)">{{ 'builder.deletePage' | translate }}</button>
|
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="editor-grid three">
|
<div class="editor-grid three">
|
||||||
<label><span>{{ 'builder.pageId' | translate }}</span><input type="text" [ngModel]="page.id" (ngModelChange)="updatePage(page.id, { id: $event })" /></label>
|
<app-form-field [label]="'builder.pageId' | translate">
|
||||||
<label><span>{{ 'builder.slug' | translate }}</span><input type="text" [ngModel]="page.slug" (ngModelChange)="updatePage(page.id, { slug: $event })" /></label>
|
<app-input [ngModel]="page.id" (ngModelChange)="updatePage(page.id, { id: $event })" />
|
||||||
<label><span>{{ 'builder.iconLabel' | translate }}</span><input type="text" [ngModel]="page.icon || ''" (ngModelChange)="updatePage(page.id, { icon: $event })" /></label>
|
</app-form-field>
|
||||||
|
<app-form-field [label]="'builder.slug' | translate">
|
||||||
|
<app-input [ngModel]="page.slug" (ngModelChange)="updatePage(page.id, { slug: $event })" />
|
||||||
|
</app-form-field>
|
||||||
|
<app-form-field [label]="'builder.iconLabel' | translate">
|
||||||
|
<app-input [ngModel]="page.icon || ''" (ngModelChange)="updatePage(page.id, { icon: $event })" />
|
||||||
|
</app-form-field>
|
||||||
|
<app-form-field [label]="'builder.footerGroup' | translate">
|
||||||
|
<app-input [ngModel]="page.footerGroup || ''" (ngModelChange)="updatePage(page.id, { footerGroup: $event })" />
|
||||||
|
</app-form-field>
|
||||||
|
<app-form-field [label]="'builder.orderLabel' | translate">
|
||||||
|
<app-input type="number" [ngModel]="page.order" (ngModelChange)="updatePage(page.id, { order: +$event })" />
|
||||||
|
</app-form-field>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="toggle-grid">
|
||||||
<label class="toggle-row"><input type="checkbox" [checked]="page.showInFooter" (change)="updatePage(page.id, { showInFooter: $any($event.target).checked })" /><span>{{ 'builder.showInFooter' | translate }}</span></label>
|
<label class="toggle-row"><input type="checkbox" [checked]="page.showInFooter" (change)="updatePage(page.id, { showInFooter: $any($event.target).checked })" /><span>{{ 'builder.showInFooter' | translate }}</span></label>
|
||||||
<label class="toggle-row"><input type="checkbox" [checked]="page.showInHeader" (change)="updatePage(page.id, { showInHeader: $any($event.target).checked })" /><span>{{ 'builder.showInHeader' | translate }}</span></label>
|
<label class="toggle-row"><input type="checkbox" [checked]="page.showInHeader" (change)="updatePage(page.id, { showInHeader: $any($event.target).checked })" /><span>{{ 'builder.showInHeader' | translate }}</span></label>
|
||||||
<label class="toggle-row"><input type="checkbox" [checked]="page.showInSitemap" (change)="updatePage(page.id, { showInSitemap: $any($event.target).checked })" /><span>{{ 'builder.showInSitemap' | translate }}</span></label>
|
<label class="toggle-row"><input type="checkbox" [checked]="page.showInSitemap" (change)="updatePage(page.id, { showInSitemap: $any($event.target).checked })" /><span>{{ 'builder.showInSitemap' | translate }}</span></label>
|
||||||
<label class="toggle-row"><input type="checkbox" [checked]="page.requiresAuthentication" (change)="updatePage(page.id, { requiresAuthentication: $any($event.target).checked })" /><span>{{ 'builder.requiresAuthentication' | translate }}</span></label>
|
<label class="toggle-row"><input type="checkbox" [checked]="page.requiresAuthentication" (change)="updatePage(page.id, { requiresAuthentication: $any($event.target).checked })" /><span>{{ 'builder.requiresAuthentication' | translate }}</span></label>
|
||||||
<label><span>{{ 'builder.footerGroup' | translate }}</span><input type="text" [ngModel]="page.footerGroup || ''" (ngModelChange)="updatePage(page.id, { footerGroup: $event })" /></label>
|
|
||||||
<label><span>{{ 'builder.orderLabel' | translate }}</span><input type="number" [ngModel]="page.order" (ngModelChange)="updatePage(page.id, { order: +$event })" /></label>
|
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
@for (locale of locales(); track locale) {
|
@for (locale of locales(); track locale) {
|
||||||
<div class="editor-grid two">
|
<div class="editor-grid two">
|
||||||
<label>
|
<app-form-field [label]="(('builder.translationTitle' | translate) + ' ' + locale)">
|
||||||
<span>{{ 'builder.translationTitle' | translate }} {{ locale }}</span>
|
<app-input [ngModel]="page.translations[locale]?.title || ''" (ngModelChange)="updateTranslation(page.id, locale, 'title', $event)" />
|
||||||
<input type="text" [ngModel]="page.translations[locale]?.title || ''" (ngModelChange)="updateTranslation(page.id, locale, 'title', $event)" />
|
</app-form-field>
|
||||||
</label>
|
|
||||||
<label class="full">
|
<label class="full">
|
||||||
<span>{{ 'builder.htmlPreview' | translate }} {{ locale }}</span>
|
<span>{{ 'builder.htmlPreview' | translate }} {{ locale }}</span>
|
||||||
<app-marketplace-html-editor
|
<app-marketplace-html-editor
|
||||||
@@ -49,7 +71,8 @@
|
|||||||
</label>
|
</label>
|
||||||
</div>
|
</div>
|
||||||
}
|
}
|
||||||
</article>
|
</div>
|
||||||
|
</app-card>
|
||||||
}
|
}
|
||||||
</div>
|
</div>
|
||||||
</section>
|
</section>
|
||||||
|
|||||||
@@ -0,0 +1,31 @@
|
|||||||
|
.page-card {
|
||||||
|
display: grid;
|
||||||
|
gap: 14px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.page-card__header {
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: space-between;
|
||||||
|
flex-wrap: wrap;
|
||||||
|
gap: 10px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.page-card__title {
|
||||||
|
margin: 0;
|
||||||
|
font-size: 1rem;
|
||||||
|
font-weight: 700;
|
||||||
|
color: var(--text-primary, #1e3c38);
|
||||||
|
}
|
||||||
|
|
||||||
|
.toggle-grid {
|
||||||
|
display: grid;
|
||||||
|
grid-template-columns: repeat(4, minmax(0, 1fr));
|
||||||
|
gap: 12px;
|
||||||
|
}
|
||||||
|
|
||||||
|
@media (max-width: 900px) {
|
||||||
|
.toggle-grid {
|
||||||
|
grid-template-columns: repeat(2, minmax(0, 1fr));
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -5,13 +5,29 @@ import { ContentManagementFacade } from '../facade/content-management.facade';
|
|||||||
import { ContentPage } from '../models/content-page.model';
|
import { ContentPage } from '../models/content-page.model';
|
||||||
import { TranslatePipe } from '../../../i18n/translate.pipe';
|
import { TranslatePipe } from '../../../i18n/translate.pipe';
|
||||||
import { MarketplaceHtmlEditorComponent } from '../../project-editor/components/html-editor/marketplace-html-editor.component';
|
import { MarketplaceHtmlEditorComponent } from '../../project-editor/components/html-editor/marketplace-html-editor.component';
|
||||||
|
import { ButtonComponent } from '../../../shared/ui/button/button.component';
|
||||||
|
import { InputComponent } from '../../../shared/ui/input/input.component';
|
||||||
|
import { CardComponent } from '../../../shared/ui/card/card.component';
|
||||||
|
import { FormFieldComponent } from '../../../shared/ui/form-field/form-field.component';
|
||||||
|
import { BadgeComponent } from '../../../shared/ui/badge/badge.component';
|
||||||
|
import { EmptyStateComponent } from '../../../shared/ui/empty-state/empty-state.component';
|
||||||
|
|
||||||
@Component({
|
@Component({
|
||||||
selector: 'app-static-pages-editor',
|
selector: 'app-static-pages-editor',
|
||||||
standalone: true,
|
standalone: true,
|
||||||
imports: [FormsModule, TranslatePipe, MarketplaceHtmlEditorComponent],
|
imports: [
|
||||||
|
FormsModule,
|
||||||
|
TranslatePipe,
|
||||||
|
MarketplaceHtmlEditorComponent,
|
||||||
|
ButtonComponent,
|
||||||
|
InputComponent,
|
||||||
|
CardComponent,
|
||||||
|
FormFieldComponent,
|
||||||
|
BadgeComponent,
|
||||||
|
EmptyStateComponent
|
||||||
|
],
|
||||||
templateUrl: './static-pages-editor.component.html',
|
templateUrl: './static-pages-editor.component.html',
|
||||||
styleUrls: ['../../project-editor/sections/section.shared.scss'],
|
styleUrls: ['../../project-editor/sections/section.shared.scss', './static-pages-editor.component.scss'],
|
||||||
changeDetection: ChangeDetectionStrategy.OnPush
|
changeDetection: ChangeDetectionStrategy.OnPush
|
||||||
})
|
})
|
||||||
export class StaticPagesEditorComponent {
|
export class StaticPagesEditorComponent {
|
||||||
|
|||||||
@@ -27,8 +27,8 @@ export class ContentPageService {
|
|||||||
}));
|
}));
|
||||||
}
|
}
|
||||||
|
|
||||||
return Object.values(config)
|
return Object.entries(config)
|
||||||
.map((page, index) => this.normalizePage(page as ContentPageBootstrapInput, index))
|
.map(([key, page], index) => this.normalizePage(page as ContentPageBootstrapInput, index, key))
|
||||||
.sort((left, right) => left.order - right.order);
|
.sort((left, right) => left.order - right.order);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -117,7 +117,7 @@ export class ContentPageService {
|
|||||||
}, {});
|
}, {});
|
||||||
}
|
}
|
||||||
|
|
||||||
private normalizePage(page: ContentPageBootstrapInput, index: number): ContentPage {
|
private normalizePage(page: ContentPageBootstrapInput, index: number, key: string): ContentPage {
|
||||||
const titleMap = typeof page.title === 'object' && page.title ? page.title : {};
|
const titleMap = typeof page.title === 'object' && page.title ? page.title : {};
|
||||||
const htmlMap = typeof page.html === 'object' && page.html ? page.html : {};
|
const htmlMap = typeof page.html === 'object' && page.html ? page.html : {};
|
||||||
const translations = { ...(page.translations ?? {}) };
|
const translations = { ...(page.translations ?? {}) };
|
||||||
@@ -131,9 +131,9 @@ export class ContentPageService {
|
|||||||
}
|
}
|
||||||
|
|
||||||
return {
|
return {
|
||||||
id: page.id,
|
id: page.id || key,
|
||||||
slug: this.normalizeSlug(page.slug || page.id),
|
slug: this.normalizeSlug(page.slug || page.id || key),
|
||||||
title: typeof page.title === 'string' ? page.title : Object.values(titleMap)[0] ?? page.id,
|
title: typeof page.title === 'string' ? page.title : Object.values(titleMap)[0] ?? page.id ?? key,
|
||||||
icon: page.icon,
|
icon: page.icon,
|
||||||
order: page.order ?? index + 1,
|
order: page.order ?? index + 1,
|
||||||
showInFooter: page.showInFooter !== false,
|
showInFooter: page.showInFooter !== false,
|
||||||
|
|||||||
Reference in New Issue
Block a user