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

test with dark mode #3

Open
staabm opened this issue Mar 13, 2023 · 7 comments
Open

test with dark mode #3

staabm opened this issue Mar 13, 2023 · 7 comments
Labels

Comments

@staabm
Copy link
Member

staabm commented Mar 13, 2023

Feature description / Feature Beschreibung

Bekannte probleme

  • diff Ansicht
@skerbis
Copy link
Member

skerbis commented Apr 1, 2023

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

@staabm
Copy link
Member Author

staabm commented Apr 1, 2023

@skerbis
Copy link
Member

skerbis commented Apr 1, 2023

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
Copy link
Contributor

Markus nutzt ein externes npm Tool diff2html und da gibts noch kein Support für den Dark-Mode.
https://github.com/rtfpessoa/diff2html
rtfpessoa/diff2html#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
Copy link
Member Author

staabm commented Jun 9, 2023

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
Copy link
Contributor

Habs nun doch direkt im redaxo addon gemacht.
Hier der Pull Request:
Hoffe es passt einigermaßen von der Optik her, bin kein Designer ;)
#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
Copy link
Member Author

staabm commented Jun 9, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants