[vlc-devel] [PATCH] Fixed #19993: Made toolTips on Full Screen consistent

David Fuhrmann david.fuhrmann at gmail.com
Tue Mar 20 23:58:27 CET 2018


Hello,

-                _NS("Go to next item"));
+                _NS("Forward"));

This looks very misleading. This button is about jumping to next item, not about forwarding the same item.

Just below, there is another button for "Seek forward“. So you should not choose a description which is too close to the other button (as its done in your proposal).

If you compare to the buttons descriptions in the main window: Please note that those buttons have different features, depending if only a back and forward button is shown, or if  „next“ / „prev“ buttons are shown additionally.


-                _NS("Go to the previous item"));
+                _NS("Backward"));

Same here.

     setupButton(_timeSlider,
                 _NS("Position"),
-                _NS("Adjust the current playback position"));
+                _NS("Position"));

Are you sure choosing the same string for Accessibility title and accessibility description is the correct thing to do? I would rather assume one might leave the description blank if you actually have no description (but I’m just guessing here…)

BR. David

> Am 19.03.2018 um 12:02 schrieb Daksh Shah <daksh17336 at iiitd.ac.in>:
> 
> Now the tooltips which are shown in the Normal View are same as the ones which are shown in Full-Screen View. Adapted the shorter version.
> And also, changed the Normal View toolTip of "Fullscreen" to "Enter fullscreen"
> 
> <0001-Fixed-19993-Made-toolTips-on-Full-Screen-consistent.patch>_______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel



More information about the vlc-devel mailing list