malmostad / sitevision_assets

Sitevision specific assets used in addition to Global Assets
http://malmostad.github.io/wag-external-v4/
GNU Affero General Public License v3.0
0 stars 1 forks source link

SV-admin studsar #276

Closed olajoh1 closed 9 years ago

olajoh1 commented 9 years ago

Mårten felsöker upptill 8 timmar

@Svempan

martent commented 9 years ago

SV klarar inte av Populär-länkarna när den kollar höjden på sidan efter att ha laddat sin verktygsrad. Obegripligt varför SV skrollar dock. :hankey:

Det är ingen idé att försöka monkey-patcha :monkey_face: en closed-source-produkt :imp:. Den billigaste :moneybag: lösningen är att dölja Populär-länkarna med CSS när SV är i admin-läge och visa dem med en fördröjning som är lite längre än SV:s fördröjning för att visa sin verktygsrad (200msec) :watch:.

Jag har lagt in ett prov i SV:s huvudhttp://www.test.malmo.se/Foretagande--jobb.html

Kan eventuellt vara snyggare att köra setInterval på typ 50msec max 40ggr och visa ul.popular om #sv-toolbar har fått klassen .sv-toolbar-loaded och då clearInterval.

@legander får tycka om lämpligheten i detta.

legander commented 9 years ago

Har kikat på det lite nu, det verkar som att de andra modulerna också spelar in så vi får nog gömma hela .wrapper istället. Jag gjorde ett test med timeout rekursion men i vissa fall så får toolbar status loaded lite snabbar än vad det tar för modulerna att renderas så man slutar på botten av sidan i alla fall. Antingen sätter vi en delay som funkar i det flesta fall eller så kikar vi efter loaded och sen lägger en lite delay.

martent commented 9 years ago

Det viktigaste här är vi inte ställer till det så att containern inte visas alls pga att det nått inte går som vi gissat att det ska i SV-admin, och vad SVAB gör i nästa patch-versison av SV.

Kör en fast delay på typ 400msec utan att kolla nått så får vi se det som att det är bättre att det studsar 1 gång av 20 istället för alla gånger. OK @Svempan?

legander commented 9 years ago

Yes, har lagt till en delay på 400ms på test nu. Jag la det i en scriptmodul så att det bara renderas ut i cms-läge. Testa och se om delayen är ok.

Svempan commented 9 years ago

Det är en klar förbättring men kan man få bort "hoppandet". Som det är nu känns det som Sitevision först visar toppen en mikrosekund, sedan sidfoten en mikrosekund, för att sedan landa rätt - överst på sidan.

martent commented 9 years ago

Det kommer att flimra i vissa fall i vissa webbläsare.

Vi blir förhoppnings av med hela problmet i SV4!

olajoh1 commented 9 years ago

Kommer vi längre? Är vi nöjda tv? @Svempan @martent

Svempan commented 9 years ago

I så fall - ja

Svempan commented 9 years ago

När passar det att vi kör ut det, Ola?

hryd commented 9 years ago

Finns nu i prod.

olajoh1 commented 9 years ago

Äntligen😃