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

R Syntax: Load QML Controls as QML Module #5776

Closed
wants to merge 14 commits into from
Closed

R Syntax: Load QML Controls as QML Module #5776

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
bfef5ff
Load QML Controls as QML Module
boutinb Jan 10, 2025
f54810c
More
boutinb Jan 10, 2025
1e6a183
Remove Filter object from QML Components
boutinb Jan 13, 2025
e32ad10
Update submodule so that it uses QML import without version
boutinb Jan 13, 2025
769d232
JASP QML Controls base objects should be registered automatically
boutinb Jan 14, 2025
104ca01
Add ALTNavigation
boutinb Jan 14, 2025
f7c488d
Use CommanData in QML Components
boutinb Jan 14, 2025
6b458a9
Final touch
boutinb Jan 14, 2025
27ea180
use DataSetPointer again, jusat make sure DatabaseInterface is loaded
JorisGoosen Jan 15, 2025
73d75df
_singleton should be accessed from cpp to make sure it is in the righ…
JorisGoosen Jan 15, 2025
631bbb5
undo ColumnUtils::doubleToString* -> Utils::
JorisGoosen Jan 15, 2025
497bde8
undo some more changes and move init of databaseinterface singleton t…
JorisGoosen Jan 15, 2025
2fda389
small tweaks
JorisGoosen Jan 15, 2025
07df33c
try to get it to build on arch linux
Jan 15, 2025
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
2 changes: 1 addition & 1 deletion Common/jaspColumnEncoder
Open in desktop
Submodule jaspColumnEncoder updated 2 files
+19 −0 utils.cpp
+3 −0 utils.h
0 CommonData/jsonutilities.cpp → Common/jsonutilities.cpp
View file
Open in desktop
File renamed without changes.
0 CommonData/jsonutilities.h → Common/jsonutilities.h
View file
Open in desktop
File renamed without changes.
8 changes: 4 additions & 4 deletions CommonData/column.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -983,7 +983,7 @@ stringvec Column::nonFilteredLevels()
levels.insert(label->label());
}
else if(!isEmptyValue(_dbls[r]))
levels.insert(ColumnUtils::doubleToString(_dbls[r]));
levels.insert(Utils::doubleToString(_dbls[r]));
}

// Use the right label order
Expand Down Expand Up @@ -1164,7 +1164,7 @@ std::string Column::doubleToDisplayString(double dbl, bool fancyEmptyValue, bool
ignoreEmptyValue = ignoreEmptyValue && !std::isnan(dbl);

if (isEmptyValue(dbl) && !ignoreEmptyValue) return fancyEmptyValue ? EmptyValues::displayString() : "";
else return ColumnUtils::doubleToString(dbl);
else return Utils::doubleToString(dbl);
}

std::string Column::operator[](size_t row)
Expand Down Expand Up @@ -1416,7 +1416,7 @@ bool Column::replaceDoubleLabelFromRowWithDouble(size_t row, double dbl)
dblsRef = dbl;

_labelsTempDbls[row] = dbl;
_labelsTemp[row] = ColumnUtils::doubleToString(dbl);
_labelsTemp[row] = Utils::doubleToString(dbl);

return true;
}
Expand Down Expand Up @@ -1580,7 +1580,7 @@ bool Column::setValue(size_t row, const std::string & value, const std::string &

if(justAValue && !newLabel && itsADouble)
{
const std::string valueDbl = ColumnUtils::doubleToString(newDoubleToSet);
const std::string valueDbl = Utils::doubleToString(newDoubleToSet);
newLabel = labelByValue(valueDbl);
newLabel = newLabel ? newLabel : labelByValueAndDisplay(valueDbl, valueDbl);
}
Expand Down
20 changes: 0 additions & 20 deletions CommonData/columnutils.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,26 +210,6 @@ std::string ColumnUtils::deEuropeaniseForImport(std::string value)
return value;
}

std::string ColumnUtils::doubleToStringMaxPrec(double dbl)
{
constexpr auto max_precision{std::numeric_limits<long double>::digits10 + 1};
return doubleToString(dbl, max_precision);
}

std::string ColumnUtils::doubleToString(double dbl, int precision)
{
JASPTIMER_SCOPE(ColumnUtils::doubleToString);

if (dbl > std::numeric_limits<double>::max()) return "∞";
if (dbl < std::numeric_limits<double>::lowest()) return "-∞";

std::stringstream conv; //Use this instead of std::to_string to make sure there are no trailing zeroes (and to get full precision)
conv << std::setprecision(precision);
conv << dbl;
return conv.str();
}


