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

Theme Update #105

Closed
andi-blafasl opened this issue Feb 17, 2022 · 17 comments
Closed

Theme Update #105

andi-blafasl opened this issue Feb 17, 2022 · 17 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@andi-blafasl
Copy link
Collaborator

Es gibt ein aktualisiertes Release vom relearn theme mit Bugfixes.
Ich hab dafür einen Branch update-theme angelegt. Bitte testen. Wäre für das nächste BITS Release interessant.

@BITS-Editor denk nach dem checkout an den "git submodule update --init --recursiv" 😉

@andi-blafasl andi-blafasl self-assigned this Feb 17, 2022
@BITS-Editor
Copy link
Collaborator

Danke! Schaue ich mir an, ggf. dauert das aber.

@BITS-Editor BITS-Editor added the enhancement New feature or request label Feb 20, 2022
@BITS-Editor
Copy link
Collaborator

Soweit ich das erkenne, wurde in der 2.9.6 nur ein Bug hinsichtlich der Pfeilnavigation gefixt. Diese funktioniert bei uns, auch sonst konnte ich keine Fehler feststellen.

Wie wollen wir mit weiteren Updates des Relearn themes umgehen? Ich denke, dass wir, wenn wir bei uns keine Fehler feststellen, nicht alleine wegen eines Updates ein neues Release von BITS erstellen sollten. Es gibt ja seit der 2.9.6 schon diverse weitere Updates und einen Wechsel zur v3.

Ich würde bei einem aus anderen Gründen geplanten Release von BITS dann die neueste Version von Relearn übernehmen bzw. testen. Wäre das OK?

@andi-blafasl
Copy link
Collaborator Author

Ja, ich würde auch nur bei einer Aktualisierung von BITS auf ein Update vom Theme schauen, oder eben wenn es Probleme gibt und die Upstream evtl. gefixed sind. Nur wegen einem neuen Theme machen wir kein neues BITS release.

@andi-blafasl
Copy link
Collaborator Author

Ok, die 3.0.0 hab ich noch garn nicht bemerkt. 😉
Was ich gerade gesehen hab: bei der 2.9.0 gab es wohl einen breaking-Change mit ref/relref und das nutzen wir doch auch im content. Müssen wir noch testen oder hast du schon gesehen das es funktioniert?

@BITS-Editor
Copy link
Collaborator

Ich habe nur die Seiten(pfeil)navigation und die Menünavigation getestet. Was müsste ich noch für die 2.9.0 testen?

@andi-blafasl
Copy link
Collaborator Author

Die aktuelle Version is im Branch update-theme eingepflegt.

Es gibt jetzt auch einen Dark-Mode des default Themes, falls wir das übernehmen wollen ;-)

Know-Issues:

@andi-blafasl andi-blafasl changed the title Theme Update 2.9.6 Theme Update 4.1.0 Jun 18, 2022
@andi-blafasl
Copy link
Collaborator Author

Demo unter https://bits-training.github.io/training/index.html online

@andi-blafasl
Copy link
Collaborator Author

andi-blafasl commented Jun 20, 2022

Header-Icon bei den Lektions-Seiten wieder an den Rand
Fixed in e8982f1

@BITS-Editor
Copy link
Collaborator

BITS-Editor commented Jun 21, 2022

