Community
Participate
Working Groups
Build Identifier: Extend Patterns: Delete / Remove / Direct Editing / Conditional Palette Item Patterns should be able to provide features for removing and deleting model objects as well as for the direct editing facility. In addition to this, it should be possible for a pattern _not_ to provide a palette item (this is currently not possible, every pattern always appears as palette item). Reproducible: Always
Created attachment 188686 [details] proposed patch
Volker, thanks for providing this enhancement! From my point of view this looks very good, but I would like a colleague with more experience in the pattern area to have a final look next week before I submit this. I would suggest only one minor change: the name of the method that prevents adding the palette entry might be irritating: what would you think of shouldAddToPalette() instead of isCreateFeatureImplemented()? Michael
Michael, thanks for the encouraging words. As for the renaming, that's fine with me - I usually try to stick to the isFoo/canBar scheme for boolean functions, but that's as arbitrary as anything... Volker
A colleague just had a look onto the fix, the stuff looks good from his point of view as well. We shortly discussed the name of the method again and also found stciking to the is-scheme better. So I renamed the method once more to isPaletteApplicable. I have just checked in the change to head. Thanks once more for the contribution! Michael
Part of Graphiti Indigo 0.8.0
