This patch adds the word "widget" onto the end of the string that's currently "Remove this [widget]", which yields the following user interface improvements:
| No Linters Available |
| No Unit Test Coverage |
| Buildable 1777 | |
| Build 1795: arc lint + arc unit |
Ping? Is the proposed wording good enough, or should we maybe instead just remove the word "this" and make it use "Remove <widget name>?
I'd say make the casing consistent, otherwise it's a clear improvement in my opinion.
| src/plasma/containment.cpp | ||
|---|---|---|
| 127 | The translation help says applet, is widget or applet the better word? But actually the "add new" dialog in plasma says widget, so I'm for that for consistency reasons. | |
| src/plasma/private/applet_p.cpp | ||
| 124 | This (Widget) is upper case (above it's lower case, which I prefer a tiny bit). | |
Now that I look at my screenshots more closely, it seems weird to have:
Configure Notes... Remove this Notes widget
Now I feel like they should match, and be either:
Configure Notes... Remove Notes
or:
Configure this Notes widget... Remove this Notes widget
Now I'm leaning towards the shorter version. Thoughts?