This patch uses the "CenterInView" message position instead of
"TopInView" and "BottomInView". The reason for this change is
better visibility, since the corner widgets quickly hide unnoticed
and the user doesn't know what's going on.
Details
make test, manual testing
Diff Detail
- Repository
- R39 KTextEditor
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
That could very well be the case. Another possibility would be to simply show an icon, like Qt Creator does it.
Or change the text, but I tried to avoid long messages (which will be a possible issue in translations anyways)... Suggestions welcome :-)
@neundorf Did you try the patch? Or anyone else? I would like to get this right instead of having yet another solution that is not good enough.
I tried it, it is more obvious, if that is perfect, I don't know.
I will push that now, before it rots ;=) (like other requests).
And if the outcry is large, we can still revert it.
I think there is no way to see if it is better beside user testing.
Retroactive +1 on the positioning and wording, but I would have stayed with an Informational (blue) KMessageWidget instead of making it Positive (and hence, green). Eh, not a huge deal. Shoulda piped up sooner!