History

Fixes cursor being hidden while Window.browse(), Window.save() are being called from the defaultScript.js thread

Thijs Wenker committed 9 years ago

Commit 6957ac51 parent 4554d8b2 files · +15 −0Browse files

Showing the changes to interface/src/scripting/WindowScriptingInterface.cpp (1 of 2 files). Show all

Unified · Split

interface/src/scripting/WindowScriptingInterface.cpp+13−0
@@ -16,6 +16,8 @@ · 15 unmodified lines
1616
1717 #include <SettingHandle.h>
1818
19+#include <display-plugins/CompositorHelper.h>
20+
1921 #include "Application.h"
2022 #include "DomainHandler.h"
2123 #include "MainWindow.h"
@@ -147,6 +149,15 @@ void WindowScriptingInterface::setPreviousBrowseLocation(const QString& location · 125 unmodified lines
147149 Setting::Handle<QVariant>(LAST_BROWSE_LOCATION_SETTING).set(location);
148150 }
149151
152+/// Makes sure that the reticle is visible, use this in blocking forms that require a reticle and
153+/// might be in same thread as a script that sets the reticle to invisible
154+void WindowScriptingInterface::ensureReticleVisible() const {
155+ auto compositorHelper = DependencyManager::get<CompositorHelper>();
156+ if (!compositorHelper->getReticleVisible()) {
157+ compositorHelper->setReticleVisible(true);
158+ }
159+}
160+
150161 /// Display an open file dialog. If `directory` is an invalid file or directory the browser will start at the current
151162 /// working directory.
152163 /// \param const QString& title title of the window
@@ -154,6 +165,7 @@ void WindowScriptingInterface::setPreviousBrowseLocation(const QString& location · 1 unmodified line
154165 /// \param const QString& nameFilter filter to filter filenames by - see `QFileDialog`
155166 /// \return QScriptValue file path as a string if one was selected, otherwise `QScriptValue::NullValue`
156167 QScriptValue WindowScriptingInterface::browse(const QString& title, const QString& directory, const QString& nameFilter) {
168+ ensureReticleVisible();
157169 QString path = directory;
158170 if (path.isEmpty()) {
159171 path = getPreviousBrowseLocation();
@@ -175,6 +187,7 @@ QScriptValue WindowScriptingInterface::browse(const QString& title, const QStrin · 15 unmodified lines
175187 /// \param const QString& nameFilter filter to filter filenames by - see `QFileDialog`
176188 /// \return QScriptValue file path as a string if one was selected, otherwise `QScriptValue::NullValue`
177189 QScriptValue WindowScriptingInterface::save(const QString& title, const QString& directory, const QString& nameFilter) {
190+ ensureReticleVisible();
178191 QString path = directory;
179192 if (path.isEmpty()) {
180193 path = getPreviousBrowseLocation();