Review Request 115124: Fix wrong computation of volume % value in muted state in tooltip

[Burkhard Lück]

> On Jan. 20, 2014, 8:25 p.m., Michael Pyne wrote:
> > volumepopupbutton.cpp, line 129
> > <https://git.reviewboard.kde.org/r/115124/diff/1/?file=234691#file234691line129>
> >
> >     The change looks good but it might be prudent to remove the "muted ? i18n(...." portion at the end and just replace with 'i18n("(muted)")', because in this portion of the code we know that 'muted' must be true.
> >     
> >     It could even be lumped into the source string but since that would involve breaking i18n string freeze that would have to be up to the translation teams.

No need to stress translators for this issue in 4.12, so I'll use
setToolTip( i18n( "Volume: %1% %2", int( 100 * volume ), i18n( "(muted)" ) ) ); in stable
and
setToolTip( i18n( "Volume: %1% (muted)", int( 100 * volume ) ) );
in master

Thanks for review


- Burkhard


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115124/#review47839
-----------------------------------------------------------


On Jan. 19, 2014, 3:13 p.m., Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115124/
> -----------------------------------------------------------
> 
> (Updated Jan. 19, 2014, 3:13 p.m.)
> 
> 
> Review request for KDE Multimedia and Michael Pyne.
> 
> 
> Repository: juk
> 
> 
> Description
> -------
> 
> Hovering the mouse pointer over the volume icon at the right end of the toolbar
> two different tooltips are displayed depending on the mute state:
> State unmuted: Volume 81%
> State muted:   Volume 0,81% (muted) -> wrong computation of % value
> 
> 
> Diffs
> -----
> 
>   volumepopupbutton.cpp 5ac5e93 
> 
> Diff: https://git.reviewboard.kde.org/r/115124/diff/
> 
> 
> Testing
> -------
> 
> With patch the tooltip displays in muted state "Volume 81% (muted)" with the correct % value.
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20140124/5d1e6f29/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia