kindred/create
1
1
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

Improve UX for Rename Property Group with tooltip

Browse Source
This commit is contained in:
Krrish777
2026-01-04 00:13:17 +05:30
committed by Chris Hennes
parent 32e47bdda6
commit 32ee950210
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/Gui/propertyeditor/PropertyEditor.cpp
View File

@@ -955,6 +955,7 @@ void PropertyEditor::removeProperties(const std::unordered_set<App::Property*>&
void PropertyEditor::contextMenuEvent(QContextMenuEvent*)
{
QMenu menu;
menu.setToolTipsVisible(true);
auto contextIndex = currentIndex();
std::unordered_set<App::Property*> props = acquireSelectedProperties();
@@ -976,10 +977,21 @@ void PropertyEditor::contextMenuEvent(QContextMenuEvent*)
// rename property group
if (!props.empty() && std::all_of(props.begin(), props.end(), [](auto prop) {
return prop->testStatus(App::Property::PropDynamic)
&& !boost::starts_with(prop->getName(), prop->getGroup());
return prop->testStatus(App::Property::PropDynamic);
})) {
menu.addAction(tr("Rename Property Group"))->setData(QVariant(MA_EditPropGroup));
QAction* renameGroupAction = menu.addAction(tr("Rename Property Group"));
renameGroupAction->setData(QVariant(MA_EditPropGroup));
// Check if any property name starts with its group name
bool hasGroupPrefix = std::any_of(props.begin(), props.end(), [](auto prop) {
return boost::starts_with(prop->getName(), prop->getGroup());
});
if (hasGroupPrefix) {
renameGroupAction->setEnabled(false);
renameGroupAction
->setToolTip(tr("Cannot rename group: one or more properties have names that start with the group name"));
}
}
// rename property