Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/documentation #48

Merged
merged 7 commits into from
Jan 21, 2021
Merged

Bugfix/documentation #48

merged 7 commits into from
Jan 21, 2021

Conversation

jfrohn
Copy link
Contributor

@jfrohn jfrohn commented Dec 22, 2020

Abnahme Protokoll

Prio 1 – Approval MAD-Components

  1. Readonly Form Field
    Docu Änderungen:
  • Eigene Card für das Beispiel „Read-only mit nachgelagerter Validierungsnachricht“
  • Fachlich besseres Beispiel als First name wählen (eg. IBAN)
  • Doc-Struktur von Clarity übernehmen (Design Guidelines | Code & Examples)
  • Empty Value Handling (“-“) is automatically handled from the readonly form field.
    Empty value means that null values, or fields where not data is inputted are shown with an “-“. (Text aus C3/VU3 Styleguide übernehmen)

Komponente

  • Funktionale Abnahme der Komponente OK.
  1. Card
    Docu Änderungen:
  • Text und Empfehlungen von Clarity Docu übernehmen (clarity.design: https://clarity.design/documentation/cards)
  • Erste Beispiel: ohne collapse/expand (EDIT)
  • Zweites Beispiel: Readonly
  • Drittes Beispiel: optional mit collapse/expand
    o „Falls gewünscht kann auch eine Variante mit collapse/expand verwendet werden“
    Komponente
  • Funktionale Abnahme der Komponente OK.
  1. Numeric Field
    Docu Änderungen:
  • Von Clarity Addons alles übernehmen
  • Felder kürzer (an den Inhalt angepasst)
    Komponente
  • Funktionale Abnahme der Komponente OK.
  1. Quick List
    Docu Änderungen:
  1. Action Table
    Docu Änderungen:
  • Von Clarity (Datagrid) Beschreibung übernehmen (so viel wie sinnvoll)
  • Umbenennen auf Table (auf originäre Mat. Componente TABLE verlinken)
  • Pageination und filter checkboxes nebeneinander
  • Sinnvolle Daten verwenden (entweder Random User Generator oder von Datagrid Clarity übernehmen)
    - Search Field (N2H) -> Das müsste in den Addons direkt geändert werden und würde unsere Styleguide in CarIN wiedersprechen

Komponente

  • Funktionale Abnahme der Komponente OK.

Prio 2 – neue Doku erstellen / Abrunden der bestehende Documentation

Material Addons Documentation

  • Titel anpassen
  • Usermenu und appswitcher löschen
  • Demos – Menüpunkt auf Pagelayouts (oder löschen)
    o Base Pagelayout (CARin)
    o NavDrawer Pagelayout (Carlos)
    o …
    -> fehlt noch
  • Context menu im table – reduzieren auf „edit“ und „delete“

@jfrohn jfrohn requested review from poi-lor and Laess3r December 22, 2020 08:33
@poi-lor poi-lor closed this Jan 21, 2021
@poi-lor poi-lor reopened this Jan 21, 2021
@@ -5,7 +5,7 @@ <h3>
<div *ngIf="itemTemplate" #row>
<ng-container *ngTemplateOutlet="itemTemplate; context: { $implicit: item }"> </ng-container>
<button *ngIf="!(readonly || readonly === '')" mat-icon-button [disabled]="!isDeleteAllowed()" (click)="removeItem(item)">
<mat-icon>delete</mat-icon>
<mat-icon color="primary">delete_outline</mat-icon>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In master-branch this change was already done by replacing it with the new

@poi-lor poi-lor force-pushed the bugfix/documentation branch from 24d16bc to 5ce2a11 Compare January 21, 2021 11:11
@Laess3r Laess3r merged commit 55303cd into master Jan 21, 2021
@Laess3r Laess3r deleted the bugfix/documentation branch January 21, 2021 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants