p0weruser icon indicating copy to clipboard operation
p0weruser copied to clipboard

Verlinkung zu Kommentaren fehlerhaft

Open rottenpuppy opened this issue 3 years ago • 20 comments

Allgemeine Informationen

Browser: Firefox 102.0.1 64Bit

Version: 2.1.0

Modul: vermutlich WIP Chat

Fehlerbeschreibung

Die Position des verlinkten Kommentars

Aktuelles Verhalten

Wenn ich aus dem Nachrichten-Popdown zu einem Kommentar springe, wird der korrekte Post angezeigt, allerdings rutscht der Sprung zum rivhtigen Kommentar nach oben des Bildschirms (einmal runterscrollen macht den kommentar sichtbar) oder es wird der oberste Kommentar des jeweiligen Posts angezeigt. NICHT DRINGLICH!

Erwartetes Verhalten

Richtig wäre (wie es vor dem Update der Fall war), dass beim Klick aus den Nachrichten auf einen Kommentar sofort der aktuelle Kommentar angezeigt wird.

Reproduzierbarkeit

Der Fehler kann jedes Mal reproduziert werden. Die Verlinkung springt entweder wie eingangs beschrieben kurz über den relevanten Kommentar, oder gänzlich zum ersten Kommentar des Posts. Ein Löschen des Caches hat keine Besserung ergeben.

PS: Im Punkt Reproduzierbarkeit hast du einen Schreibfehler drin, nämlich "repsoruziert". Einfach nur so, falls du Lust hast das zu ändern ;)

rottenpuppy avatar Jul 19 '22 18:07 rottenpuppy

Danke. Auch für den Hinweis mir dem Typo. Mach ich irgendwann:tm:

Eine Rückfrage habe ich dazu:

Hast du den Widescreen Mode aktiv?

Notiz für mich: Realtiert zu #93

twobiers avatar Jul 19 '22 20:07 twobiers

Gerne :)

Ja habe ich. Wie bereits geschrieben funktionierte es vor dem Update einwandfrei, erst nach dem Update fiel mir das auf. Hab den Widescreenmode mal testweise deaktiviert, Fehler bleibt bestehen. Sollte ich mehr über das Fehlerbild bzw. das schuldhafte Modul herausfinden gebe ich dir ein Update.

Am Di., 19. Juli 2022 um 22:05 Uhr schrieb Tobi @.***>:

Danke. Auch für den Hinweis mir dem Typo. Mach ich irgendwann™️

Eine Rückfrage habe ich dazu:

Hast du den Widescreen Mode aktiv?

— Reply to this email directly, view it on GitHub https://github.com/Scarwolf/p0weruser/issues/94#issuecomment-1189500941, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMSG333CPXSFRUZ5OMKCA43VU4C7RANCNFSM54A6DWXQ . You are receiving this because you authored the thread.Message ID: @.***>

rottenpuppy avatar Jul 19 '22 20:07 rottenpuppy

In Firefox ist das Fokusverhalten tatsächlich nicht ganz sauber. Ich kann das insofern reproduzieren, dass ein Kommentar leicht außerhalb des Sichtfelds scrollt (Rot markiert ist der fokussierte Kommentar). Ist das das was du meinst?

image

twobiers avatar Jul 20 '22 07:07 twobiers

Genau, allerdigs tendiert es teilweise dazu, gar keinen Kommentar zu fokussieren sondern schlicht den allerersten anzeigt. Dann muss man den Kommentar erst mühsam suchen. Meinst du, das ist ein FF Problem?

Am 20.07.2022 um 09:32 schrieb Tobi @.***>:

 In Firefox ist das Fokusverhalten tatsächlich nicht ganz sauber. Ich kann das insofern reproduzieren, dass ein Kommentar leicht außerhalb des Sichtfelds scrollt (Rot markiert ist der fokussierte Kommentar). Ist das das was du meinst?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

rottenpuppy avatar Jul 20 '22 07:07 rottenpuppy

Also bei mir im Edge (Chromium) funktioniert es einwandfrei. Ich schau mir das mal genauer an, sollte schon auch in FF funktionieren.

twobiers avatar Jul 20 '22 07:07 twobiers

Ist scheinbar ein Darstellungsproblem des scrollbaren Bereichs in Firefox. Der Kommentar wird aus Firefox sicht korrekt reingescrollt, ist aber für den Nutzer nicht sichtbar, weil der unter der Navbar liegt. Sollte nicht so schwierig zu fixen sein, ich schau wenn ich Luft hab. Ich geb hier nochmal Rückmeldung, wenn ich weiß wann mit einem entsprechenden Bufxi Release zu rechnen ist.

twobiers avatar Jul 20 '22 07:07 twobiers

Habe einen Offset fürs scrollen verwendet, jetzt wird bei mir sowohl in Firefox als auch Edge der Kommentar richtig reingescrollt. Vielleicht fällt mir da noch was besseres ein. Den Fehler, dass kein Kommentar fokussiert wird, kann ich gerade nicht reproduzieren.

Wenn du möchtest, kannst du hier die Dev-Version testen und schauen ob das für dich passt. Würde spätestens Sonntag dann releasen. https://github.com/Scarwolf/p0weruser#installation

twobiers avatar Jul 20 '22 10:07 twobiers

Im Widescreen Mode ist der Fehler immer noch reproduzierbar, im Anhang hast du zwei Bildschirmaufnahmen, was genau ich meine. Wenn ich auf den verlinkten Kommentar klicke springt er manchmal zum Kommentar, manchmal lädt er aber den Post ganz normal, als ob man via Pfeiltasten ein Medium vor/zurück springt.

Nach dem dritten Versuch dir Dateien zu schicken lasse ich es, kleiner kann ich die Videos nicht mehr machen.

Am Mi., 20. Juli 2022 um 12:50 Uhr schrieb Tobi @.***>:

Habe einen Offset fürs scrollen verwendet, jetzt wird bei mir sowohl in Firefox als auch Edge der Kommentar richtig reingescrollt. Vielleicht fällt mir da noch was besseres ein. Den Fehler, dass kein Kommentar fokussiert wird, kann ich gerade nicht reproduzieren.

Wenn du möchtest, kannst du hier die Dev-Version testen und schauen ob das für dich passt. Würde spätestens Sonntag dann releasen. https://github.com/Scarwolf/p0weruser#installation

— Reply to this email directly, view it on GitHub https://github.com/Scarwolf/p0weruser/issues/94#issuecomment-1190124702, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMSG334323H3OEXYLNUQXPTVU7KXRANCNFSM54A6DWXQ . You are receiving this because you authored the thread.Message ID: @.***>

rottenpuppy avatar Jul 20 '22 11:07 rottenpuppy

@rottenpuppy Ich kann es leider immer noch nicht reproduzieren, hättest du nochmal die Güte die neue Dev-Version zu installieren und zu testen? Vergiss nicht dafür in Tampermonkey/Greasemonkey die v2.1.0 zu deaktivieren und nur v2.1.0-dev aktiv zu halten. Scarwolf/p0weruser#installation

twobiers avatar Jul 21 '22 12:07 twobiers

Hey, die habe ich seit gestern Abend installiert und logischerweise das normale deaktiviert. Fehler besteht auch in der dev-Version, ich schicke zwei Videos an regshit@ einmal mit und ohne andere Addons.

Am 21.07.2022 um 14:41 schrieb Tobi @.***>:

 @rottenpuppy Ich kann es leider immer noch nicht reproduzieren, hättest du nochmal die Güte die Dev-Version zu testen? Vergiss nicht dafür in Tampermonkey/Greasemonkey die v2.1.0 zu deaktivieren und nur v2.1.0-dev aktiv zu halten. Scarwolf/p0weruser#installation

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

rottenpuppy avatar Jul 21 '22 13:07 rottenpuppy

Ich habe die Version eben aktualisiert, du musst die manuell neu installieren. Das passiert leider nicht automatisch.

twobiers avatar Jul 21 '22 13:07 twobiers

Gibt keine Updates, habs soeben überprüft.

Am Do., 21. Juli 2022 um 15:33 Uhr schrieb Tobi @.***>:

Ich habe die Version eben aktualisiert, du musst die manuell neu installieren. Das passiert leider nicht automatisch.

— Reply to this email directly, view it on GitHub https://github.com/Scarwolf/p0weruser/issues/94#issuecomment-1191491549, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMSG337TVG6YXPMHEZK2Y4LVVFGRFANCNFSM54A6DWXQ . You are receiving this because you were mentioned.Message ID: @.***>

rottenpuppy avatar Jul 21 '22 13:07 rottenpuppy

Du musst manuell diesen Link aufrufen: https://scarwolf.github.io/p0weruser/p0weruser.dev.user.js Ansonsten rafft der das leider nicht, ändere ich nachher.

twobiers avatar Jul 21 '22 13:07 twobiers

Habt ihr discord, Telegram oder sonstwas? Die Scherereien mit dem Mail-Client gehen mir dezent auf den Piss

Am Do., 21. Juli 2022 um 15:33 Uhr schrieb Tobi @.***>:

Ich habe die Version eben aktualisiert, du musst die manuell neu installieren. Das passiert leider nicht automatisch.

— Reply to this email directly, view it on GitHub https://github.com/Scarwolf/p0weruser/issues/94#issuecomment-1191491549, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMSG337TVG6YXPMHEZK2Y4LVVFGRFANCNFSM54A6DWXQ . You are receiving this because you were mentioned.Message ID: @.***>

rottenpuppy avatar Jul 21 '22 13:07 rottenpuppy

Mit dem nächsten Release wird die Scrollbar getauscht, damit sollte der Fehler hoffentlich Geschichte sein. Kannst du dann nochmal Rückmeldung geben?

twobiers avatar Aug 07 '22 19:08 twobiers

Na klar, schreibt bitte nochmal falls ich es vergesse.

Am 07.08.2022 um 21:31 schrieb Tobi @.***>:

 Mit dem nächsten Release wird die Scrollbar getauscht, damit sollte der Fehler hoffentlich Geschichte sein. Kannst du dann nochmal Rückmeldung geben?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

rottenpuppy avatar Aug 08 '22 07:08 rottenpuppy

Problem besteht leider immer noch. Wenn ich aus der Benachrichtigung, wenn sie denn richtig lädt und ich nicht deswegen schon F5 drücken muss, etwas anklicke, wird immer noch der gecachte Beitrag geladen und somit in den meisten Fällen nicht der aktuelle Content inkl. des Kommentars und selbst wenn ich dann F5 drücke springt man nicht zum verlinkten Kommentar, sondern bleibt ganz oben.

AkumaNoTsubasa avatar Aug 23 '22 17:08 AkumaNoTsubasa

selbst wenn ich dann F5 drücke springt man nicht zum verlinkten Kommentar, sondern bleibt ganz oben.

Wenn ich dich richtig verstehe, sprechen wir hier über ein anderes Problem. Wenn ein Kommentar über den Direktlink aufgerufen wird (bspw. /new/1234567:comment1234567 direkt über den Browser), dann funktioniert der Fokus nicht. Das Problem ist in #93 getrackt. Problem daran ist, dass das "normale" pr0gramm Frontend den Link entfernt und wenn das p0wersuser Frontend geladen wurde, nicht mehr gelesen werden kann. Könnte was tricky werden das einzubauen. Das heißt der Fokus eines Kommentars funktioniert aktuell nur, wenn p0weruser einmal geladen wurde und entsprechende Links dann über den internen Router aufgerufen werden.

twobiers avatar Aug 23 '22 17:08 twobiers

Moin, hab aktuell keinen Zugang zum Rechner/Laptop, kann also nichts testen. Versuche daran zu denken wenn ich wieder heeme bin. Teilweise funktioniert es, oft aber eben nicht.

Am 23.08.2022 um 19:25 schrieb AkumaNoTsubasa @.***>:

 Problem besteht leider immer noch. Wenn ich aus der Benachrichtigung, wenn sie denn richtig lädt und ich nicht deswegen schon F5 drücken muss, etwas anklicke, wird immer noch der gecachte Beitrag geladen und somit in den meisten Fällen nicht der aktuelle Content inkl. des Kommentars und selbst wenn ich dann F5 drücke springt man nicht zum verlinkten Kommentar, sondern bleibt ganz oben.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

rottenpuppy avatar Aug 23 '22 17:08 rottenpuppy

Das verstehe ich nicht, habe momentan aber auch keine Möglichkeit das zu testen.

Am 23.08.2022 um 19:34 schrieb Tobi @.***>:

 selbst wenn ich dann F5 drücke springt man nicht zum verlinkten Kommentar, sondern bleibt ganz oben.

Wenn ich dich richtig verstehe, sprechen wir hier über ein anderes Problem. Wenn ein Kommentar über den Direktlink aufgerufen wird (bspw. /new/1234567:comment1234567 direkt über den Browser), dann funktioniert der Fokus nicht. Das Problem ist in #93 getrackt. Problem daran ist, dass das "normale" pr0gramm Frontend den Link entfernt und wenn das p0wersuser Frontend geladen wurde, nicht mehr gelesen werden kann. Könnte was tricky werden das einzubauen. Das heißt der Fokus eines Kommentars funktioniert aktuell nur, wenn p0weruser einmal geladen wurde und entsprechende Links dann über den internen Router aufgerufen werden.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

rottenpuppy avatar Aug 23 '22 17:08 rottenpuppy