Diskussion zu "phpBB 3.3.1 erschienen - bitte updaten"

[Kirk]

Und wie das jetzt neu anders ist, wird leider nicht erklärt

Bei einem Childstyle hast man in der overall_header.html u. a. jenes eingetragen:

Code: Alles auswählen

<link href="{ROOT_PATH}styles/prosilver/theme/normalize.css?assets_version={T_ASSETS_VERSION}" rel="stylesheet">

Wenn man sich im Browser den Quelltext anzeigen lässt, sieht man folgendes:

Code: Alles auswählen

<link href="./styles/prosilver/theme/normalize.css?assets_version=12712" rel="stylesheet">

Leert man den Foren und Browser Cache, sieht man das hier:

Code: Alles auswählen

<link href="./styles/prosilver/theme/normalize.css?assets_version=12713" rel="stylesheet">

Der Wert hat sich also um 1 erhöht, das bedeutet das der Browser die neue bzw. überarbeitete Datei übernommen hat.
In der stylesheet.css von prosilver werden die css Dateien per import sozusagen an den Browser übermittelt, damit gab es im FF bei manchen Schwierigkeiten. Deshalb wurde das hash eingeführt, damit der Browser immer die neue css Datei nimmt.
Bei einem Childstyle hat man in der stylesheet.css ja die Änderungen direkt eingegeben, von daher muss da nicht mit hash arbeiten.

[Scanialady]

Danke für die Erläuterung. Nun bin ich schlauer.

[Wolkenbruch]

Update von 3.3.0 nach 3.3.1 lief problemlos durch.
Die bei mir geänderten Hash Werte konnte ich einfach mit dem => CSS Hash Tool von @wintstar anpassen, obwohl dieses Tool nicht in einem Live Forum verwendet werden soll!

[Joyce&Luna]

Ich muss mal hier was korrigieren.

Konnte nicht verstehen, warum der Hash Code meinen Style zerschießt.
Vor Wochen habe ich eine Extension installiert um diese für ein anderes Forum anzupassen.
Diese Extension hatte ich nicht wieder deaktiviert.
Das war deine Extension @Kirk Profil on Leftside. Es kommt zu Verschiebungen auf der Startseite und im Lese Bereich.
Solltest du mal bitte Prüfen.
Dieses hier kommt dem ganzen in der Quere bei deiner Ext

Code: Alles auswählen

+@media (min-width: 701px) {
	.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;
 	}

Das ist mir aber erst gestern aufgefallen. Als ich diese deaktiviert habe, war alles wieder normal.
Also liegt es nicht am Hash aber trotzdem ist es schon sehr stressig und Zeitaufwendig den Hash zu nutzen.
Alle Extensionen müssen nun auch geprüft werden.
Nur wegen diesem blöden Hash wird alles durch einander geworfen.

[Tastenplayer]

Nicht der Hash sonder dieser neue Code in der responsive.css bringt vieles durcheinander
Meine Styles stimmten hinten und vorne nicht mehr. Ich konnte anpassen was ich wollte. Bis ich endlich herausfand, dass ich in fast allen meinen Styles das margin-right: 16px entfernen muss. Warum auch immer - aber so sehen meine Styles wieder aus, wie sie sollen.
In einigen komplizierten Styles habe ich die alte responsive Anpassung gelassen. Sonst hätte ich die ganzen Styles ändern müssen.

Logisch kann diese responsive.css Änderung auch Auswirkungen auf einige Extensions haben.

Man kommt sich in 3.3.1 schon fast vor, wie in einer völlig neuen phpBB Reihe und nicht wie in einem Update von 3.3.0 zu 3.3.1!

Edit: Offizielle Stellungsnahme von DavidIQ

by DavidIQ » Tue Aug 11, 2020 8:44 pm
Official response: we are not requiring that style authors add a hash or any other cache-busting parameter to the imports in their stylesheet. Of course if you update something in one of those imported stylesheets it would be of benefit to add a hash/number/whatever to the import so that frequent visitors see the changes right away but, again, leaving it up to the style author.

