A timer is used to automatically hide the notification 3 seconds after it is being shown.
The duration is currently not user configurable.
BUG: 406603
ngraham |
Elisa |
A timer is used to automatically hide the notification 3 seconds after it is being shown.
The duration is currently not user configurable.
BUG: 406603
3 seconds after it is being shown, the notification is automatically removed
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Thanks for this!
I think 3 seconds might be a bit too fast. I can imagine non-tech-users barely even noticing it during that time. I would recommend 5 or 6 seconds.
An hardcoded value can still be too short, especially when the notification message is translated (usually translations are longer than English texts...).
What about making it dependent on the length of the text itself?
The notification system in Plasma currently does this. In practice... it doesn't work. Calculating how long it takes to read something isn't really possible; because everyone reads at a different pace, and some people will notice the message sooner than other people, it's just a guess, same as a hardcoded value. However there's the additional irritation that the timeout changes over time.
If we think that 6 seconds might still too short for some languages, I would recommend increasing it to 7 or 8 instead of trying to be clever. :) But I think there's a good chance 6 will be fine.
I am a bit lost about the best solution.
Do you think it could make sense to let translators have a way to tweak the delay (including a minimum duration enforced by Elisa) ?
I think the simplest approach is to just make sure the duration covers the worst case scenario for languages like Brazilian Portuguese and German that can be very wordy. If it sticks around for a second or two longer than necessary for English, I don't think that's really a problem.
I think 6 seconds is probably long enough already, but if we think it isn't, then let's increase the timeout to 7 seconds and then ship it. :)
rebase on top of 0.4 and sets the hide delay to 7 seconds
src/qml/MediaPlayListView.qml | ||
---|---|---|
262 | Please remove this line and the extra line below |
src/qml/MediaPlayListView.qml | ||
---|---|---|
262 | Is it really easier to read? The readability is an important quality criteria. |
src/qml/MediaPlayListView.qml | ||
---|---|---|
262 | That's how the rest file reads ; ) |