Codeeditor_XH

[lck]

Die Einstellung als Editor in der Konfiguration flutscht auch, ist dann auch beim Template und Stylsheets aktiv, super.

[frase]

2) Was aber nicht stimmt, ist der Hinweis zu "Suche vorheriges" per Shift-Ctrl-F. Das ist Shift-Ctrl-G.
Shift-Ctrl-F ist "Ersetzen". Ich glaube, das habe ich schon mal hier irgendwo gemeldet, vor langer Zeit.

In der Pluginhilfe steht es allerdings korrekt.

3) Auch die Hervorhebung des momentan aktiven Suchtextes (per Strg+G) ist fast nicht erkennbar, ist nur leicht heller. Geht das besser?

Das hängt vom gewählten Theme ab. Du könntest dir auch ein eigenes basteln.
Dumm nur, dass bei der Seitenbearbeitung das Theme fest vorgegeben ist, sodass man hier evtl. das eine und bei Konfiguration → Template (oder Stylesheet) ein anderes Theme hat.
Man müsste die Tiny-Init-Datei händisch anpassen.
Dieser Fakt sört mich a weng.

Übrigens:
Der Fokus wird tatsächlich bei der Seitenbearbeitung verloren, sobald man in der Quelltextansich OK klickt.

[frase]

Hilfedatei Tastenkürzel:

Einige der angeführten Tastenkürzel funktionieren innerhalb des TinyMCE nicht, sondern nur, wenn man in einem Fenster mit nur dem Codeeditor ist (Template, Stylesheet usw.)

Vollbild umschalten → Esc → nicht im Tiny
Faltung umschalten → Ctrl-Q → gar nicht
Zeilenumbruch umschalten → Alt-W → nicht im Tiny
Vorschau umschalten → Crtl-P → normal gar nicht → im Tiny wir der Druck-Dialog aufgerufen (Firefox)

Bilder durchsuchen → Ctrl-I
Downloads durchsuchen → Ctrl-L
Mediadateien durchsuchen → Ctrl-M
Benutzerdateien durchsuchen → Ctrl-U

Die letzten 4 Kürzel funktionieren im TinyMCE nicht.
In den anderen Fenstern funktionieren sie nur, wenn man Popups explizit zulässt.

[cmb]

Nur kurz zwischendurch: der CodeMirror im TinyMCE hat nicht mit dem Codeeditor_XH zu tun (außer, dass er dessen CodeMirror-Code verwendet). Tastaturkürzel und Suchfunktion können sich also anders verhalten; da müsste aber eher im TinyMCE-CodeMirror-Plugin nachgebessert werden.

[frase]

Nur kurz zwischendurch: der CodeMirror im TinyMCE hat nicht mit dem Codeeditor_XH zu tun (außer, dass er dessen CodeMirror-Code verwendet). Tastaturkürzel und Suchfunktion können sich also anders verhalten; da müsste aber eher im TinyMCE-CodeMirror-Plugin nachgebessert werden.

Ja, das ist schon klar.
Ich zeige ja auch nur die Probleme auf, wie sie einem Anwender so bei der Arbeit begegnen.
Tiny mit CodeMirror → bedingt Codeeditor_XH.
Und das Zusammenspiel funktioniert eben nicht.
Da muss Manu ran (muss natürlich nicht ), wenn er wieder verfügbar ist.

Es müsste doch möglich sein, einige Einstellungen vom einen im anderen Plugin zu übernehmen.
Da der Tiny mit CodeMirror Codeeditor_XH erfordert, bestehen da Abhängigkeiten.

[cmb]

Tiny mit CodeMirror → bedingt Codeeditor_XH.

Ja, aber die Teilen sich nur das codemirror Verzeichnis. Es wäre durchaus möglich den TinyMCE mit eigenem CodeMirror auszuliefern; dann wäre die Abhängigkeit gebrochen, und es würde wohl niemand mehr davon ausgehen, dass alle Funktionen von Codeeditor_XH auch im TinyMCE verfügbar sind. Viele sind es halt nicht.

Oder man versucht halt eine volle Integration; das dürfte knifflig werden (falls überhaupt möglich), und zumindest solange die Zukunft des CodeMirror-Plugins für TinyMCE etwas unklar ist, möchte ich da keine Arbeit investieren.

Dumm nur, dass bei der Seitenbearbeitung das Theme fest vorgegeben ist, sodass man hier evtl. das eine und bei Konfiguration → Template (oder Stylesheet) ein anderes Theme hat.

Bitte etwas mit der Formulierung aufpassen. Wird der Codeeditor_XH als Inhaltseditor konfiguriert, dann sollte das eingestellte Theme auch verwendet werden. Nur die CodeMirror-Quelltextansicht des TinyMCE ignoriert diese Einstellung. Der TinyMCE könnte als Workaround auf $plugin_cf['codeeditor']['theme'] zurückgreifen; ich könnte dafür im Codeeditor_XH auch eine API zur Verfügung stellen.

