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

Change mouse pointer to "waiting" while opening documents #1111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uliska
Copy link
Collaborator

@uliska uliska commented Sep 25, 2018

Relates to #1109

When loading large documents there may be a significant moment of
seeming inactivity. This commit changes the mouse pointer to "wait"
during that time.

NOTE: This only works when opening a document into the first tab.
Obviously some code inside AbstractDocument.load() must take
care of this (i.e. returning only when the document is fully loaded).
Therefore this pull request should not be merged yet.

@fedelibre please test this both with large original files and in conjunction with file import.

When loading large documents there may be a significant moment of
seeming inactivity. This commit changes the mouse pointer to "wait"
during that time.

NOTE: This only works when opening a document into the *first* tab.
Obviously *some* code inside AbstractDocument.load() must take
care of this (i.e. returning only when the document is *fully* loaded).
@fedelibre
Copy link
Member

I've tested it and it works as expected. It helps.
Thanks

@uliska
Copy link
Collaborator Author

uliska commented Sep 26, 2018

Maybe, but I'm of course not satisfied by the fact that it only works for the first tab ...

I have the suspicion that this delay is related to the known problem of too-long-running tasks to update the autocompletion and highlighting data. So maybe when that will have been fixed the situation might become different: Files would be opened quickly, with the highlighting kicking in delayed.

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

Successfully merging this pull request may close these issues.

None yet

2 participants