search

okTurtles / group-income-simulator

Visualizer for https://github.com/okTurtles/group-income
https://groupincome.org/simulator/
1 stars 0 forks source link

#3 - mobile tooltip issue #4

Closed SebinSong closed 1 week ago

SebinSong commented 1 week ago

closes #3

Proof of the issue fix.

The app needed additional logic to detect user-click on touch-devices. so Added it there.

SebinSong commented 1 week ago

@taoeffect While testing the re-deployed website https://groupincome.org/simulator/ again, I realised this bug-fix was incomplete. When the group-size becomes large and a scroll-bar appears, The click position detection is still wrong like in the screenshot below.

So I just created a follow-up PR here: https://github.com/okTurtles/group-income-simulator/pull/5