Auf jeden Fall könnte die TinyMCE 5 Hilfe noch etwas nachgebessert werden. Freiwillige vor!

[frase]

Wird der Codeeditor_XH als Inhaltseditor konfiguriert, dann sollte das eingestellte Theme auch verwendet werden.

Das ist auch so - also in Ordnung.

Nur die CodeMirror-Quelltextansicht des TinyMCE ignoriert diese Einstellung. Der TinyMCE könnte als Workaround auf $plugin_cf['codeeditor']['theme'] zurückgreifen; ich könnte dafür im Codeeditor_XH auch eine API zur Verfügung stellen.

Wenn alleine das ohne größeren Aufwand gelöst werden könnte, dann wär schon sehr viel erreicht.
Die unterschiedlichen Tastenkürzel könnte man verschmerzen, denke ich.
Wenn die Optik des Quelltextes im gesamten System identsch wäre, gibt's auch keine Meckerei mehr
Also:
Nochmal vielen Dank, Christoph, für deine Erläuterungen.
$plugin_cf['codeeditor']['theme'] → als Theme im Tiny = alles OK.

[frase]

Auf jeden Fall könnte die TinyMCE 5 Hilfe noch etwas nachgebessert werden. Freiwillige vor!

Das kann wohl erst ordentlich gemacht werden, wenn feststeht, wie und was realisiert wird.

[cmb]

$plugin_cf['codeeditor']['theme'] → als Theme im Tiny = alles OK.

Vermutlich genügt dafür schon:

Code: Select all

 plugins/tinymce5/classes/Editor.php                  | 5 +++++
 plugins/tinymce5/inits/init_dropdown-codemirror.json | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/plugins/tinymce5/classes/Editor.php b/plugins/tinymce5/classes/Editor.php
index 66948f4c..af93bfd1 100644
--- a/plugins/tinymce5/classes/Editor.php
+++ b/plugins/tinymce5/classes/Editor.php
@@ -145,6 +145,11 @@ protected static function config($config, $selector)
 
         $parsedconfig = str_replace('%STYLESHEET%',$pth['folder']['template'] . 'stylesheet.css',$parsedconfig);
         $parsedconfig = str_replace('%CMSIMPLE_ROOT%',CMSIMPLE_ROOT,$parsedconfig);
+
+        /*
+         * Use the codemirror theme configured for Codeeditor_XH if available.
+         */
+        $parsedconfig = str_replace('%CODEMIRROR_THEME%',$plugin_cf['codeeditor']['theme'],$parsedconfig);
         
         /* 
          * Enable the file_picker_callback in Admin mode only (functions/callbacks not possible in JSON)
diff --git a/plugins/tinymce5/inits/init_dropdown-codemirror.json b/plugins/tinymce5/inits/init_dropdown-codemirror.json
index 058fdd31..5cc1a518 100644
--- a/plugins/tinymce5/inits/init_dropdown-codemirror.json
+++ b/plugins/tinymce5/inits/init_dropdown-codemirror.json
@@ -37,7 +37,7 @@
   "images_reuse_filename": false,
   "codemirror":{
     "config": {
-      "theme": "zenburn"
+      "theme": "%CODEMIRROR_THEME%"
     }
   }
 }

Auf jeden Fall könnte die TinyMCE 5 Hilfe noch etwas nachgebessert werden. Freiwillige vor!

Das kann wohl erst ordentlich gemacht werden, wenn feststeht, wie und was realisiert wird.

Man könnte aber auch einfach die TinyMCE4-Hilfe kopieren und anpassen. Oder vielleicht etwas wie im Wiki. Mir geht es nicht darum, dass alles gleich detailliert beschrieben wird, sondern darum, dass der Leser überhaupt mal weiß, um was es geht (und sich nicht wundert, wo er beim Aufruf der Hilfe denn gelandet ist), und um einen Ort zu haben, wo ggf. Details dokumentiert werden können.

[cmb]

1) Getestet mit Firefox, Edge und Iron (Chrome Klon).
Ist der Fokus/Cursor im Codemirror-Fenster, aktiviert F3 sogar für die Browsersuche. Mit STRG+F die Browsersuche zu aktivieren, bedarf es einen Klick ausserhalb des Codemirror-Fensters. Also alles normal.

2) Was aber nicht stimmt, ist der Hinweis zu "Suche vorheriges" per Shift-Ctrl-F. Das ist Shift-Ctrl-G.
Shift-Ctrl-F ist "Ersetzen". Ich glaube, das habe ich schon mal hier irgendwo gemeldet, vor langer Zeit.

3) Auch die Hervorhebung des momentan aktiven Suchtextes (per Strg+G) ist fast nicht erkennbar, ist nur leicht heller. Geht das besser?

Da geht es wohl um die Quellcode-Ansicht im TinyMCE; da gilt das bereits Gesagte (TL;DR: Theme ließe sich anpassen; besondere Tastenkürzel und Features von Codeeditor_XH eher nicht).