Theme Update 4.1.0 sieht grundsätzlich gut aus, bis auf zwei Punkte. Ich habe die Release-Notes der Änderungen ab 3.0 überflogen und https://bits-training.github.io/training/index.html getestet.

  1. z.B. unter https://bits-training.github.io/training/200-ansprechpersonen/ ragt die Überschrift in das Icon rein. Offenbar ist das noch nicht weit genug entfernt wie in der früheren Version. Andere Seiten sind auch betroffen, z.B. https://bits-training.github.io/training/110-lektion-mein-arbeitsplatz/05.vorsicht-bei-mobilen-datentraegern/
  2. schon in den früheren Versionen, aber auch in 4.1.0 gibt es auf der Ergebnis-Seite des Quiz offenbar keine Möglichkeit, per Tastatur die Fragen auszuwählen und die Links zu aktivieren. Dazu habe ich eine Issue beim Quiz-Repo unter Keyboard navigation on results page bonartm/quizdown-js#27 eröffnet. Dieses Problem habe ich auch in Quiz barrierefrei gestalten mit Tastaturnavigation #79 ergänzt.
  3. Wenn man CSS im Browser abschaltet, erscheint jetzt der Inhalt oben, die Navigation unten. Das ist eine Verbesserung gegenüber früher.
  4. Allerdings gibt es keine Navigationspfeile und
  5. außerdem gibt es unten zwei Dropdown-Boxen. Eine für die Sprachwahl (Englisch - Deutsch) und eine nur mit "BITS" gefüllt. Beide brauchen wir mE nicht.

@BITS-Editor BITS-Editor added this to the Version 6.2 milestone Sep 5, 2022
@andi-blafasl
Copy link
Collaborator Author

Theme auf Version 5.2.3 aktualisiert. Damit auch das Icon oben rechts neu implementiert, das sollte Punkt 1 beheben.

Die Navigations-Pfeile sind jetzt oben in der Brotkrumen-Leiste über FontAwesome Icons realisiert, ohne CSS gibt's die nicht.

Die beiden Dropdown-Boxen sind im Theme für die Sprach-Auswahl und eine für eine Design-Variante. Das wird im Normalzustand über CSS ausgeblendet wenn deaktiviert, aber der HTML-Code wird immer generiert. Lässt sich nicht abschalten. Ich würde die Design-Variante für die Barriere-Armut empfehlen. Dann kann man ein "High-Kontrast" Theme machen und einen Dark-Mode 😉

@andi-blafasl andi-blafasl changed the title Theme Update 4.1.0 Theme Update Oct 1, 2022
@BITS-Editor
Copy link
Collaborator

Ok.

Nur ein Punkt fällt mir hier dann noch auf: Auf der Lektionseinstiegsseite ist das Icon deutlich näher in der Mitte auf den Unterseiten. Kann man das noch vereinheitlichen, also am besten so wie auf den Unterseiten darstellen?

@andi-blafasl
Copy link
Collaborator Author

andi-blafasl commented Oct 3, 2022

Das ist leider nicht so einfach möglich. Da die Lektions-Startseiten nutzen einen komplett anderen Aufbau der Überschrift. Das Icon wird aber immer auf die selbe weise eingebunden. Und damit sieht es unterschiedlich aus. Meiner Meinung nach ist das OK so, da es auf den Startseiten prominenter platziert sein soll.

@andi-blafasl
Copy link
Collaborator Author

Achso, das Icon ist immer an der gleiche stelle: Rechts am Rand des Textkörpers. Der Textkörper ist bei den Startseiten deutlich schmaler und deshalb geht das Icon wirklich nicht weiter nach rechts. In der alten Version war das Icon ziemlich schlecht an den rechten Rand der Seite gepfuscht. Deshalb kam es da auch zu diesen Überlappungen und so. Das ist bei der neuen Variante halt sauber im Layout integriert und deshalb an der neuen Stelle. Ich bekomm das nur so hin das es beim ändern der Browser-Breite sauber funktioniert.

@BITS-Editor
Copy link
Collaborator

Ok. Ab dafür.

@BITS-Editor
Copy link
Collaborator

Wir warten noch auf die v5.3
https://github.com/McShelby/hugo-theme-relearn/milestone/68

@BITS-Editor BITS-Editor reopened this Oct 6, 2022
@BITS-Editor
Copy link
Collaborator

Die 5.3.0 ist da.

https://github.com/McShelby/hugo-theme-relearn/releases/tag/5.3.0 und hat das Issue #307 umgesetzt.

Wir können daher mE jetzt das übernehmen und die Beta der v6.2 erstellen.

@andi-blafasl
Copy link
Collaborator Author

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants