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

changes to support yrewrite #16

Closed
wants to merge 1 commit into from

Conversation

tyrant88
Copy link
Member

@tyrant88 tyrant88 commented Apr 29, 2018

Die Ressourcen auf absoluten Pfad umgestellt und im Header Modul von dem Scroll-Link den href auf leer gesetzt.

fixes #31

@polarpixel
Copy link
Member

Da ich das gern in meiner lokalen Kopie anpassen ergänzen würde: Könntest Du mir bitte noch genauer beschreiben, was Du hier geändert hast und warum?

@tyrant88
Copy link
Member Author

tyrant88 commented Nov 5, 2018

Puh, das ist schwer nach 6 Monaten...
Also:

  1. das <a href="next-REX_SLICE_ID"> wird so ohne "#" nicht funktionieren und wenn dann auch nur auf der Startseite, da müsste dann wenigstens die URL vom aktuellen Artikel ran...
  2. Wenn man YRewrite nutzt, werden ja URLs mit Unterverszeichnissen erzeugt. Um aus denen die Javascript Resourcen zu laden braucht man dann src="/resources/...", mit nem relativen Pfad findet der Browser das nicht.
  3. Ein wohl-geformtes HTML-Dokument endet auf </body></html>
    (höchstens noch </frameset> :-)

@schuer
Copy link
Member

schuer commented Jan 5, 2019

Ich wollte gerade ein Issue zum Thema YRewrite aufmachen und habe dann gesehen, dass das Thema hier schon behandelt wurde. Zu Punkt 2, den relativen URLs, wäre aber sicherlich noch sinnvoller, statt nur auf absolute Pfade zu wechseln lieber mit <?= rex_url::base('…') ?> zu arbeiten, oder?

Generell die Frage: Kriegen wir die Conflicts noch gelöst und den PR gemerged, oder sollten @tyrant88 s Anpassungen nochmal in einem neuen PR mit dem frischen Stand der Demo nachgezogen werden?

@schuer schuer mentioned this pull request Jan 5, 2019
@polarpixel
Copy link
Member

Unbedingt frischer Stand! Denn es sind inzwischen auch noch andere Änderungen eingeflossen. Und ich wollte das auch mit Base-Pfad umsetzen statt mit absoluten Pfaden und vor allem verschiedene Szenarien testen. Letztlich müssen wir im Hauptverzeichnis und im Unterverzeichnis und jeweils mit Yrewrite sowie ohne. Wobei man notfalls auch im Unterverzeichnis ignorieren könnte, wenn sich nicht alles gut unter einen Hut bringen lässt.

@tyrant88
Copy link
Member Author

tyrant88 commented Jan 9, 2019

Klar ist Base-Pfad noch besser!
Für die Demo fände ich es nicht schlimm, wenn du einfach yrewrite mit einbaust. Oder hast du ein speziellen Grund es nicht zu nutzen?

@polarpixel
Copy link
Member

Naja, ich versuche, die Zahl der Pflicht-AddOns möglichst gering zu halten. Und Rewrite ist aus meiner Sicht (auch wenn es jeder in der Praxis früher oder später nutzen wird) halt ein "Extra".

@tyrant88
Copy link
Member Author

Okay. Ich denke die Verwendung vom base-Pfad erschlägt eh beides (Unterverzeichnis und rewrite), denn es ist ja eigentlich dasselbe.

@polarpixel
Copy link
Member

Ich schließe den PR hier mal (vielen Dank trotzdem!) und werde das im nächsten Release hoffentlich endlich mal selbst einbauen.

@polarpixel polarpixel closed this May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support für YRewrite
3 participants