// hex should be 4 hexadecimals characters
std::string ColumnUtils::_convertEscapedUnicodeToUTF8(std::string hex)
{
Expand Down
3 changes: 0 additions & 3 deletions CommonData/columnutils.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ class ColumnUtils

static void convertEscapedUnicodeToUTF8( std::string & inputStr);
static std::string deEuropeaniseForImport( std::string value); //Convert a string to a double with a dot for a separator

static std::string doubleToString( double dbl, int precision = 10);
static std::string doubleToStringMaxPrec( double dbl);

static bool convertVecToInt( const stringvec & values, intvec & intValues, intset & uniqueValues);
static bool convertVecToDouble( const stringvec & values, doublevec & doubleValues);
Expand Down
6 changes: 6 additions & 0 deletions CommonData/dataset.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -619,3 +619,9 @@ bool DataSet::initColumnWithStrings(int colIndex, const std::string & newName, c
return anyChanges || column->type() != prevType;
}

bool DataSet::isFilterNameFree(const std::string& filterName)
{
return -1 == db().filterGetId(filterName);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes more sense to have this on Filter because the name is independent of the DataSet in question.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you make it non-static?

}


1 change: 1 addition & 0 deletions CommonData/dataset.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ class DataSet : public DataSetBaseNode
void setWorkspaceEmptyValues( const stringset& values);
const std::string & description() const { return _description; }
void setDescription( const std::string& desc);
bool isFilterNameFree( const std::string& filterName);

private:
void upgradeTo019(const Json::Value & emptyVals);
Expand Down
5 changes: 0 additions & 5 deletions CommonData/filter.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,11 +170,6 @@ bool Filter::checkForUpdates()
return false;
}

bool Filter::filterNameIsFree(const std::string &filterName)
{
return -1 == DatabaseInterface::singleton()->filterGetId(filterName);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which is also where it was...

}

