Project

General

Profile

Actions

Feature #9052

open

Report: CodeMirror with SQL Syntax Highlight in FE

Added by Carsten Rose about 3 years ago. Updated 3 months ago.

Status:
New
Priority:
High
Assignee:
Carsten Rose
Target version:
Start date:
06.09.2019
Due date:
14.01.2022 (about 11 months late)
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No

Description

  • Edit Reports im Frontend mittels CodeMirror
  • Damit werden wir unabhaengig von T3 BE CodeMirror.
  • Switch um das FE Editing von QFQ Elemente im FE systemweit auszuschalten.
Actions #1

Updated by Benjamin Baer about 3 years ago

Ist einfach:
<textarea id="text" class="qfq-codemirror" name="codemirror" data-config='{ "mode": "text/x-sql", "lineNumbers": true }'></textarea>

javascript file codemirror muss eingebunden werden. Gab schon ein Template von Rafi namens codemirror.html im Template folder.

Actions #2

Updated by Carsten Rose about 3 years ago

Hallo Benj

Danke

CU
Carsten

Actions #3

Updated by Carsten Rose about 3 years ago

  • Due date changed from 11.09.2019 to 18.09.2019
  • Assignee changed from Carsten Rose to Benjamin Baer
Actions #4

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 18.09.2019 to 27.09.2019
Actions #5

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 27.09.2019 to 11.10.2019
Actions #6

Updated by Carsten Rose about 3 years ago

  • Priority changed from Normal to High
Actions #7

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 11.10.2019 to 15.11.2019
Actions #8

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 15.11.2019 to 25.10.2019
Actions #9

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 25.10.2019 to 15.11.2019
Actions #10

Updated by Benjamin Baer about 3 years ago

  • Due date changed from 15.11.2019 to 12.12.2019
Actions #11

Updated by Carsten Rose almost 3 years ago

  • Status changed from New to Some day maybe
Actions #12

Updated by Carsten Rose almost 3 years ago

  • Status changed from Some day maybe to New
Actions #13

Updated by Benjamin Baer almost 3 years ago

  • Status changed from New to Priorize
Actions #14

Updated by Benjamin Baer almost 3 years ago

http://webwork16.math.uzh.ch/bbaer/index.php?id=14

Hier funktioniert doch alles.

 Benoetigt:

  1.        file09 = typo3conf/ext/qfq/Resources/Public/Css/codemirror.css
  2.        file16 = typo3conf/ext/qfq/Resources/Public/JavaScript/codemirror.js
           file17 = typo3conf/ext/qfq/Resources/Public/JavaScript/code-mirror-mode/sql/sql.js
Actions #15

Updated by Carsten Rose almost 3 years ago

  • Target version changed from 146 to 55
Actions #16

Updated by Carsten Rose almost 3 years ago

  • Target version changed from 55 to next7
Actions #17

Updated by Carsten Rose over 1 year ago

  • Target version changed from next7 to next4
Actions #18

Updated by Benjamin Baer 11 months ago

  • Status changed from Priorize to Feedback
  • Assignee changed from Benjamin Baer to Carsten Rose
Actions #19

Updated by Carsten Rose 11 months ago

  • Due date changed from 12.12.2019 to 14.01.2022
  • Status changed from Feedback to ToDo
  • Assignee changed from Carsten Rose to Enis Nuredini
  • Target version changed from next4 to 23.1.0

Hallo Enis

Koenntest du das fuer die naechste Version integrieren?

Evtl. sollten noch Details angeschaut werden. Z.B.:

- Bei API Aufrufen darf das nie gerendert werden.
- Test das es auch mit mehreren QFQ Records pro Seite funktioniert.
- Wie sieht es aus wenn das in der linken (=schmalen) Spalte gerendert wird?
- Sollen wir eine Config-Option vorsehen, um die Funktion generell ein/auszuschalten?
- Beispiel machen (TypoScript),wie der Editor fuer spezifische Seiten ausgeschaltet werden kann (falls es Probleme gibt).
- ...

Actions #20

Updated by Carsten Rose 11 months ago

  • Status changed from ToDo to Priorize
Actions #21

Updated by Enis Nuredini 10 months ago

  • Status changed from Priorize to ToDo
Actions #22

Updated by Enis Nuredini 9 months ago

  • Status changed from ToDo to In Progress
Actions #23

Updated by Enis Nuredini 9 months ago

  • Status changed from In Progress to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose

Eine Lösung für den Codemirror ist im Branch zu finden.

Actions #24

Updated by Carsten Rose 7 months ago

  • Target version changed from 23.1.0 to 22.5.0
