Project

General

Profile

Bug #4562

Performance på sektioner er meget langsom

Added by Stefan Søndervang 8 months ago. Updated 17 days ago.

Status:
Reviewed - Needs info/rework
Priority:
Urgent
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Sektioner og temaer

Description

Fordelen ved sektioner er at vi kan opbygge miniuniverser på vores sider med forskelligt indhold samlet med tags, kategorier etc. Problemet er at der ikke skal meget indhold til før at sektionerne tager ekstremt lang tid om at loade. Det er et problem for redaktører, der skal sidde og vente for at redigere, og brugerne mister tålmodigheden.

Jeg har testet loadtiden på tre af vores sektioner efter en cachetømning:

Siderne er hurtigere cachet, men er stadig langsomme.

Skærmbillede 2020-05-04 kl. 18.26.27 (1).png (396 KB) Skærmbillede 2020-05-04 kl. 18.26.27 (1).png Gitte Barlach, 05/05/2020 08:42 AM
Skærmbillede 2020-05-04 kl. 18.20.46 (1).png (627 KB) Skærmbillede 2020-05-04 kl. 18.20.46 (1).png Gitte Barlach, 05/05/2020 08:42 AM
Skærmbillede 2020-05-04 kl. 18.37.49 (1).png (633 KB) Skærmbillede 2020-05-04 kl. 18.37.49 (1).png Gitte Barlach, 05/05/2020 08:42 AM
Skærmbillede 2020-05-05 kl. 12.44.25.png (530 KB) Skærmbillede 2020-05-05 kl. 12.44.25.png nyhed med vedhæftede materialer Gitte Barlach, 05/05/2020 12:47 PM
Skærmbillede 2020-05-05 kl. 12.41.58.png (740 KB) Skærmbillede 2020-05-05 kl. 12.41.58.png nyhed med vedhæftede materialer Gitte Barlach, 05/05/2020 12:47 PM

Related issues

Related to DDB CMS - Bug #4175: Oprettelse af karruseller er til tider ekstremt langsomResolved (tag version)
Related to DDB CMS - Bug #4640: Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4562; gen-testes når denne er løst)Technical test
Has duplicate DDB CMS - Bug #4554: Sektionsside loader langsomtClosed
Has duplicate DDB CMS - Bug #4786: Post kan ikke bruges i forsidekarrusel INGEN KODEÆNDRING - TESTES EFTER #4562Resolved

History

#1 Updated by Rolf Madsen 8 months ago

  • Status changed from New to Needs analysis
  • Assignee set to Christel Krabbenhøft
  • Priority changed from Normal to Urgent
  • Target version set to Release 31-2 - Bug fixes (7.x-5.?.?)

Det er et problem, som vi skal have løst.

#2 Updated by Rolf Madsen 8 months ago

Jesper har tidligere meldt ind om samme oplevelse, derfor har jeg sat ham på som wathcer på dette issue.

#3 Updated by Stefan Søndervang 8 months ago

Jeg tænker problemet stadig er forsidekarrusellerne. Men måske godt at teste det fra bunden.

#4 Updated by Rolf Madsen 8 months ago

Karrusellerne burde blive hentet asynkront via lazy-load.

Det skal undersøges om der er noget der er gået i stykker.

#5 Updated by Stefan Søndervang 7 months ago

  • Related to Bug #4175: Oprettelse af karruseller er til tider ekstremt langsom added

#6 Updated by Stefan Søndervang 6 months ago

  • Has duplicate Bug #4554: Sektionsside loader langsomt added

#7 Updated by Stefan Søndervang 6 months ago

  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs

#8 Updated by Thomas Hansen 5 months ago

Har på fornemmelsen at det er mere end en ting.

Men den første synder jeg fandt er at karusellen checker availability for materialer. Det bliver ikke cachet (for det giver ikke mening), men det koster.

En hurtig test:

Kolonne A er med availability, B er uden, page load på https://rudbib.dk/section/videnskab  i sekunder. Kolonne C er forbedringen fra A til B.

Det er kun availability for det første "sæt" af forsider i hver karousel, altså de første, resten bliver loaded via ajax.

Så den nemmeste løsning er ikke at "pre-loade" nogen forsider fra serveren, men lade dem alle blive loaded via ajax, det får talene til at se endnu bedre ud:

