This patch adds the showFloatingMessage() function to the libkis View python API, and uses it in the Ten Brushes plugin so that the plugin's mechanism to activate the previous preset can behave and appear just like the Switch to Previous Brush Preset action (a native Krita action).
showFloatingMessage could be useful for other python plugins that implement actions.
Attached is a picture of showFloatingMessage:
| Lint Skipped |
| Unit Tests Skipped |
I'm not sure how strict we are with this stuff in the C++ code, but you introduced a couple of trailing whitespaces:
> git apply /tmp/D18668.diff
/tmp/D18668.diff:13: trailing whitespace.
* @param priority: 0 = High, 1 = Medium, 2 = Low. Higher priority
/tmp/D18668.diff:33: trailing whitespace.
p = static_cast<KisFloatingMessage::Priority>(priority);
warning: 2 lines add whitespace errors.Otherwise this looks like a useful feature to me. What do other people think?
| plugins/python/tenbrushes/tenbrushes.py | ||
|---|---|---|
| 86 | The floating messegage needs to be called in this else-branch, too. Otherwise you only get it when you switch back to the previous brush by hitting a shortcut a second time. | |