style(KolButton, KolLinkButton): increase font-size

[Chrisdo82]

No description provided.

[deleonio]

Hi Christian (@Chrisdo82 ),

kannst Du bitte die Snapshots aktualisieren.

• https://github.com/public-ui/kolibri/actions/runs/6717334391

Dort kannst Du ganz unten die Abweichungen herunterladen, "Artefakt".

IF YOU LIKE - Um die Snapshots zu aktualisieren.

Gehe einfach ins Submodul (nur Mac): packages/themes/bmf - pnpm test-update

[Chrisdo82]

@deleonio Habe ich soeben versucht, bekomme da aber wieder mal einen Fehler Error: Command failed: npm run build -- --output-path=/var/folders/xy/_psf9hsj4nzd_rpcwgsqyt5m0000gn/T/kolibri-visual-testing-build-b2e462f6-3a7e-4bb2-a78e-f292626b276c

und weiter unten dann: "Module not found: Error: Can't resolve '@public-ui/react'.

[deleonio]

Hi @Chrisdo82,

der Grund ist ganz einfach. Wir haben ein in sich geschlossenen Workspace über alle Packages. Du musst neben dem pnpm i auch pnpm -r build initial machen, damit alles in sich verfügbar ist.

Danach in packages/themes/bmf wechseln und pnpm test-update ausführen.

Hinweis: Der aktuelle PR geht in den develop als in die v2. Ist das korrekt?

Ansonsten muss der PR auf release/1.7 geändert oder in beide gerichtet werden.

[Chrisdo82]

Ich wusste nicht dass der develop die v2 ist. Ändern geht aber nicht, sonst habt ihr alle Changes von v2 auch in der 1.7

[deleonio]

Hi @Chrisdo82,

die Änderungen sollen ja eh in v1 und in v2 gehen. Dann moven wir die Änderungen noch einmal separat in v1.

Wenn Du zukünftig in v1 und v2 was bringen möchtest, dann kannst Du genau diese Abzweigung nutzen, wo v1 und v2 noch identisch waren: release/v1to2-branch

Solange dabei keine Konflikte auftreten, kannst du eine Änderungen problemlos in beide Releases mergen.

[Chrisdo82]

macht ihr das oder soll ich nochmal einen PR mit den Änderungen erstellen?

Snapshots habe ich aktualisiert

[deleonio]

Bitte nicht die locks einchecken.

[Chrisdo82]

Eine Contribution Guideline in der sowas drin steht, wäre für die Zukunft echt hilfreich.

Ich möchte doch einfach nur die Styles updaten

[deleonio]

Hi Christian,

da ich an Deinen Fork nicht herankomme, konnte ich die Änderung nicht reinholen. Leider musste ich diesmal, alles an Dich zurückschieben.

Bei Aspekte sind gut für die Contributor-Readme:

• pnpm i und pnpm -r build sind wichtig (ist jetzt im postinstall fest drin)
• nur relevante Änderungen einchecken
• Features für v1 und v2 immer von release/v1to2-branch abzweigen

Ich versuche mal den Style 1zu1 auf v1 zu kopieren und update die Snapshots.