Actions #25

Updated by Carsten Rose 7 months ago

  • Assignee changed from Carsten Rose to Enis Nuredini
Actions #26

Updated by Carsten Rose 7 months ago

  • Target version changed from 22.5.0 to 22.5.feedback-enis
Actions #27

Updated by Carsten Rose 6 months ago

  • Damit der CodeMirror funktioniert muessen auch die CSS,JS Files eingebunden werden. Open for .min.
  • Da jetzt neu der CodeMirror immer angeboten wird (FE Report Editing), muss das in die Installationsanweisung. Open for .min.
  • Bitte die minimized (.min.) Versionen erstellen lassen (GRUNT).
  • Doku in Release.rst und Installation.rst anpassen auf die minimized Versionen.
  • Neues Ticket erstellen das wir alle JS und CSS mandatory Files in ein qfq.js und qfq.css einbauen. Es macht keinen Sinn das immer noch alle einzeln eingebunden werden.
Actions #28

Updated by Enis Nuredini 3 months ago

  • Status changed from Feedback to Ready to sync (develop)
  • Prio Planung set to No
Actions #29

Updated by Carsten Rose 3 months ago

  • Status changed from Ready to sync (develop) to New

1) Wenn ich es richtig verstanden haben sollten die minimized Versions jetzt da sein - fuer codemirror.min.js, code-mirror-mode/sql/sql.min.js stimmt das auch, aber codemirror.min.css scheint noch zu fehlen.

-Es besteht aktuell keine Möglichkeit über Gruntfile CSS-Files zu minimizen. Bisherige CSS Files von uns werden auch nicht über Gruntfile minimized. bootstrap.min.css wurde schon so mitgeliefert und nicht über Gruntfile erstellt.

Hinweis: wenn codemirror.min.css gemacht ist, muessen folgendes File im UZH CD (https://git.math.uzh.ch/typo3/uzh_cd_template) noch angepasst werden:

  • README.md (Bei "All Constant Options...")
  • Configuration/TypoScript/constants.txt Zeile 31

Danach benoetigen dann alle T3 Installationen das neueste QFQ & UZH_CD

2) [Comment 19] Auch in diesem Ticket fehlt eine Stellungnahme welche Teile bereits umgesetzt sind und welche nicht. Bitte zu jedem der Punkte kurz schreiben ob 'done':

  • Bei API Aufrufen darf das nie gerendert werden.
    -Sehe hier keine Möglichkeit dass dies passieren kann, da Editor in separatem Fenster geöffnet wird.
  • Test das es auch mit mehreren QFQ Records pro Seite funktioniert.
    -Mehrere Records werden parallel in eigene Fenster geöffnet.
  • Wie sieht es aus wenn das in der linken (=schmalen) Spalte gerendert wird?
    -Ist hinfällig da Records in eigene Fenster geöffnet werden.
  • Sollen wir eine Config-Option vorsehen, um die Funktion generell ein/auszuschalten?
    -Config-Option ist schon im letzten Merge implementiert.
  • Beispiel machen (TypoScript),wie der Editor fuer spezifische Seiten ausgeschaltet werden kann (falls es Probleme gibt).
    -Probleme auf Seiten werden allgemein vermieden da der Editor nicht auf der gleichen Seite geöffnet wird.

3) [Comment 27]

Damit der CodeMirror funktioniert muessen auch die CSS,JS Files eingebunden werden.

done: via UZH_CD

Da jetzt neu der CodeMirror immer angeboten wird (FE Report Editing), muss das in die Installationsanweisung.

done

Bitte die minimized (.min.) Versionen erstellen lassen (GRUNT).

done: codemirror.js, sql.js. Open: codemirror.css
- Minimized CSS aktuell nicht möglich über Gruntfile, siehe oben.

Doku in Release.rst und Installation.rst anpassen auf die minimized Versionen.

done

Neues Ticket erstellen das wir alle JS und CSS mandatory Files in ein qfq.js und qfq.css einbauen.

Open - koennte man mit Benj nochmal besprechen - aktuell ist das UZH_CD erweitert so das alles, ausser Fabric, via UZH_CD eingebunden wird.

-Mit Benj schon besprochen: Es wurden mal mehrere JS/CSS Files kombiniert und dies führte zu verschiedenen Fehlern. Weshalb von Benj abgeraten wird eine solche Umsetzung weiterhin zu verfolgen.

Actions #30

Updated by Enis Nuredini 3 months ago

  • Assignee changed from Enis Nuredini to Carsten Rose
Actions

Also available in: Atom PDF