Project

General

Profile

Bug #4196

Bogmærker / interne links

Added by Jørgen Pedersen 3 months ago. Updated 8 days ago.

Status:
Reviewed
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
https://silkeborgbib.dk/tilbud-til-dig/mobile-tilbud-0/biblioteket-paa-din-mobil
Kategorier:
Biblioteksinfo - Sider

Description

Funktionen bogmærker i side/nyhedseditoren virker ikke optimalt.  Når man linker til indhold på samme side, så burde den linje, hvor bogmærket optræder, være synlig øverst på siden. Men pt. bliver bogmærket dækket af topmenuen.  I forhold til tilgængelighedskravene tænker jeg ikke at bogmærket må være placeret adskillige linjer over det sted det egentlig burde være placeret, fx. i forrige afsnit.

bogmærker_backend.jpg (163 KB) bogmærker_backend.jpg Jørgen Pedersen, 03/04/2019 10:41 AM
Bogmærker_frontend.jpg (199 KB) Bogmærker_frontend.jpg Jørgen Pedersen, 03/04/2019 10:41 AM

History

#1 Updated by Rolf Madsen 3 months ago

  • Status changed from New to Need more info
  • Assignee set to Jørgen Pedersen
  • Target version set to DDB CMS - Analyse og prioritering udestår

Kan du sende mig et link til den side du oplever fejlen på?

Jeg kan ikke rigtig følge "burde den linje, hvor bogmærket optræder, være synlig øverst på siden".

Så vidt jeg kan se skal du selv lave et link der refererer bogmærket, og altså ikke noget der sker af sig selv.

#2 Updated by Jørgen Pedersen 3 months ago

Rolf Madsen skrev:

Kan du sende mig et link til den side du oplever fejlen på?

Jeg kan ikke rigtig følge "burde den linje, hvor bogmærket optræder, være synlig øverst på siden".

Så vidt jeg kan se skal du selv lave et link der refererer bogmærket, og altså ikke noget der sker af sig selv.

Det er denne side (samme link som i beskrivelsen) https://silkeborgbib.dk/tilbud-til-dig/mobile-tilbud-0/biblioteket-paa-din-mobil

Her er link til et af bogmærkerne "lånerkort" https://silkeborgbib.dk/tilbud-til-dig/mobile-tilbud-0/biblioteket-paa-din-mobil#L%C3%A5nerkort Som du kan se dækker den orange bjælke for den linje, hvor bogmærket egentlig er placeret. Det betyder, at hvis overskriften til afsnittet om lånerkoretet skulle være synligt når man bruger bogmærket, så skulle jeg i editoren placere bogmærket adskillige linjer længere oppe. Det kan betyde at du sætter bogmærket i et helt andet afnit end jeg egentlig ønsker at linke til. Og det kan vel give problemer for tilgælgeligheden.

#3 Updated by Simon Holt 3 months ago

Jeg render også tit ind i det her problem og det er jo pga vores fixed header. Browseren kan jo ikke vide hvor meget den skal offset. Den scroller bare ned til anchoren.

Det er et velkendt problem: https://stackoverflow.com/questions/4086107/fixed-page-header-overlaps-in-page-anchors. 

Jeg synes faktisk denne her løsning ser lovende ud: https://stackoverflow.com/a/28824157. Ser ud til at man med den CSS-regel kan fortælle browseren hvor meget den skal offset. Det der nok bliver sværest er at bestemme headeren højde (søgning udfoldet, under-menuer osv.)

#4 Updated by Rolf Madsen 3 months ago

OK, nu tror jeg at jeg har fattet det.

Når jeg linker til et bookmark der er oprettet på "Lånerkort" overskriften og trykker på det så scroller siden så langt ned at den pågældende overskrift er skjult.

Der linkes fra "Bookmark" til "Lånerkort" (Se røde rammer)

Siden scroller så langt ned at overskriften "Lånerkort" er skjult.

#5 Updated by Rolf Madsen 3 months ago

  • Status changed from Need more info to Ready for development
  • Assignee changed from Jørgen Pedersen to Simon Holt

@Simon, vil du tage den på listen og grave videre i om din løsning er gangbar ved lejlighed.

#6 Updated by Simon Holt 3 months ago

@Rolf jeg tager et kig på det. Kan muligvis bruge noget at der arbejde Benjamin lavede, til at bestemme højden af headeren.

#7 Updated by Simon Holt about 2 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

PR: https://github.com/ding2/ding2/pull/1397

Benjamins arbejde med at definere de forskellige header højder og kombinationsmuligheder har været en stor hjælp her. Anchor links skulle nu gerne offsettes korrekt og der skulle gerne tages højde for alt: extended search, admin bar, admin bar genveje osv.

Se PR for før/efter screenshots.

#8 Updated by Rolf Madsen 10 days ago

  • Target version changed from DDB CMS - Analyse og prioritering udestår to Release 31 - bugfixes

#9 Updated by Gitte Barlach 10 days ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#10 Updated by Jørgen Nielsen 8 days ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

Also available in: Atom PDF