mwganson / DynamicData

FreeCAD workbench for creating and managing custom property container objects.
GNU Lesser General Public License v2.1
41 stars 8 forks source link

DynamicData feedback #53

Open ifohancroft opened 3 years ago

ifohancroft commented 3 years ago

I absolutely love the workbench and its idea, so obviously I decided to give it a try. However, I have found a couple of things that made me decide that I will not be using it, and I will stick to using Spreadsheet instead.

I am hoping that sharing my feedback and my deal breakers would be helpful and not considered as me bashing on it.

  1. As soon as you remove the DynamicData property from the DynamicData object, you can no longer add custom properties to it.
  2. The property name has a prefix (the dd prefix). So if I create a DynamicData object and rename it to Variables then add DynamicData property of type Length and rename it to width, I can't later in an expression call it with Variables.width, I must call it with Variables.ddwidth.
  3. The tooltip can't be changed and doesn't automatically change with the name. If I add a DynamicData property of type Length the tooltip automatically becomes Length and will always be Length, where I'd like to change it to what I have chosen for the name, or to a custom string.
silopolis commented 2 years ago
1. As soon as you remove the DynamicData property from the DynamicData object, you can no longer add custom properties to it.

If I understand correctly and after testing, this seems to be fixed.

2. The property name has a prefix (the dd prefix). So if I create a DynamicData object and rename it to `Variables` then add DynamicData property of type `Length` and rename it to `width`, I can't later in an expression call it with `Variables.width`, I must call it with `Variables.ddwidth`.

Like said in YouTube video comments, I fully second that.

3. The tooltip can't be changed and doesn't automatically change with the name. If I add a DynamicData property of type `Length` the tooltip automatically becomes `Length` and will always be `Length`, where I'd like to change it to what I have chosen for the name, or to a custom string.

Tooltip can be set using the "Set Tooltip" command in the WB menu.

silopolis commented 2 years ago
2. The property name has a prefix (the dd prefix). So if I create a DynamicData object and rename it to `Variables` then add DynamicData property of type `Length` and rename it to `width`, I can't later in an expression call it with `Variables.width`, I must call it with `Variables.ddwidth`.

I've opened a dedicated issue for this here #65 As mentioned there, in the mean time, properties can be renamed to remove the prefix.