Auswirkungen der Komponentenpositionierung mit 2.5.0.beta6

Unterstützung
1

Kein riesiges Problem, aber ich wollte die Leute einfach informieren, falls sie in ihren Themes eine benutzerdefinierte Positionierung verwenden, da sie durch das neueste Update negativ betroffen sein könnten.

Das neueste Update scheint den Code von @smrtey oder @pacharanero für die Anzeige von Discourse im Breitbildmodus beschädigt zu haben:

Die Navigationsleiste (innerhalb von Themen) befindet sich größtenteils an der korrekten (Breitbild-)Position. Beim Hochscrollen kehrt sie jedoch vorübergehend zur normalen Position zurück, bevor sie wieder in die Breitbildposition springt.

Zusätzlich hatte es unbeabsichtigt negative Auswirkungen auf das Theme „Big Header – Little Header

2

Unser Header verwendet nun position: sticky in CSS, sodass es möglich ist, dass diese Themes von dieser Änderung betroffen waren. Versatile Banner und Discourse Category Banners wurden beide aktualisiert, um Probleme zu beheben, falls Sie nach einem Ausgangspunkt suchen.

3

Danke @Alec, dass du mich darauf aufmerksam gemacht hast. Ich habe kürzlich seit 2.5.0.beta6 einige seltsame Dinge bemerkt, was den Abstand unter Überschriften betrifft, auf einigen Discourse-Instanzen, die ich betreibe, wo ich die Kopfzeile höher gemacht habe, um besser zu einem Site-Design zu passen, und etwas Padding eingefügt habe, um #main-outlet etwas nach unten zu drücken. Nach 2.5.0.beta6 war dieses zusätzliche Padding nicht mehr erforderlich.

Ich werde mir ansehen, was ich aktualisieren muss, um dies zu beheben, wenn ich Zeit habe. Ich habe 2 Instanzen, die diesen TC verwenden, und obwohl eine etwas seltsam aussieht, ist die andere in Ordnung. Und natürlich sind PRs für eine Lösung willkommen, falls jemand Zeit hat.

Danke @justin für den Hinweis zur Lösung.

4

Das hatte bei mir keine Auswirkungen auf das Breitbildformat meiner Website, da dort noch eine ganze Reihe weiterer Dinge im Gange sind, die nicht in dem von Ihnen verlinkten Bestandteil enthalten sind. Allerdings hat es meine Anpassung der Desktop-Ansicht beschädigt, die dafür sorgen sollte, dass sie auf mobilen Bildschirmen passt, damit Benutzer auf Mobilgeräten eingebettete Antworten verwenden können. Aus irgendeinem Grund blieb position: sticky bei einer Breite von < 768px nicht haften. Ich habe keine Ahnung, warum sticky nicht funktioniert hat, daher musste ich mich schließlich für folgende Lösung entscheiden:

@media (max-width: 767px) {
   .d-header-wrap {
      position: fixed;
      top: 0;
      width: 100%;
   }
   #main-outlet {
       margin-top: 48px;
   }
}
5

Hallo :wave:. Ich wollte nur kurz Bescheid geben, dass ich heute später einen Fix für diese Komponente bereitstellen werde!

Edit: Ich habe gerade ein Update für die Komponente veröffentlicht. Bitte aktualisiere sie über dein Admin-Dashboard.