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

Some corrections(Json and correct order of function calls) and some improvements(saveAs,Open). #317

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

Conversation

bcvieira
Copy link

File DeviceFilterWidget: Correction based on https://doc.qt.io/qt-6/qcombobox.html#setView
Note: If you want to use the convenience views (like QListWidget, QTableWidget or QTreeWidget), make sure to call setModel() on the combobox with the convenience widgets model before calling this function.

File RundowWidget.cpp: open on dir of the last opened file, to avoid going to the entire path again to open a file on the same directory.

File RundownTreeWidget.cpp: on save as, open folder of the active Rundown or HomeDir if active == DEFAULT_NAME

Files src/Core/Commands/TemplateCommand.cpp, src/Core/Commands/TemplateCommand.h: Uses QT Json library to convert to single line json. Avoids errors on new lines and other caracters that could break the update function call.

…combobox.html#setView, Note: If you want to use the convenience views (like QListWidget, QTableWidget or QTreeWidget), make sure to call setModel() on the combobox with the convenience widgets model before calling this function.

File RundowWidget.cpp: open on dir of the last opened file, to avoid going to the entire path again to open a file on the same directory.
File RundownTreeWidget.cpp: on save as, open folder of the active Rundown or HomeDir if active == DEFAULT_NAME
Files src/Core/Commands/TemplateCommand.cpp, src/Core/Commands/TemplateCommand.h: Uses QT Json library to convert to single line json. Avoids errors on new lines and other caracters that could break the update function call.
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.

1 participant