Project

General

Profile

Actions

Bug #14463

closed

TinyMCE: FormElement.size = <width>,<height>

Added by Carsten Rose over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Enis Nuredini
Target version:
Start date:
03.07.2022
Due date:
08.07.2022
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

  • Der Value fuer 'height' hat scheinbar keine Bedeutung mehr
  • Stattdessen gibt <width> die Hoehe vor.
  • Haben wir das angepasst weil der 'width' Parameter keinen Sinn macht im Bootstrap Design?
  • Oder ist da irgendwann ein Fehler reingerutscht und wir haben das nicht bemerkt - das kann eigentlich noch nicht so lange her sein.
  • Zumindest die Doku ist da nicht auf dem aktuellen Stand.
  • Ich bin nicht sicher ob Auto-Grow im TinyMCE supported ist (vergleiche Textarea) - ggfs. Benj fragen

Related issues

Related to QFQ - Bug #14455: TinyMCE: Activate save button after upload picture over drag&dropClosedEnis Nuredini01.07.2022

Actions
Related to QFQ - Feature #10782: Tiny MCE: Image UploadClosedEnis Nuredini22.06.2020

Actions
Actions #1

Updated by Carsten Rose over 1 year ago

  • Related to Bug #14455: TinyMCE: Activate save button after upload picture over drag&drop added
Actions #2

Updated by Carsten Rose over 1 year ago

  • Tracker changed from Support to Bug
  • Prio Planung set to No
Actions #3

Updated by Carsten Rose over 1 year ago

Actions #4

Updated by Enis Nuredini over 1 year ago

Description

  • Der Value für 'height' hat scheinbar keine Bedeutung mehr
    - Nein, in der Doku vom Branch F10782_TinyMce_Image_Upload_2 wird nur <min_height> und <max_height> angegeben, welche über FormElement.size eingetragen werden können. Funktioniert auch entsprechend mit Ausnahme des max_height, welcher von tinyMce ignoriert wird. Mit dem Resize Button lässt sich die Höhe vom Benutzer selbst anpassen. Denn Sinn des max_height vom TinyMce konnte ich nicht ermitteln. Die Breite wird vom Formular und Bootstrap vorgegeben was auch am meisten Sinn macht. Es kann auch nur eine Zahl im FormElement.size eingegeben werden, welcher dann für den min_height verwendet wird. Entsprechend habe ich die Doku so angepasst und max_height aus der Erklärung entfernt.
  • Stattdessen gibt <width> die Hoehe vor.
    - Dies wurde so in der bisherigen Version definiert im Code. Eine gezielte Eingabe des width und height wäre möglich, macht aber bei einem von Bootstrap definierten Formular keinen Sinn. Aktuell wird einfach die gesamt zur Verfügung stehende Breite verwendet.
  • Haben wir das angepasst weil der 'width' Parameter keinen Sinn macht im Bootstrap Design?
    - Implementation war bewusst so ausgeführt. Auf jedenfall weisen die Kommentare im Code und die hinterlassene Doku darauf hin.
  • Oder ist da irgendwann ein Fehler reingerutscht und wir haben das nicht bemerkt - das kann eigentlich noch nicht so lange her sein.
    - Siehe oben
  • Zumindest die Doku ist da nicht auf dem aktuellen Stand.
    - Die Doku ist vorhanden im entsprechenden Branch.
  • Ich bin nicht sicher ob Auto-Grow im TinyMCE supported ist (vergleiche Textarea) - ggfs. Benj fragen
    - Mithilfe der Statusbar unten kann die Höhe des Editors selbst geändert werden.
Actions #5

Updated by Carsten Rose over 1 year ago

  • Status changed from New to Closed
  • Target version changed from 355 to 22.10.0
Actions

Also available in: Atom PDF