FriendsOfREDAXO / rexfactor

redaxo rector addon
MIT License
24 stars 1 forks source link

test with dark mode #3

Open staabm opened 1 year ago

staabm commented 1 year ago

Feature description / Feature Beschreibung

Bekannte probleme

skerbis commented 1 year ago

wo kommen die Stile der Diff-Ansicht her? Ich finde es nicht

staabm commented 1 year ago

Ich vermute du meinst diese hier

https://github.com/FriendsOfREDAXO/rexfactor/blob/main/lib/DiffHtml.php#L65

skerbis commented 1 year ago

ah ok, ich denke die brauchen wir lokal um was mit dem darkmode machen zu können. Oder wir nehmen generell eine dark Variante.

michael-kreatif commented 1 year ago

Markus nutzt ein externes npm Tool diff2html und da gibts noch kein Support für den Dark-Mode. https://github.com/rtfpessoa/diff2html https://github.com/rtfpessoa/diff2html/issues/403

Deswegen müsste man das entweder in dem Tool fixen oder was eigenes schreiben, wo man den diff bekommt und das Template selber generieren kann. Bin dafür, dass man das in dem externen Tool fixt, weils doch im Endeffekt nicht viel mit dem CMS ansich zu tun hat.

Habs grad schnell versucht aber leider noch nicht zum Laufen bekommen. Fehler: Screenshot 2023-06-09 at 09 09 16

staabm commented 1 year ago

wäre großartig wenn diff2html support für darkmode bekommen könnte oder jemand daran arbeiten würde.


vllt. könnte man zusätzlich als zwischenlösung im rexfactor irgendwie den light mode immer erzwingen, auch wenn der user dark mode eingestellt hat.

michael-kreatif commented 1 year ago

Habs nun doch direkt im redaxo addon gemacht. Hier der Pull Request: Hoffe es passt einigermaßen von der Optik her, bin kein Designer ;) https://github.com/FriendsOfREDAXO/rexfactor/issues/3

Diff2html mittlerweile zum Laufen gebracht. Beim diff2html hätte man die Möglichkeit eigene Templates zu entwickeln und könnte so auch mit den redaxo Klassen arbeiten, ist auch MIT deswegen könnte mans lokal builden. Aber ist wahrscheinlich vorerst den Aufwand nicht werd.

staabm commented 1 year ago

danke für den fix. schau es mir heute abend an