void Filter::reset()
{
if(!_data->writeBatchedToDB())
Expand Down
7 changes: 2 additions & 5 deletions CommonData/filter.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,8 @@ class Filter : public DataSetBaseNode
void dbLoad();
bool dbLoadResultAndError(); ///< Loads (updated) filtervalues from database and the (possible) error msg, returns true if an error is set
void dbDelete();
void incRevision() override;
bool checkForUpdates();

static bool filterNameIsFree(const std::string & filterName);

void incRevision() override;
bool checkForUpdates();
void reset();

DatabaseInterface & db();
Expand Down
2 changes: 1 addition & 1 deletion Desktop/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ qt_add_resources(
file(GLOB_RECURSE QML_RESOURCE_FILES "${CMAKE_CURRENT_LIST_DIR}/components/*")
qt_add_resources(
JASP
"widgets"
"widgetsandtheme"
PREFIX
"/"
BASE
Expand Down
0 ...nents/components/JASP/Theme/DarkTheme.qml → Desktop/components/JASP/Theme/DarkTheme.qml
View file
Open in desktop
File renamed without changes.
6 changes: 3 additions & 3 deletions ...omponents/components/JASP/Theme/Theme.qml → Desktop/components/JASP/Theme/Theme.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
//


import QtQuick 2.12
import QtQuick.Controls 2.12
import JASP 1.0
import QtQuick
import QtQuick.Controls
import JASP

JaspTheme
{
Expand Down
0 QMLComponents/components/JASP/Theme/qmldir → Desktop/components/JASP/Theme/qmldir
View file
Open in desktop
File renamed without changes.
6 changes: 3 additions & 3 deletions Desktop/components/JASP/Widgets/AnalysisFormExpander.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import QtQuick 2.11
import QtQuick.Controls 2.4
import JASP.Controls 1.0
import QtQuick
import QtQuick.Controls
import JASP.Controls
import Qt5Compat.GraphicalEffects

DropArea
Expand Down
5 changes: 2 additions & 3 deletions Desktop/components/JASP/Widgets/AnalysisForms.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import QtQuick 2.11
import QtQuick.Controls 2.4
import QtQuick
import JASP
import JASP.Controls 1.0
import JASP.Controls


FocusScope
Expand Down
10 changes: 5 additions & 5 deletions Desktop/components/JASP/Widgets/ComputeColumnWindow.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import QtQuick 2.7
import QtQuick.Controls 2.12
import QtQuick.Layouts 1.3
import JASP.Controls 1.0 as JaspControls
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import JASP.Controls as JaspControls
import JASP

import "FilterConstructor"
Expand Down Expand Up @@ -105,7 +105,7 @@ FocusScope
{
id: computeColumnEdit

RSyntaxHighlighterQuick
JaspControls.RSyntaxHighlighterQuick
{
textDocument: computeColumnEdit.textDocument
}
Expand Down
4 changes: 2 additions & 2 deletions Desktop/components/JASP/Widgets/CreateComputeColumnDialog.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import QtQuick 2.9
import QtQuick.Controls 2.2
import QtQuick
import QtQuick.Controls
import JASP.Controls as JaspControls


Expand Down
6 changes: 3 additions & 3 deletions Desktop/components/JASP/Widgets/CustomMenu.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@
// <http://www.gnu.org/licenses/>.
//

import QtQuick 2.15
import QtQuick.Controls 2.4
import JASP.Controls 1.0 as JASPControl
import QtQuick
import QtQuick.Controls
import JASP.Controls as JASPControl
import Qt5Compat.GraphicalEffects

FocusScope
Expand Down
6 changes: 3 additions & 3 deletions Desktop/components/JASP/Widgets/DataPanel.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import QtQuick.Controls 6.0
import QtQuick.Layouts 1.0
import QtQuick 2.15
import QtQuick.Controls
import QtQuick.Layouts
import QtQuick


Rectangle
Expand Down
10 changes: 5 additions & 5 deletions Desktop/components/JASP/Widgets/EnginesWindow.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import QtQuick 2.12
import QtQuick.Window 2.12
import QtWebEngine 1.8
import JASP.Widgets 1.0
import JASP.Controls 1.0
import QtQuick
import QtQuick.Window
import QtWebEngine
import JASP.Widgets
import JASP.Controls

Window
{
Expand Down
11 changes: 5 additions & 6 deletions Desktop/components/JASP/Widgets/FileMenu/BreadCrumbs.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import QtQuick 2.11
import QtQuick.Layouts 1.3
import QtQuick.Controls 2.4

import JASP.Widgets 1.0
import JASP.Controls 1.0
import QtQuick
import QtQuick.Layouts
import QtQuick.Controls
import JASP.Widgets
import JASP.Controls

ListView
{
Expand Down
11 changes: 5 additions & 6 deletions Desktop/components/JASP/Widgets/FileMenu/Computer.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import QtQuick 2.9
import QtQuick.Controls 2.2
import QtQuick.Layouts 1.3

import JASP.Controls 1.0
import JASP.Widgets 1.0
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import JASP.Controls
import JASP.Widgets

Item
{
Expand Down
4 changes: 2 additions & 2 deletions Desktop/components/JASP/Widgets/FileMenu/CurrentFile.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import QtQuick 2.9
import QtQuick.Controls 2.2
import QtQuick
import QtQuick.Controls


Item
Expand Down
6 changes: 3 additions & 3 deletions Desktop/components/JASP/Widgets/FileMenu/DataLibrary.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import QtQuick 2.9
import QtQuick.Controls 2.2
import QtQuick.Layouts 1.3
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts


Item
Expand Down
8 changes: 4 additions & 4 deletions Desktop/components/JASP/Widgets/FileMenu/DataLibraryBreadCrumbs.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import QtQuick 2.0
import QtQuick.Layouts 1.3
import QtQuick.Controls 2.2
import JASP.Theme 1.0
import QtQuick
import QtQuick.Layouts
import QtQuick.Controls
import JASP.Theme

ListView
{
Expand Down
8 changes: 4 additions & 4 deletions Desktop/components/JASP/Widgets/FileMenu/FileList.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import QtQuick 2.11
import QtQuick.Controls 2.4
import JASP.Controls 1.0
import QtQuick
import QtQuick.Controls
import JASP.Controls as JC


ListView
Expand Down Expand Up @@ -81,7 +81,7 @@ ListView
hasBreadCrumbs: listView.breadCrumbs !== null
}

JASPScrollBar
JC.JASPScrollBar
{
id: rightscrollbar
flickable: parent
Expand Down
10 changes: 5 additions & 5 deletions Desktop/components/JASP/Widgets/FileMenu/FileMenu.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import QtQuick 2.11
import QtQuick.Controls 2.2
import QtQuick.Layouts 1.3
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts

import JASP.Controls 1.0
import JASP.Controls
import JASP
import FileOperation 1.0
import FileOperation

FocusScope
{
Expand Down
6 changes: 2 additions & 4 deletions Desktop/components/JASP/Widgets/FileMenu/ListItem.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
import QtQuick 2.11
import QtQuick.Controls 2.4
import JASP.Controls 1.0

import QtQuick
import QtQuick.Controls

FocusScope
{
Expand Down
13 changes: 6 additions & 7 deletions Desktop/components/JASP/Widgets/FileMenu/MenuHeader.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import QtQuick 2.9
import QtQuick.Controls 2.2
import QtQuick.Layouts 1.3

import JASP.Widgets 1.0
import JASP.Controls 1.0
import QtQuick
import QtQuick.Controls as QtC
import QtQuick.Layouts
import JASP.Widgets
import JASP.Controls

Item
{
Expand Down Expand Up @@ -63,7 +62,7 @@ Item
}
}

ToolSeparator
QtC.ToolSeparator
{
id: firstSeparator

Expand Down
11 changes: 5 additions & 6 deletions Desktop/components/JASP/Widgets/FileMenu/OSF.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,10 @@
// <http://www.gnu.org/licenses/>.
//

import QtQuick 2.0
import QtQuick.Controls 2.2

import JASP.Controls 1.0
import JASP.Widgets 1.0
import QtQuick
import QtQuick.Controls as QtC
import JASP.Controls
import JASP.Widgets

//import QtQuick.Layouts 1.3

Expand Down Expand Up @@ -120,7 +119,7 @@ Item
sortMenuModel: fileMenuModel.osf.sortedMenuModel
}

ToolSeparator
QtC.ToolSeparator
{
id : firstSeparator
visible : loggedin
Expand Down
8 changes: 4 additions & 4 deletions Desktop/components/JASP/Widgets/FileMenu/PrefsAdvanced.qml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import QtQuick 2.12
import QtQuick.Controls 2.12
import JASP.Widgets 1.0
import JASP.Controls 1.0
import QtQuick
import QtQuick.Controls
import JASP.Widgets
import JASP.Controls

ScrollView
{
Expand Down
Loading
Loading