RegionHalland / general

0 stars 0 forks source link

Möjligt att ha samma rubrik mer än en gång i Hitta på sidan #460

Closed JohannaOlin closed 5 years ago

JohannaOlin commented 5 years ago

Beskrivning

I Terapirekommendationerna har man ibland använt samma rubrik mer än en gång på en sida. Detta har ställt till det i Hitta på sidan som då inte visar rubriken på bägge ställen. Detta är inget problem på vårdgivarwebben idag, men vi måste verifiera att det fungerar när vi lägger in komponenten på Terapirekommendationerna.

Acceptanskriterier

viktor-sarge commented 5 years ago

Har också tänkt på detta. Problemet är att det idag skapas ID som utgår från att rubrikerna är unika ID = "detta-ar-min-rubrik" ungefär. Men det är nog ganska snabbt gjort att lägga till ett uniqueid

RolandHyden commented 5 years ago

Hej på er. Jag har fixat detta genom att lägga på ett löpnummer efter slugen, dvs det ser ut enligt:

mitt-exempel-1 mitt-exempel-2 ett-annat-exempel-3 osv.

Tack var löpnumret så blir alla slugar unika oavsett om det redan finns eller inte

Version 1.2.0 https://github.com/RegionHalland/region-halland-prepare-the-content

Version 2.0.0 https://github.com/RegionHalland/region-halland-find-on-page

viktor-sarge commented 5 years ago

@RolandHyden Gött! Kör någon av sajterna den nya koden? Om vi har det på typ test eller stage kan vi ju snabbt verifiera att det funkar och släcka den. (Har svårt att tänka mig att löpnummer inte skulle lösa problemet, men får väl ändå dubbelkolla)

RolandHyden commented 5 years ago

Tjena. Jod, vi kör koden på regionhalland.se och det funkar fint :)

viktor-sarge commented 5 years ago

+1 på det :) Bryr vi oss om att flytta de gamla här på zenhub till closed @JohannaOlin ?

JohannaOlin commented 5 years ago

@regionhallandviktor Ja, vi stänger dem, så ser vi sen när allt är tomt :)