Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix endless refresh loop #34

Merged

Conversation

Bajena
Copy link

@Bajena Bajena commented Dec 7, 2017

Z jakiegos powodu chrome przy scrollowaniu wielokrotnie wywoluje akcję tabs.onUpdate ze stanem complete. Ten PR zapobiega przed wielokrotnym ladowaniem faktów w takiej sytuacji.

Demko:
scrollfix

fixes #10

@KrzysztofMadejski
Copy link
Member

Dzięki za tę poprawkę! Nie widziałem tego tabs.onUpdate jako źródła.

U mnie błąd jednak cały czas występuje:

  1. Wchodzę na stronę http://www.rp.pl/Polityka/304069841-Andrzej-Duda-Nie-ma-dogmatu-o-nieomylnosci-prezydenta.html
  2. Skroluję Page Down do dołu
  3. Gdy strona zaczyna dociągać nowy kontent to pojawia się wiele "F"-ek w banele na dole.

Debugging mówi, że wiąże się to z wielokrotnym odebraniem zdarzenia content-loaded co wydaje się naturalne. Problem jest z tym, że content już raz zaznaczony jest zaznaczany kolejny raz skutkując wielokrotnie zagnieżdżoną strukturą (patrz screenshot).
screenshot from 2017-12-08 11-49-25

Przejrzałem dokumentację zewnętrznej biblioteki: https://github.com/julmot/mark.js i wykorzystanie opcji exclude pozwalana na rozwiązanie tego. Przetestowałem to u siebie i pomogło.

Dodaj to proszę w swoim forku i sprawdź, czy działa też u Ciebie.
https://github.com/kodujdlapolski/factchecker-plugin-chrome/blob/demagog/src/js/factual_client.js#L165

    this.marker.mark(fact.text, {
      className: `${factClass} factchecker-fact-mark factchecker-fact-mark-${fact.rating}`,
      exclude: ['.factchecker-fact-mark'], 
      acrossElements: true,

Copy link
Member

@KrzysztofMadejski KrzysztofMadejski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dodaj proszę opcję exclude w wywołaniu markera. Pomaga to na case, który u mnie cały czas występuje.
https://github.com/kodujdlapolski/factchecker-plugin-chrome/blob/demagog/src/js/factual_client.js#L165

    this.marker.mark(fact.text, {
      className: `${factClass} factchecker-fact-mark factchecker-fact-mark-${fact.rating}`,
      exclude: ['.factchecker-fact-mark'], 
      acrossElements: true,

@KrzysztofMadejski
Copy link
Member

BTW: Cały czas widzę, że dublują się banery na dole. Związane jest to z tym, że wielokrotnie jest wykrywany brak niedopasowania faktów. To zostanie jednak rozwiązane w innym issue: #15

@KrzysztofMadejski KrzysztofMadejski merged commit dd28829 into kodujdlapolski:demagog Dec 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Przy scrollowaniu w dół baner dolny wariuje
2 participants