gaenseklein / slidenotes

0 stars 0 forks source link

context-feld mit "select a markup" (default) ändert sich, wenn es eine selection gibt #63

Closed gaenseklein closed 5 years ago

gaenseklein commented 5 years ago

das context-feld bekommt einen default, welcher immer angezeigt wird wenn nix anderes da ist. der default ändert sich, je nachdem ob es eine selection im text gibt (text ausgewählt ist) oder ob nur ein cursor da ist. wenn der cursor innerhalb eines aktiven objekts ist (bspw. image, chart) oder diesen in einer selection beinhaltet soll sich das context-feld dementsprechend anpassen und nicht mehr den default sondern den zum jeweils aktiven objekt gehörigen kontext anzeigen. kontext-feld erstellen für images charts code latex ...

gaenseklein commented 5 years ago

@jochmann im pdf ist das context-feld auf der linken seite, über der sidebar und verdeckt aber nicht links die buttons. soweit klar. dann aber ist es mal unter der aktuellen cursorline bzw. wenn das insert-feld auf ist auch unter diesem. beim bild-context hast du es dann aber auf einmal über die cursorline gepackt. hast du da eine logik drinne, die ich grad nicht sehe? oder war das bild ein versehen und sollte eigentlich auch unter die cursorline?

des weiteren ist mir dabei aufgefallen, dass das die sidebar ganz schön voll macht - von der eigentlichen sidebar ist dann irgendwann relativ wenig zu sehen. ich würd das aber erstmal fertig machen und dann sehen wir uns das an ob uns das wirklich gefällt? mir fallen dann zwei optionen ein, wie wir damit umgehen:

  1. wir packen weniger default-content in das context-feld (nicht immer anzeigen lassen sondern nur wenn auch was da ist - default wäre dann nichts anzeigen)
  2. wir machen das insert-menü nur dann aufklappbar, wenn man an der stelle auch was inserten kann. bspw. wenn der cursor auf einem bild ist ist das kontext-feld sinnvoll (das bild darstellen etc.) - das insert-feld allerdings nicht - denn in ein image kannst du keine sections einfügen (bzw. solltest du nicht weil sonst das image nicht mehr funktioniert). das sind nicht viele fälle, aber in den fällen wäre es praktisch.
jochmann commented 5 years ago

die Logik, wann das Kontext-Feld über den Zeilenanzeiger rutscht ist: Standard ist unterhalb, wenn mehr Platz benötigt wird rutscht sie nach oberhalb. Begrenzung ist bündig mit Höhe des Textfeldes.

angezeigt werden soll das Kontext-Feld nur, wenn Text markiert ist oder das Menü ausgeklappt.

On 11. Dec 2018, at 19:43, gaenseklein notifications@github.com wrote:

@jochmann https://github.com/jochmann im pdf ist das context-feld auf der linken seite, über der sidebar und verdeckt aber nicht links die buttons. soweit klar. dann aber ist es mal unter der aktuellen cursorline bzw. wenn das insert-feld auf ist auch unter diesem. beim bild-context hast du es dann aber auf einmal über die cursorline gepackt. hast du da eine logik drinne, die ich grad nicht sehe? oder war das bild ein versehen und sollte eigentlich auch unter die cursorline?

des weiteren ist mir dabei aufgefallen, dass das die sidebar ganz schön voll macht - von der eigentlichen sidebar ist dann irgendwann relativ wenig zu sehen. ich würd das aber erstmal fertig machen und dann sehen wir uns das an ob uns das wirklich gefällt? mir fallen dann zwei optionen ein, wie wir damit umgehen:

wir packen weniger default-content in das context-feld (nicht immer anzeigen lassen sondern nur wenn auch was da ist - default wäre dann nichts anzeigen) wir machen das insert-menü nur dann aufklappbar, wenn man an der stelle auch was inserten kann. bspw. wenn der cursor auf einem bild ist ist das kontext-feld sinnvoll (das bild darstellen etc.) - das insert-feld allerdings nicht - denn in ein image kannst du keine sections einfügen (bzw. solltest du nicht weil sonst das image nicht mehr funktioniert). das sind nicht viele fälle, aber in den fällen wäre es praktisch. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gaenseklein/slidenotes/issues/63#issuecomment-446314457, or mute the thread https://github.com/notifications/unsubscribe-auth/AB6hBOlHkSAHnukcMBtoZ4AbuDTcxaTQks5u3_zRgaJpZM4ZHrXz.

gaenseklein commented 5 years ago

ah ok, also ist das context-field eigentlich teil des insert-blocks - wird nur grafisch ein wenig anders dargestellt? fraglich ist nach wie vor der umgang bei aktivem objekt = bild-tag. hier machen die "insert"-funktionen keinen sinn weil sie den bild-tag zerstören würden. alternative zum anzeigen bei aufgeklapptem insert-menü wäre, wenn es nach einer zeit des stillstands angezeigt wird - bspw. user hat 5 sekunden keine eingabe gemacht oder ähnliches... ich setze das erstmal so um wie oben vorgeschlagen, also nur wenn insert-feld auf ist und wir gucken uns das mal an. (in der pdf war das nicht so)

gaenseklein commented 5 years ago

