Plume

Fix the navigation menu not opening on touch #690

Integrado

marek-lach integrou 6 cometimento(s) do ramo master no ramo master

marek-lach comentou (Migrado de github.com)

Fixes the issue of the mobile menu not reacting to touch, in cases where mouse clicking is not being simulated.

elegaanz comentou (Migrado de github.com)

As you can see here, add_event_listener only takes one argument in Rust.

However, I think replacing |_: ClickEvent| with |_: TouchEnd| may work.

As you can see [here](https://circleci.com/gh/Plume-org/Plume/9186?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link), `add_event_listener` only takes one argument in Rust. However, I think replacing `|_: ClickEvent|` with `|_: TouchEnd|` may work.

marek-lach comentou (Migrado de github.com)

The problem is that touchend is exclusive for iOS. Click event should still
probably stay for everything else...

On Wed, Nov 6, 2019 at 1:05 PM Ana Gelez notifications@github.com wrote:

As you can see here
https://circleci.com/gh/Plume-org/Plume/9186?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link,
add_event_listener only takes one argument in Rust.

However, I think replacing |: ClickEvent| with |: TouchEnd| may work.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Plume-org/Plume/pull/690?email_source=notifications&email_token=AK6JO6PMTLX7PJAWICWRN53QSKXKBA5CNFSM4JI3LRB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDGJZXA#issuecomment-550280412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK6JO6P37W6LS2U26ZM4KG3QSKXKBANCNFSM4JI3LRBQ
.

The problem is that touchend is exclusive for iOS. Click event should still probably stay for everything else... On Wed, Nov 6, 2019 at 1:05 PM Ana Gelez <notifications@github.com> wrote: > As you can see here > <https://circleci.com/gh/Plume-org/Plume/9186?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link>, > add_event_listener only takes one argument in Rust. > > However, I think replacing |_: ClickEvent| with |_: TouchEnd| may work. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/Plume-org/Plume/pull/690?email_source=notifications&email_token=AK6JO6PMTLX7PJAWICWRN53QSKXKBA5CNFSM4JI3LRB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDGJZXA#issuecomment-550280412>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AK6JO6P37W6LS2U26ZM4KG3QSKXKBANCNFSM4JI3LRBQ> > . >

elegaanz comentou (Migrado de github.com)

It seems to work on all mobile browsers: https://developer.mozilla.org/en-US/docs/Web/API/Document/touchend_event#Browser_compatibility and we don't really care about desktop here I think, right?

👍 1

codecov[bot] comentou (Migrado de github.com)

Codecov Report

Merging #690 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #690   +/-   ##
=======================================
  Coverage   39.25%   39.25%           
=======================================
  Files          72       72           
  Lines        9498     9498           
  Branches     2265     2265           
=======================================
  Hits         3728     3728           
  Misses       4713     4713           
  Partials     1057     1057

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/690?src=pr&el=h1) Report > Merging [#690](https://codecov.io/gh/Plume-org/Plume/pull/690?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/f0846ff5465e3e28807570f0110437098f60efa1?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #690 +/- ## ======================================= Coverage 39.25% 39.25% ======================================= Files 72 72 Lines 9498 9498 Branches 2265 2265 ======================================= Hits 3728 3728 Misses 4713 4713 Partials 1057 1057 ```

marek-lach comentou (Migrado de github.com)

Ok, I've just modified it that way then.

👍 1

igalic (Migrado de github.com) aprovou estas modificações

Revisores

A integração foi executada no cometimento c66ef5f85d.

Também pode ver as instruções para a linha de comandos.

Passo 1:

No seu repositório, crie um novo ramo e teste as modificações.

git checkout -b master master

git pull origin master

Passo 2:

Integre as modificações e envie para o Forgejo.

git checkout master

git merge --no-ff master

git push origin master

Inicie a sessão para participar neste diálogo.

Sem revisores

Sem etapa

Sem encarregados

1 Participantes

Notificações

Data de vencimento

A data de vencimento é inválida ou está fora do intervalo permitido. Por favor, use o formato 'aaaa-mm-dd'.

Sem data de vencimento definida.

Dependências

Não estão definidas dependências.

Carregando…

Ainda não há conteúdo.

Rótulos Etapas

Fix the navigation menu not opening on touch #690

Codecov Report

Revisores

Passo 1:

Passo 2: