Some small code improvement suggestions

[danielnolde]

README.md

Sinnvoll wäre eine Ergänzung um einen Hinweis wie:

yaml_editor currently works only on admin pages. To use it on node/entity forms, make sure you configure to "Use the administration theme when editing or creating content" at /admin/appearance.

yaml-editor.js

Wäre statt

      // Check if Ace editor is already available and load it from source cdn otherwise.
      if (typeof ace !== 'undefined') {
        initEditor();
      }
      else {
        $.getScript(drupalSettings.yamlEditor.source, initEditor);
      }

nicht sinnvoller Ace bei Bedarf zu laden, aber ihn danach auch zu nutzen (im obigen Original-Code erfolgte nach dem ersatzweisen Laden kein Aufruf von initEditor mehr.

      // Check if Ace editor is already available and load it from source cdn otherwise.
      if (typeof ace == 'undefined') {
        $.getScript(drupalSettings.yamlEditor.source, initEditor);
      }

      initEditor();

yaml_editor.module - yaml_editor_page_attachments

Wäre vermutlich sinnvoll, in dieser Fkt. einfach nur $attachments['#attached']['drupalSettings']['yamlEditor']['source'] = $source zu setzen, aber das ['#attached']['library'][] = 'yaml_editor/yaml_editor' dann besser im YamlEditorWidget::formElement - oder?
Dann könnte man sich a) die Einschränkung auf admin-pages ersparen ohne performance-Probleme, und b) wären die Sachen eher dort, wo sie auch hingehören (oder? ;)

[danielnolde]

Ich mache dazu vll demnächst noch nen PR, wenn's beliebt.