ich habs jetzt umgesetzt. schau es dir an wenn du zeit hast. glücklich bin ich nicht damit. ich denke, wenn du es so verwenden willst wie ich dich verstehe ist es ja jetzt an das insert-menü gebunden. wie oben bereits beschrieben macht das insert-menü bei bestimmten selections keinen sinn - bspw. bei einem image, aber auch bei allen sections, welche keinen md-code erlauben (also chart, table und latex zur zeit). bisher war das context-field nur als "hilfe" gedacht, d.h. es werden die entsprechenden hilfe-texte angezeigt. bei sachen wie image war bereits auswahl und vorschau von aktuellem bild zu sehen. da gab es auch noch nicht das insert-menü, daher hat das da sinn gemacht. jetzt gibt es aber ein insert-menü und das hat optionen, mit welchen gearbeitet werden kann, welche aber wie oben beschrieben bei sections, welche keinen md-text erlauben sowie bei bild-tags keinen sinn ergeben. daher wäre es doch ganz gut, das zu berücksichtigen und irgendwie umzusetzen - bspw. dass das insert-menü sich ändert, wenn es auf einem bild-tag geöffnet wird und dort den bildupload bereitstellt - oder anderes beispiel wenn es auf einer chart ist, dass du dort per buttons den chart-typen auswählen kannst. (ist grad per context-field gelöst.) und das context-field ist ein reines hilfsfeld, welches tipps darstellt (wie bspw. den satz "select a markup..."

sprich: das kontext-feld und das insert-menü neu denken, als gesamt-paket betrachten und komplett neu strukturieren. auch die buttons sollten dann, wo sie nicht verwendet werden können, entweder verschwinden oder (imho besser:) ausgegraut werden.

gaenseklein commented 5 years ago

hier mal eine idee, wie das insert-menü für chart aussehen könnte (das kontext-feld ist da noch wie momentan): chart-insertmenu

jochmann commented 5 years ago

die Logik werden wir uns ja auch wegen der Performanz noch mal ansehen müssen. Ich schau mal bei den anderen issues, was mir dazu einfällt.

Am 17.12.2018 um 18:30 schrieb gaenseklein notifications@github.com:

ich habs jetzt umgesetzt. schau es dir an wenn du zeit hast. glücklich bin ich nicht damit. ich denke, wenn du es so verwenden willst wie ich dich verstehe ist es ja jetzt an das insert-menü gebunden. wie oben bereits beschrieben macht das insert-menü bei bestimmten selections keinen sinn - bspw. bei einem image, aber auch bei allen sections, welche keinen md-code erlauben (also chart, table und latex zur zeit). bisher war das context-field nur als "hilfe" gedacht, d.h. es werden die entsprechenden hilfe-texte angezeigt. bei sachen wie image war bereits auswahl und vorschau von aktuellem bild zu sehen. da gab es auch noch nicht das insert-menü, daher hat das da sinn gemacht. jetzt gibt es aber ein insert-menü und das hat optionen, mit welchen gearbeitet werden kann, welche aber wie oben beschrieben bei sections, welche keinen md-text erlauben sowie bei bild-tags keinen sinn ergeben. daher wäre es doch ganz gut, das zu berücksichtigen und irgendwie umzusetzen - bspw. dass das insert-menü sich ändert, wenn es auf einem bild-tag geöffnet wird und dort den bildupload bereitstellt - oder anderes beispiel wenn es auf einer chart ist, dass du dort per buttons den chart-typen auswählen kannst. (ist grad per context-field gelöst.) und das context-field ist ein reines hilfsfeld, welches tipps darstellt (wie bspw. den satz "select a markup..."

sprich: das kontext-feld und das insert-menü neu denken, als gesamt-paket betrachten und komplett neu strukturieren. auch die buttons sollten dann, wo sie nicht verwendet werden können, entweder verschwinden oder (imho besser:) ausgegraut werden.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gaenseklein/slidenotes/issues/63#issuecomment-447929147, or mute the thread https://github.com/notifications/unsubscribe-auth/AB6hBGAt7jEwmhViNt_7wJK7dLR_ns1fks5u59SjgaJpZM4ZHrXz.

gaenseklein commented 5 years ago

Wie in dem Beispiel zu sehen wäre es doch für die GUI am praktischsten und für den User am hilfreichsten wenn wir das insert-menü dynamisch anpassen können. Hilft bei der Erstellung der Chart doch sehr, wenn das so ausgewählt werden kann. Soll ich das mal umsetzen, dass Sections vom Default abweichen können?

jochmann commented 5 years ago

macht vermutlich Sinn, ja

Am 06.03.2019 um 03:09 schrieb gaenseklein notifications@github.com:

Wie in dem Beispiel zu sehen wäre es doch für die GUI am praktischsten und für den User am hilfreichsten wenn wir das insert-menü dynamisch anpassen können. Hilft bei der Erstellung der Chart doch sehr, wenn das so ausgewählt werden kann. Soll ich das mal umsetzen, dass Sections vom Default abweichen können?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gaenseklein/slidenotes/issues/63#issuecomment-469937947, or mute the thread https://github.com/notifications/unsubscribe-auth/AB6hBN9ZiokCodQOJx-YnqG0JLzOSUrdks5vTyNQgaJpZM4ZHrXz.

gaenseklein commented 5 years ago

chartinsertmenu: chartinsertmenu chartinsertmenu mit hover über auswahl von charts: chartinsertmenuhover

gaenseklein commented 5 years ago

um das bei dir zu sehen musst du warscheinlich erst das neue chartist-plugin aktivieren (unter options -> advanced options -> extensions -> charts with chartist )

jochmann commented 5 years ago

cool!

Am 10.03.2019 um 18:59 schrieb gaenseklein notifications@github.com:

um das bei dir zu sehen musst du warscheinlich erst das neue chartist-plugin aktivieren (unter options -> advanced options -> extensions -> charts with chartist )

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gaenseklein/slidenotes/issues/63#issuecomment-471328677, or mute the thread https://github.com/notifications/unsubscribe-auth/AB6hBEZf5sX3Yh46UaNqk-bTkVGMbcmQks5vVUgWgaJpZM4ZHrXz.

gaenseklein commented 5 years ago

context-feld kommt weg, issue ist daher geschlossen. folgeissue: #78 neues context-menu (ehemals insertmenu)