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

Leseposition-Häckchen in Action Bar zeigt unklickbares call out #245

Open
patrickvenetz opened this issue Oct 31, 2022 · 1 comment
Open
Labels
app: www Frontend server bug Something isn't working

Comments

@patrickvenetz
Copy link
Contributor

Bug Description

Leseposition-Häckchen in Action Bar zeigt unklickbare Call-Out-Option «als gelesen markieren»; das Häckchen selbst markiert einen Beitrag bereits als gelesen, auf die Call-Out-Option kann ich gar nicht erst klicken.

Bildschirmaufnahme.2022-10-29.um.15.05.59.mov

(Das Leseposition-Häckchen gibts auch in der "floating" Action Bar bei einem Artikel. Da sehe ich die Call-Out-Option gar nicht erst, sondern wird bei Klick aufs Häckchen direkt als gelesen markiert.)

Expected Behaviour

Ich kann die Call-Out-Option anklicken (oder sie wird mir gar nicht erst angezeigt).

Steps to reproduce

  1. Öffne Artikel
  2. Scrolle Artikel etwas nach unten, so, dass die Leseposition greift und z. B. 38 Prozent gelesen angezeigt wird.
  3. Scrolle zur Action Bar unter der Autorinnenzeile und klick auf Häckchen neben Leseposition

Environment

Other (please specify in version field below)

Version

Brave, MacOS

@patrickvenetz patrickvenetz added the bug Something isn't working label Oct 31, 2022
@trm217
Copy link
Contributor

trm217 commented Nov 17, 2022

The issue seems to be these two onClick-handlers (set-read, unset-read).
In the code below these two components are then used as the CalloutMenu trigger.
(The callout passed a onPointerDown handler to both of them, so they have both a onPointerDown and onClick defined.
On mobile this doesn't seems to be an issue, however on desktop the behaviour you mentioned occurs.
I suppose we should be fine if we just remove the two onClick handlers 🤔

@trm217 trm217 added the app: www Frontend server label Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app: www Frontend server bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants