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.

Sign up for GitHub

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

Jump to bottom

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

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

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

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion frescobaldi_app/app.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
import os
import sys

from PyQt5.QtCore import QSettings, QThread
from PyQt5.QtCore import QSettings, QThread, Qt
from PyQt5.QtWidgets import QApplication

import appinfo
Expand Down Expand Up @@ -78,6 +78,7 @@ def openUrl(url, encoding=None):
If there is already a document with that url, it is returned.

"""
qApp.setOverrideCursor(Qt.WaitCursor)
d = findDocument(url)
if not d:
# special case if there is only one document:
Expand All @@ -92,8 +93,12 @@ def openUrl(url, encoding=None):
if not url.isEmpty():
d.load(url)
else:
#FIXME: Here no "time" is used, so the mouse pointer is
# restored prematurely. Somehow restoring should be
# postponed until syntax hinglighting is completed.
import document
d = document.EditorDocument.new_from_url(url, encoding)
qApp.restoreOverrideCursor()
return d

def findDocument(url):
Expand Down