Text in markdown field is not displayed on following tabs

it-at-m / digiwf-project

Project repo for the DigiWF project automation platform.

MIT License

10 stars 1 forks source link

Text in markdown field is not displayed on following tabs #442

Closed dominikhorn93 closed 1 year ago

dominikhorn93 commented 1 year ago

Description

Text in a markdown field is only displayed when the focus is on the field. For example, by clicking in the field.

Reproduce

Call task with markdown field
Enter text in markdown field
complete task and pass to next task in priozess
call new task
no text appears in markdown (only on second or following tabs)
if you click in the markdown field, the text appears

Note: the markdown is not located on the first stepper of the form

internal issues:

StephanStrehlerCGI commented 1 year ago

Bug konnte reproduziert werden: https://github.com/it-at-m/vuetify-jsonschema-form/blob/bugfix/442-markdown-text-is-not-visible/442-markdown-editor-bug-reproduction.html Dreckiger Workaround auf nativem JavaScript / HTML Basis konnte gefunden werden. Bug liegt an easymde / codemirror.

Bugfix mittels elegantem Weg so einfach nicht möglich. Vielmehr müssten wir codemirror bug fixen. @darenegade muss entscheiden, ob aufwendiger Bugfix sinnvoll ist.

LenaB34 commented 1 year ago

Umsetzung & Darstellung noch unklar. Abstimmung mit Rene & Julia erfolgt für die Darstellung LHM. Schätzung aktuell nicht machbar, wird noch erfolgen

darenegade commented 1 year ago

@StephanStrehlerCGI Wo würden wir den "dreckigen" Fix einbauen?

VJSF macht uns immer wieder Probleme. Wir werden wohl langfristig wechseln müssen (#424). Daher würde ich die schnelle Lösung als Temp.-Lösung ins Auge fassen. Bin mir aber noch nicht sicher, wo wir das setzen des Values dann integrieren in der Lib.

StephanStrehlerCGI commented 1 year ago

Wir müssten in den vuetify tabs einen click listener einbauen und immer wenn dieser geklickt wird (also der Tab gewechselt wird) den Dom / das Schema durchgehen und schauen ob / wo wir EasyMDE haben. dann müssten wir von diesem den aktuellen Wert auslesen und wieder setzen.

darenegade commented 1 year ago

Braucht man wirklich einen Listener? Mir erscheint das nur nach einem Init-Problem.

Wenn ich dein Beispiel so anpasse, dass ich das Element Ein-/Ausblende, dann besteht das Problem nur beim ersten Mal.

Bei den Tabs einen Listener zu bauen, der alle Markdown-Fields kennt oder findet, wäre nicht so toll. Lass uns mal darüber reden.

darenegade commented 1 year ago

https://codepen.io/darenegade/pen/yLRbjGK

darenegade commented 1 year ago

@StephanStrehlerCGI @lehju

Hiermit konnte ich das Problem lösen:

Siehe CodePen

Könnt ihr das verifizieren?

StephanStrehlerCGI commented 1 year ago

@darenegade

Mach ich. ich bilde mir ein, dass schon probiert zu haben. aber dein beispiel funktioniert ja. ich schaue es mir gleich mal an

darenegade commented 1 year ago

Abstimmung zwischen @dominikhorn93 und @StephanStrehlerCGI notwendig für release der Änderung