Skip to content

Update Angular 16 - #1996

Open
lthuong wants to merge 29 commits into
devfrom
le/update-angular-16
Open

Update Angular 16#1996
lthuong wants to merge 29 commits into
devfrom
le/update-angular-16

Conversation

@lthuong

@lthuong lthuong commented Oct 29, 2023

Copy link
Copy Markdown
Contributor
  • Update Angular 16
  • Delete Storybook since we didn't use the package in our project

@sheepyhollow

Copy link
Copy Markdown
Member

Okay! Ich habe es jetzt geschafft, das neue Frontend lokal zu starten. Insgesamt scheint alles zu funktionieren - gute Arbeit!

Mit sind ein paar Sachen aufgefallen:

  • letter-spacing hat sich verändert - an sich stört es nicht, aber Glossar und Überschriften in der Info-Component sehen nicht so gut aus. Kann das global auf normal gesetzt werden?
  • Quiz MC hat wieder das "alte" Checkbox-Styling. Ich sehe aber, dass Du an der Komponente bzw. SCSS noch einiges geändert hast. Ist das noch Work-in-progress oder gibt es da Probleme?

@sheepyhollow

Copy link
Copy Markdown
Member

Ich sehe außerdem beim Install-Prozess viele deprecation warnings. Das könnte evtl. ein Problem werden?

@lthuong

lthuong commented Dec 11, 2023

Copy link
Copy Markdown
Contributor Author
  1. Also ja. Ich habe das ein bisschen geändert, aber ich wusste nicht, ob es gut aussieht. Kannst du mir genau sagen, in welchen Stellen der Font angepasst werden soll ?
  2. Ich muss mal nachschauen, wie es damals ausgehen hat. Ich habe an diese Stelle hauptsächlich nur die neue selectors hinzufügen und ein paar Styling angepasst, damit es nicht chaotisch aussieht 😅
  3. Gute Beobachtung! Ich schätze ein, dass manche angepasst/updated/gelöscht werden müssen. Bis jetzt habe ich noch keine "sichtbare" Probleme gesehen. Aber ich muss das kurz recherchieren

Vielen Dank noch mal für die Mühe und Feedbacks 👍

@sheepyhollow

Copy link
Copy Markdown
Member

Font spacing / letter spacing an folgenden Stellen

  • Menü
  • Überschriften in der Info Componente (Info, News, Changelog etc.)
  • Glossar

g = g[3];
g = decodeURIComponent(g);
h && (g = atob(g));
h && (g = Buffer.from(g, 'base64').toString());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

omg... was ist das? An dieser Komponente haben wir seit Jahren nichts gemacht?


::ng-deep .mdc-list-item__primary-text {
font-size: 14px !important;
font-size: 13.4px !important;

@sheepyhollow sheepyhollow Dec 20, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also... die 14px fand ich schon gut - aber es ist das letter-spacing, das mich stört.
Wenn Du hier

letter-spacing: normal;

einfügst, müsste es passen; auch an den anderen Stellen.

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.

2 participants