search

insper-education / active-handout-plugins-js

DEPRECATED: javascript repo for active handout
0 stars 0 forks source link

Recursively unhide elements on progress button click #18

Closed toshikurauchi closed 2 years ago

toshikurauchi commented 2 years ago

Fix https://github.com/insper-education/devlife-content/issues/563

igordsm commented 2 years ago

Isso me parece muito errado. Não deveria ser necessário. Já tentou usar next/previousSibling em vez de nextElementSibling?

Pelo que olhei no MDN é isso que a gente quer.

toshikurauchi commented 2 years ago

O melhor é a sequência: "Isso me parece muito errado" -> merged commit into main 😆

toshikurauchi commented 2 years ago

Só pra registrar aqui o motivo:

Esse código do mkdocs material coloca os wrappers nas tabelas: https://github.com/squidfunk/mkdocs-material/blob/master/src/assets/javascripts/templates/table/index.tsx

Ele é chamado na função mountContent que está dentro de um defer do rxjs.

Assim esse wrapper é colocado depois que a gente já escondeu tudo o que está depois do botão continuar. Por isso um elemento que era irmão quando escondemos tudo (no caso a table) não é mais irmão na hora de mostrar. Por esse mesmo motivo o bug não ocorre quando o usuário já clicou no botão continuar e recarrega a página.

igordsm commented 2 years ago

Obrigado :D

On Wed, Feb 9, 2022 at 9:52 AM Toshi Kurauchi @.***> wrote:

Só pra registrar aqui o motivo:

Esse código do mkdocs material coloca os wrappers nas tabelas: https://github.com/squidfunk/mkdocs-material/blob/master/src/assets/javascripts/templates/table/index.tsx

Ele é chamado na função mountContent https://github.com/squidfunk/mkdocs-material/blob/master/src/assets/javascripts/bundle.ts#L206 que está dentro de um defer do rxjs https://rxjs.dev/api/index/function/defer.

Assim esse wrapper é colocado depois que a gente já escondeu tudo o que está depois do botão continuar. Por isso um elemento que era irmão quando escondemos tudo (no caso a table) não é mais irmão na hora de mostrar. Por esse mesmo motivo o bug não ocorre quando o usuário já clicou no botão continuar e recarrega a página.

— Reply to this email directly, view it on GitHub https://github.com/insper-education/active-handout-plugins-js/pull/18#issuecomment-1033729593, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABWCBR3W6W5LYVAGH35VX3U2JPR3ANCNFSM5N2RR4SA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you modified the open/close state.Message ID: @.*** .com>

-- Igor Montagner http://igormontagner.blogspot.com/