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

Background clicking is now not working #1273

Closed
tuomas2 opened this issue Jul 8, 2021 · 4 comments
Closed

Background clicking is now not working #1273

tuomas2 opened this issue Jul 8, 2021 · 4 comments

Comments

@tuomas2
Copy link
Contributor

tuomas2 commented Jul 8, 2021

Due to #1187 background clicking is not working at all.
Earlier it was used to stop highlighting etc.
Need to think about how to fix this...

Perhaps these rules would work:

  • If 1-click modal is open, close it first (and don't open again until next click)
  • if there's highlight to be removed, do it first, and only next tap open 1-click modal
@tuomas2
Copy link
Contributor Author

tuomas2 commented Jul 8, 2021

Now it is also impossible to change active window by tapping, without opening a modal.
Fix: only open modal if window is already active.

@agrogers
Copy link
Contributor

agrogers commented Jul 8, 2021 via email

@agrogers
Copy link
Contributor

agrogers commented Jul 8, 2021

One thing I would suggest is that the dialog is closed on a single click if it is already open and the user clicks on an alternate verse. I liked that way of closing the previous dialog.

I can see that the selected verse changes when I click on a different verse. So that is a nice option. And of course I can close the dialog with the X.

Oh, by the way, good work!

@agrogers
Copy link
Contributor

agrogers commented Jul 9, 2021

I like both those suggestions you made.

I would add that if a dialog is open and the user clicks on a bookmark then the dialog should change to reflect the new bookmark selection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants