search

RosenborgSupporterSoftware / RUSK

RBKweb Ultimate Survival Kit
MIT License
1 stars 2 forks source link

Bedre implementasjon av scrolling ved tastaturnavigasjon av forum, tråder og poster #82

Open havremunken opened 5 years ago

havremunken commented 5 years ago

Når man bruker bl.a. j/k for å navigere forum/tråder/poster vil den respektive extmod bruke scrollIntoView() for å sørge for at man faktisk ser det man er på i browser-vinduet.

Denne er ikke optimal. Den scroller når det strengt tatt ikke er nødvendig, og den scroller litt for nøyaktig noen ganger. F.eks. hvis man kommer til siste post kunne den med fordel ha scrollet ennå litt lenger ned så man så at man er på bunnen og ikke trenger å lure på om det er plass til en liten luring til lenger ned.

Så dette issuet blir et sted for å eksperimentere med bedre scrolle-stratergier.

larsjaas commented 5 years ago

Annen case: En tråd har plutselg i løpet av en kveld fått flere sider med nye innlegg. Mens man catcher opp, hver gang man navigerer til neste side, så flyttes man til nederste innlegget på siden, og markering/farvelegging av hva som er uleste innlegg fungerer heller ikke i den situasjonen.

Mulig RUSK burde tracke hvor langt man faktisk var kommet i tråden selv...?

havremunken commented 5 years ago

Ja, i det øyeblikket du går inn på en eller annen side i en tråd som har uleste meldinger, vil phpbb merke hele tråden som ulest. Det er vanskelig å omgå og eneste muligheten jeg ser er som du sier å tracke hvilken post man sist har sett i hver tråd, og bruke dette som en bedre indikasjon på hva som er ulest.

Da må man kanskje begrense seg til øverste post-id man har sett på en side, og ikke prøve å bestemme om brukeren har scrollet innom og sett posten eller ikke?

larsjaas commented 5 years ago

Ja, å tracke hva som har vært synlig på skjermen føler jeg er litt out-of-scope.

Om vi derimot tracker høyeste postid for alle tråder man har besøkt, så kan vi kalkulere riktig jumping-point for tråder, og post-farvelegging, og få det til å fungere selv om man ikke er logget inn på rbkweb.

larsjaas commented 5 years ago

Ikke out-of-scope for mulige forbedringer, men hva vi skal legge ned ressurser på per dags dato :)

havremunken commented 5 years ago

Har oppdatert til å bruke scrollIntoViewIfNeeded() som oppfører seg litt bedre. Ikke perfekt, men bedre.