feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

DesignerCore: Make stylesheet merger transfer custom properties

Browse Source 
Do not ignore root level properties and add them to the final
model (ignore the state changes properties though).

Task-number: QDS-13688
Change-Id: Icbd544e0c1cb866d7bec19fe305950bb6a384247
Reviewed-by: Marco Bubke <marco.bubke@qt.io>
Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>
This commit is contained in:
Przemyslaw Lewandowski
2025-03-25 14:13:57 +01:00
parent bc4779f7d6
commit 4f4ad2d4b9
2 changed files with 29 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
src/plugins/qmldesigner/libs/designercore/designercoreutils/stylesheetmerger.cpp
View File

@@ -328,8 +328,20 @@ void StylesheetMerger::adjustNodeIndex(ModelNode &node)
parentListProperty.slide(currentIndex, info.parentIndex);
}
void StylesheetMerger::applyStyleProperties(ModelNode &templateNode, const ModelNode &styleNode)
void StylesheetMerger::applyStyleProperties(ModelNode &templateNode,
const ModelNode &styleNode,
bool isRootNode)
{
// using isRootNode allows transferring custom properties that may have been added in Qt Bridge
auto addProperty = [&templateNode, isRootNode](const VariantProperty &variantProperty) {
if (isRootNode)
templateNode.variantProperty(variantProperty.name())
.setDynamicTypeNameAndValue(variantProperty.dynamicTypeName(),
variantProperty.value());
else
templateNode.variantProperty(variantProperty.name()).setValue(variantProperty.value());
};
const QRegularExpression regEx("[a-z]", QRegularExpression::CaseInsensitiveOption);
for (const VariantProperty &variantProperty : styleNode.variantProperties()) {
if (templateNode.hasBindingProperty(variantProperty.name())) {
@@ -337,16 +349,20 @@ void StylesheetMerger::applyStyleProperties(ModelNode &templateNode, const Model
// replace it with the corresponding variant property.
if (!templateNode.bindingProperty(variantProperty.name()).expression().contains(regEx)) {
templateNode.removeProperty(variantProperty.name());
templateNode.variantProperty(variantProperty.name()).setValue(variantProperty.value());
addProperty(variantProperty);
}
} else {
if (variantProperty.holdsEnumeration()) {
templateNode.variantProperty(variantProperty.name()).setEnumeration(variantProperty.enumeration().toEnumerationName());
} else {
templateNode.variantProperty(variantProperty.name()).setValue(variantProperty.value());
}
if (variantProperty.holdsEnumeration())
templateNode.variantProperty(variantProperty.name())
.setEnumeration(variantProperty.enumeration().toEnumerationName());
else
addProperty(variantProperty);
}
}
if (isRootNode)
return;
syncBindingProperties(templateNode, styleNode);
syncNodeProperties(templateNode, styleNode, true);
syncNodeListProperties(templateNode, styleNode, true);
@@ -473,6 +489,9 @@ void StylesheetMerger::merge()
// build a hash of generated replacement ids
setupIdRenamingHash();
// transfer custom root properties
applyStyleProperties(templateRootNode, styleRootNode, true);
//in case we are replacing the root node, just do that and exit
if (m_styleView->hasId(templateRootNode.id())) {
replaceRootNode(templateRootNode);

4
src/plugins/qmldesigner/libs/designercore/designercoreutils/stylesheetmerger.h
View File

@@ -40,7 +40,9 @@ private:
bool idExistsInBothModels(const QString& id);
void replaceNode(ModelNode&, ModelNode&);
void replaceRootNode(ModelNode& templateRootNode);
void applyStyleProperties(ModelNode &templateNode, const ModelNode &styleNode);
void applyStyleProperties(ModelNode &templateNode,
const ModelNode &styleNode,
bool isRootNode = false);
void adjustNodeIndex(ModelNode &node);
void setupIdRenamingHash();
ModelNode createReplacementNode(const ModelNode &styleNode, ModelNode &modelNode);