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

Do not show the "please wait" overlay while the citation dialog is open. #58

Open
nsrosenqvist opened this issue Jul 2, 2021 · 2 comments

Comments

@nsrosenqvist
Copy link

I find it really inconvenient that the document no longer responds to scroll input while the citation dialog is open. Often I have notes in the document with page numbers, or I might forget where or what I wanted to cite and must then close the dialog, find the reference, and then redo the entire process. When the citation dialog opens, the view scrolls down to where the cursor is adding the entry, and I often can't see the text passage I'm working on. It doesn't help either that it often takes a while for it to open and close.

My suggestion is to either remove the white frame covering the entire document that is capturing the scroll events (while still keeping the overlaid notice message), or even better, simply allow the scroll events to propagate up to the document element (then perhaps also lower the opacity of the white frame so that the text can be seen more easily).

Thanks for considering this issue!

@adomasven
Copy link
Member

Do note that all feature requests should be posted on the Zotero Forums. See CONTRIBUTING.md.

The change to scrolling behavior is due to Google Docs switching to canvas rendering of document contents on the backend. However we could probably disable the overlay while the citation dialog is open completely, since technically Zotero does not need exclusive access to the doc at that point.

@adomasven adomasven changed the title Unable to scroll the document while citation dialog is open Do not show the "please wait" overlay while the citation dialog is open. Jul 2, 2021
@nsrosenqvist
Copy link
Author

@adomasven oh, sorry about that! This repo didn't have a CONTRIBUTING.md, and I arrived here directly by searching for the git repo, so I assumed this was the way to go about it. My apologies! Thanks for still considering it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants