-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
drad'n'drop improvents part 2 #34433
Conversation
part 2 display the container editor in the right panel this avoids opening a modal dialog a new type of itemData has been added as WidgetType to distinguish them in the available widgets from the containers in the form layout fixes qgis#29063
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far so awesome.
I approve - see my comments.
explicit QgsAttributeFormContainerEdit( QTreeWidgetItem *item, QWidget *parent = nullptr ); | ||
|
||
|
||
void updateItemData(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't it need docs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's in app, I added a short description for the class only
@@ -941,6 +981,9 @@ QTreeWidgetItem *DnDTree::addItem( QTreeWidgetItem *parent, QgsAttributesFormPro | |||
case QgsAttributesFormProperties::DnDTreeItemData::HtmlWidget: | |||
//no icon for HtmlWidget | |||
break; | |||
|
|||
case QgsAttributesFormProperties::DnDTreeItemData::WidgetType: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will never happen since this is all in the other if clause checking for container (old thing I mentioned in the last pr). Maybe you are keeping it for your upcoming implementations.
Anyway, I would check here for WidgetType and add this newItem->setBackground( 0, QBrush( Qt::lightGray ) );
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will take care of the icons later.
But sadly the svg are missing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry, the fix for the light gray is coming in next round
drad'n'drop improvents part 2
* [FEATURE] Merge pull request #34411 from 3nids/dnd_sync_selec improvement of the Drag'n'Drop designer * Merge pull request #34433 from 3nids/dnd_part2 drad'n'drop improvents part 2 * drag'n'drop improvements part 3 (#34479) * drag'n'drop improvements part 3 this completely removes hidden config accessible under double click now a widget groupbox (by opposition to the field config) is shown on top of the right panel * remove assert from a left over (fix crash) (#34503) the assert was copied from other class and I was running with RelWithDebugInfo and not Debug. * Update CMakeLists.txt * Update CMakeLists.txt * move to moc headers * sip include
display the container editor in the right panel
this avoids opening a modal dialog
a new type of itemData has been added as WidgetType
to distinguish them in the available widgets from the containers in the form layout
fixes #29063