[Crizzo]

Es wurden gerade die Deutschen Pakete online gestellt:

• Deutsche Komplettpakete: https://www.phpbb.de/downloads/pakete/
• Deutsche Sprachpakete: https://www.phpbb.de/downloads/sprachpakete/

[chris1278]

Ich muss mal hier was korrigieren.

Konnte nicht verstehen, warum der Hash Code meinen Style zerschießt.
Vor Wochen habe ich eine Extension installiert um diese für ein anderes Forum anzupassen.
Diese Extension hatte ich nicht wieder deaktiviert.
Das war deine Extension @Kirk Profil on Leftside. Es kommt zu Verschiebungen auf der Startseite und im Lese Bereich.
Solltest du mal bitte Prüfen.
Dieses hier kommt dem ganzen in der Quere bei deiner Ext

Code: Alles auswählen

+@media (min-width: 701px) {
	.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;
 	}

Das ist mir aber erst gestern aufgefallen. Als ich diese deaktiviert habe, war alles wieder normal.
Also liegt es nicht am Hash aber trotzdem ist es schon sehr stressig und Zeitaufwendig den Hash zu nutzen.
Alle Extensionen müssen nun auch geprüft werden.
Nur wegen diesem blöden Hash wird alles durch einander geworfen.

Ich gehe mal davon aus das es sich bei der Codestelle um diese Änderung in der Responsiv.css handelt:

[ externes Bild ]


Das Problem ist mir aufgefallen als ich einen Style manuell anpassen wollte. Dort fehlt eine Schließende }

So sieht der Code an der stelle in Version 3.3.0 aus:

Code: Alles auswählen

@media (min-width: 700px) {
	.postbody { width: 70%; }
}

Wenn ich jetzt die Änderung einfüge laut den codechanges sähe der code Teil so aus:

Code: Alles auswählen

@media (min-width: 701px) {
	.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;
}

Man beachte hierbei das dann zum Schluss eine Schließende Klammer fehlt. Das solltet ihr mal überprüfen.

[Kirk]

Dort fehlt eine Schließende }

Sieh dir mal die Code Changes genauer an, da fehlt nichts.

@Joyce&Luna
Das mit der EXT ist bekannt, es wird bereits an ein Update gearbeitet.

[chris1278]

Aber sicher fehlt da die schliesende klammer.

Den teil hier:

Code: Alles auswählen

@media (min-width: 700px) {
	.postbody { width: 70%; }

Soll ich durch diesen ersetzen:

Code: Alles auswählen

@media (min-width: 701px) {
	.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;

in dem teil vorher sind zwei { und zwei } Klammern da wo sie sein sollen

Diese müsste auch in dem Teil sein der ersetzt werden mus. Dort fehlt aber die Schliessende klammer:

Code: Alles auswählen

.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;

Wenn das der code Wäre würde mit der danach stehenden Klammer der Code Geschlossen aber durch das einfügen davor von

Code: Alles auswählen

@media (min-width: 701px) {

Hätte man nach oder Vor der einen Schliessenden Klammer eine Weitere setzen müssen. Selbst wenn man das mit dem Prosilver style vom original 3.3.0 vergleicht kommt das selbe bei raus.

Es fehlt eine } Klammer um die media anweisung zu schliessen

[Dr.Death]

Nein, das tut es nicht....

befolge die weitergehenden Änderungen in Zeile 140-149 dann passt es wieder.... wie Kirk bereits sagte.



Ergebnis:

Code: Alles auswählen

@media (min-width: 701px) {
	.postbody {
		/** deduct postprofile width (including border width, margin and padding) and postbody margin and padding */
		width: calc(100% - 200px - 1px - 16px);
		margin-right: 16px;
	}

	.postprofile {
		width: 200px;
	}
}