Hi,
ich habe bei mir den Wert für die Größe des Codemirror auf "auto" gesetzt und bin hoch zufrieden, dass ich jetzt auch längere Codestücke komplett angezeigt bekomme.
Aus diesem Grund möchte ich mal den Vorschlag zur Diskussion stellen, dies standardmäßig zu aktivieren:
https://codemirror.net/doc/manual.html#styling
CodeMirror
The outer element of the editor. This should be used for the editor width, height, borders and positioning. Can also be used to set styles that should hold for everything inside the editor (such as font and font size), or to set a background. Setting this class' height style to auto will make the editor resize to fit its content (it is recommended to also set the viewportMargin option to Infinity when doing this.
www/codemirror/codemirror.css:
Zeile 1: .CodeMirror{[...]height:auto[...]} ( Warum ist diese Datei eigtl im Blocksatz formatiert? )
www/codemirror/codemirror.js:
Zeile 229: [...]"viewportMargin",Infinity,[...]
Grüße
Stephan
Hi Stephan,
ungern, zumindest so wie du es hier vorgeschlagen hast.
Diese beiden Datein sind unformatiert da sie die minimized Versionen der standardmäßig mit Codemirror ausgelieferten Dateien sind.
Da diese bei jedem Codemirror Update überschrieben werden ist hier der falsche Platz für diese Änderung.
Entweder das ganze über die fhem.js und die <theme>.css (z.B. blackboard.css) als Standard für alle realisieren,
oder besser über die fhem_codemirror.js realisieren und über codemirrorParam individuell aktivierbar gestalten, sowie statt Auto auch andere Height/Width Werte konfigurierbar machen. (Das ganze hatte ich sogar schonmal vor einiger Zeit angefangen einzubauen, allerdings bisher nicht abgeschlossen :-\)
Gruß
Claudiu
Hi, ich habe mich mal an einer Änderung versucht. Anbei ist die fhem_codemirror.js inclusive meiner Änderungen sowie eine patch-datei.
Schau mal bitte drüber, ob das so okay wäre, ist quasi meine erste :-) Bei meinen Tests hats funktioniert.
Ich hab mit dem Befehl diff -Naur original.c original_new.c > original.c.patch eine patch-Datei erzeugt. War das so korrekt?
Danke!
Stephan
Hi Stephan,
danke dir, das sieht doch gut aus! Ich verwende zwar immer "svn diff", aber für diesen Zweck taugt das :D
Habs mal eingebaut und eingecheckt (bisschen abgeändert): https://sourceforge.net/p/fhem/code/11286/
Und im Wiki kurz beschrieben: http://www.fhemwiki.de/wiki/Konfiguration#Integrierter_Editor
Gruß
Claudiu