Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wrong info under plugin-info #10

Open
olape-git opened this issue Apr 4, 2022 · 13 comments · May be fixed by #12
Open

wrong info under plugin-info #10

olape-git opened this issue Apr 4, 2022 · 13 comments · May be fixed by #12
Labels
Milestone

Comments

@olape-git
Copy link
Member

image

@frase-git
Copy link
Contributor

Was ist daran falsch?

@olape-git
Copy link
Member Author

{{{expand 'Seite 1, Seite 2, Seite 3, Seite 4'}}} --> {{{expand('pages = Seite 1, Seite 2, Seite 3, Seite 4')}}}
{{{expand 'Seite X', 'Linktext'}}} --> {{{expand('pages = Seite X', 'headlines = Linktext')}}}

@frase-git
Copy link
Contributor

Danke. Klassischer Fall von Betriebsblindheit.

@olape-git olape-git added the bug label Apr 5, 2022
frase-git added a commit to frase-git/expandcontract that referenced this issue Apr 5, 2022
@frase-git frase-git linked a pull request Apr 5, 2022 that will close this issue
@frase-git
Copy link
Contributor

Übrigens: Dein Code hat auch ein paar Klammern zu viel.
Die Parameter müssen hier (eigenartigerweise) keine Klammern ( ) haben.
Oder, bin ich da schon wieder auf dem Holzweg?

@TN03
Copy link
Member

TN03 commented Apr 5, 2022

Irgendwo stand auch, dass bei Angabe eines Seitennamens die "erste gefundene" Seite genommen wird. Das stimmt ja. Normalerweise müssten Seiten mit gleichem Namen in unterschiedlichen Ebenen, bzw. unter unterschiedlichen Elternseiten trotzdem ansprechbar sein, indem man den Pfad entsprechend angibt: Allgemeine Frage/Antwort 1 Andere Frage/Antwort 1
Eventuell wäre ein Hinweis darauf ganz gut.

@TN03 TN03 added this to the v.1.1 milestone Apr 5, 2022
@frase-git
Copy link
Contributor

frase-git commented Apr 5, 2022

Eventuell wäre ein Hinweis darauf ganz gut.

Im Moment funktioniert das mit den Pfaden ja nicht - oder?
Also ist doch der Hinweis ausreichend, dass immer nur die erste im Seitenbaum gefundene Seite genommen wird.

Es steht übrigen hier:
https://expandcontract.cmsimple-xh.org/?Variante-2

@TN03
Copy link
Member

TN03 commented Apr 5, 2022

Im Moment funktioniert das mit den Pfaden ja nicht - oder?

Das müsste doch gehen: 'pages = Eine Seite/Seite1, Andere Seite/Seite 1' oder irre ich mich da???

@olape-git
Copy link
Member Author

Übrigens: Dein Code hat auch ein paar Klammern zu viel. Die Parameter müssen hier (eigenartigerweise) keine Klammern ( ) haben. Oder, bin ich da schon wieder auf dem Holzweg?

Ich schreibe es immer mit ().

Das müsste doch gehen: 'pages = Eine Seite/Seite1, Andere Seite/Seite 1' oder irre ich mich da???

Das glaube ich eigentlich nicht. Dann hätten wir pages mit $u umsetzen müssen, nicht mit $h.
Ich habe es nicht getestet, aber es sollte mich wundern.

@TN03
Copy link
Member

TN03 commented Apr 5, 2022

Das müsste doch gehen: 'pages = Eine Seite/Seite1, Andere Seite/Seite 1' oder irre ich mich da???

Das glaube ich eigentlich nicht. Dann hätten wir pages mit $u umsetzen müssen, nicht mit $h. Ich habe es nicht getestet, aber es sollte mich wundern.

Stimmt. Kann hier noch nicht gehen. Verwechselt, sorry.

Ich habe bereits an einer anderen Alternativ-Version gebastelt, die dann (optional) gar keine Unterseiten braucht, sondern alles aus einer Seite holt.
Momentan "passt" das aber noch nicht richtig zusammen. Weiß auch nicht, ob ich das weiter verfolge.

@olape-git
Copy link
Member Author

olape-git commented Apr 5, 2022

sondern alles aus einer Seite holt.

Ich finde das gerade gut, dass man den Inhalt etwas aufteilen kann.
Das macht das Bearbeiten leichter / übersichtlicher.
Wenn alles in einer Seite steht ist das Bearbeiten wesentlich schwerer, auch wenn man mal im Quelltext ist.

@TN03
Copy link
Member

TN03 commented Apr 5, 2022

Ich finde das gerade gut, dass man den Inhalt etwas aufteilen kann.

Ja, Ansichtssache. Mir sind das zu viele Seiten in der Navi. Da muss man ewig suchen, scrollen, Zweige öffnen. Eine Seite / je Akkordeon finde ich für meine (Kunden-)Projekte simpler. Gesplittet wird nach (konfigurierbarer) Überschrift.
Und ohne Edit / Preview - Links sind viele mit der jetzigen Lösung auch schon überfordert.

Quelltext? Das klappt bei meinen Durchnittsusern in der Regel so oder so nicht.

Und was die Übersicht angeht: alles in einer Seite macht es viel übersichtlicher und einfacher in der Bearbeitung.

BTW: Aktuell habe ich mal versucht "Snippets" (wie von Frank) bei einem Projekt zu vermitteln. Den Block am richtigen Platz im Quelltext zu positionieren funktioniert schon gut. Aber nur, so lange ich gerade erkläre und vorführe. Einen Tag später war alles vermurkst...

Aber das muss ja hier in diesem Plugin nicht so sein und aktuell brauche ich das auch nicht.

@olape-git
Copy link
Member Author

Ja, Ansichtssache. Mir sind das zu viele Seiten in der Navi. Da muss man ewig suchen, scrollen, Zweige öffnen. Eine Seite / je Akkordeon finde ich für meine (Kunden-)Projekte simpler. Gesplittet wird nach (konfigurierbarer) Überschrift.
Und ohne Edit / Preview - Links sind viele mit der jetzigen Lösung auch schon überfordert.

Hm, ich pflege aktuell eine Site, da ist so etwas über Accordions (JS / CSS) realisiert.
Da ist der normale Seiteninhalt und dann der Inhalt der Accordions eben alles in einem.
Das ist nicht wirklich lustig.

{{{expand('one-page = Seite X')}}} --> "Gesplittet wird nach (konfigurierbarer) Überschrift." --> und alles andere könnte bleiben wie gehabt.

@TN03
Copy link
Member

TN03 commented Apr 7, 2022

Da ist der normale Seiteninhalt und dann der Inhalt der Accordions eben alles in einem.
Das ist nicht wirklich lustig.

Das geht natürlich nicht, klar. Aber eine Seite / Akkordeon finde ich gut.

{{{expand('one-page = Seite X')}}} --> "Gesplittet wird nach (konfigurierbarer) Überschrift." --> und alles andere könnte bleiben wie gehabt.

Ist schon versuchsweise in der Mache :).
Allerdings stehen jetzt erstmal dringend andere Sachen auf dem Plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants