<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">HI Daksh,<div class=""><br class=""></div><div class="">See below.<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">Am 21.03.2018 um 12:00 schrieb Daksh Shah <<a href="mailto:daksh17336@iiitd.ac.in" class="">daksh17336@iiitd.ac.in</a>>:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Thanks for your feedback. Kindly have a look at the replies in-line<br class=""><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Wed, Mar 21, 2018 at 4:28 AM David Fuhrmann <<a href="mailto:david.fuhrmann@gmail.com" class="">david.fuhrmann@gmail.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br class="">
<br class="">
- _NS("Go to next item"));<br class="">
+ _NS("Forward"));<br class="">
<br class="">
This looks very misleading. This button is about jumping to next item, not about forwarding the same item.</blockquote><div class="">I am sorry. I will fix that and change the toolTip to "Next" </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
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).<br class=""></blockquote><div class="">By proposal you mean this patch right? </div><div class=""><br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
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.<br class=""></blockquote><div class="">I understood that now. I think that it is another place where some enhancement can be done. When the two new buttons show up, the toolTips remain the same. And the functionalities change. This does not seem intuitive for new users. <br class="">The only reason I realized that was because you pointed it out.</div><div class=""><br class=""></div><div class="">Should I try to change this?</div></div></div></div></blockquote><div><br class=""></div><div>Sure, if you want to.</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
<br class="">
- _NS("Go to the previous item"));<br class="">
+ _NS("Backward"));<br class="">
<br class="">
Same here.<br class="">
<br class="">
setupButton(_timeSlider,<br class="">
_NS("Position"),<br class="">
- _NS("Adjust the current playback position"));<br class="">
+ _NS("Position"));<br class="">
<br class="">
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 class=""></blockquote><div class="">I could not understand properly. The title is what shows up in the toolTip. But is there a place where the description shows up? I kept it just in case it might be describing the button in some other place.</div><div class="">If not, I think we should remove it. </div></div></div></div></blockquote><div><br class=""></div>Your patch modifies the attribute NSAccessibilityDescriptionAttribute, mainly. Would be great if you can check what this is and where its used in detail, and if you can find any guidelines how it should be filled. (IIRC is has something to do if you use accessibility options like voice control or similar, but I do not remember details out of my head.)</div><div><br class=""></div><div>Just from the naming of both attributes, I find it weird to have a „title“ and a „description“ with the same content.</div><div><br class=""><blockquote type="cite" class=""><div style="margin: 0px; font-stretch: normal; font-size: 11px; line-height: normal; font-family: Menlo; color: rgb(120, 73, 42);" class=""><br class=""></div></blockquote><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
BR. David<br class="">
<br class="">
> Am 19.03.2018 um 12:02 schrieb Daksh Shah <<a href="mailto:daksh17336@iiitd.ac.in" target="_blank" class="">daksh17336@iiitd.ac.in</a>>:<br class="">
><br class="">
> 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.<br class="">
> And also, changed the Normal View toolTip of "Fullscreen" to "Enter fullscreen"<br class="">
><br class="">
> <0001-Fixed-19993-Made-toolTips-on-Full-Screen-consistent.patch>_______________________________________________<br class="">
> vlc-devel mailing list<br class="">
> To unsubscribe or modify your subscription options:<br class="">
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank" class="">https://mailman.videolan.org/listinfo/vlc-devel</a><br class="">
<br class="">
_______________________________________________<br class="">
vlc-devel mailing list<br class="">
To unsubscribe or modify your subscription options:<br class="">
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank" class="">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div></div>
_______________________________________________<br class="">vlc-devel mailing list<br class="">To unsubscribe or modify your subscription options:<br class=""><a href="https://mailman.videolan.org/listinfo/vlc-devel" class="">https://mailman.videolan.org/listinfo/vlc-devel</a></div></blockquote></div><br class=""></div></body></html>