Det vil betyde at karusellerne bliver populeret efter at siden er loaded, men det er vist det bedste vi kan gøre.

 

#9 Updated by Rolf Madsen 5 months ago

@Thomas, tak for gennemgangen!

Det lyder som en fornuftig løsning.

Make it so! :-)

#11 Updated by Rolf Madsen 5 months ago

Så vidt jeg kan se findes billedet ikke, så måske den falder tilbage på et gammelt cached billede?

            <ns1:identifierInformation>
                <ns1:identifierKnown>false</ns1:identifierKnown>
                <ns1:identifier>
                    <ns1:pid>870970-basis:52743494</ns1:pid>
                </ns1:identifier>
            </ns1:identifierInformation>

#12 Updated by Rolf Madsen 5 months ago

Forklaringen er at vi får nye billeder fra Bogportalen, og her findes det pågældende billede ikke.

#13 Updated by Thomas Hansen 5 months ago

Aha...

 

https://github.com/ding2/ding2/pull/1581

Tog lidt længere tid, fordi jeg lige skulle finde en bug i ddbasic der fik mine karouseller til at dø halvvejs nede af siden. Viste sig at `add_ellipsis()` ikke var multibyte kompatibel, og kunne finde på at klippe multibyte tegn midt over, hvilket fik json_encode til at give op.

 

#14 Updated by Thomas Hansen 5 months ago

  • Status changed from Needs analysis to Needs code review

#15 Updated by Gitte Barlach 3 months ago

  • Assignee changed from Kasper Garnæs to Jesper Kristensen

#16 Updated by Jesper Kristensen 3 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

Reviewed og godkendt

#17 Updated by Kasper Garnæs about 1 month ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#18 Updated by Kasper Garnæs about 1 month ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#19 Updated by Kasper Garnæs about 1 month ago

  • Status changed from Reviewed to Technical test

#20 Updated by Rolf Madsen about 1 month ago

  • Has duplicate Bug #4786: Post kan ikke bruges i forsidekarrusel INGEN KODEÆNDRING - TESTES EFTER #4562 added

#21 Updated by Gitte Barlach 26 days ago

Testet på upgrade-fbs med 7.x-6.1.0-beta2

Ift. performance så oplever jeg her i første testrunde ikke nogen problemer med at indsættelse af karruseller på hhv forside og sektioner er langsom. 

Ift. visningen er vi ikke helt i mål; jeg har overført mine kommentarer fra #4786

Okay nu kan posten "Dengang dinosaurerne var små"  i al fald vises :-)

Men der er noget der skal rettes:

a) med gradient slået TIL:
a1) indholdsnoten mangler, eller den blive vist, men forkert placeret 
a2) pile-knapperne er også placerede lidt sjovt
Se vedlagte skærmbilleder


b) med gradient slået FRA:
b.1) over og under forsidebilledet er der en grå baggrund. Den skal ikke være der
b.2) noten bliver ikke  vist på den rigtige måde
b.3) overlay, der indeholder knapperne "Læs mere" og "Reservér" passer ikke rigtigt i størrelsen 
Se vedlagte skærmbillede

#22 Updated by Gitte Barlach 26 days ago

Okay problemer med visning af materialer gælder ikke kun i karruseller men også når man vedhæfter materialer til en nyhed...
så nu ved jeg ikke om det er dette eller et andet issue, der har medført situationen. 

#23 Updated by Christel Krabbenhøft 24 days ago

  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs

#24 Updated by Tue Gaston 24 days ago

Ligeledes ved brug af paragraphs af typen "Enkelt materiale, ½ spalte" ser det helt forfærdligt ud ved mouse-over:

#25 Updated by Benjamin Rasmussen 17 days ago

Bemærk at jeg muligvis har fikset et par dele af dette i 4640.

Mere info her: https://platform.dandigbib.org/issues/4640#note-11 / https://github.com/ding2/ding2/pull/1624/

#26 Updated by Tue Gaston 4 days ago

  • Related to Bug #4640: Forsidekarrusel. Uheldig visning af visse materialer når overlay er slået fra (overlap til #4562; gen-testes når denne er løst) added

Also available in: Atom PDF