-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104762/#review13010
-----------------------------------------------------------
Hi Nancy,
I was just playing with time representing stuff using the javascript Date and
came up with this very short example that does the exact same :)
onCurrentTimeChanged: {
var dateTimeObject = new Date(0, 0, 0, 0, 0, currentTime, 0)
mediaController.curMediaTime = Qt.formatTime(dateTimeObject, "hh:mm:ss")
}
What i'm doing above is simply initializing the Date object with 0 values and
only fill in the seconds. In my test the date object just takes that. Then i'm
using Qt.formatTime to format the date object into a nice string. Here is the
documentation for both:
Date: http://www.w3schools.com/jsref/jsref_obj_date.asp
Qt.formatTime: http://qt-project.org/doc/qt-4.8/qml-qt.html#formatTime-method
(scroll up a little)
You can try and replace the current onCurrentTimeChanged with the example from
above, I haven't tested this on your QML file but it should just work.
I hope it works since it seems nicely cleaned up to me. This also makes it a
lot easier to change the displayed time format :)
Note: i'm not the author of the mediacenter stuff so someone else should decide
if this is ok.
- Mark Gaiser
On April 27, 2012, 5:28 p.m., nancy kumari wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104762/
> -----------------------------------------------------------
>
> (Updated April 27, 2012, 5:28 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Displaying playing media time in correct format. previously when value of
> hour, minute or second was less than 10, then it was displaying in single
> digit format(0:13:5). Now it displays in proper format(00:13:05).
>
>
> Diffs
> -----
>
> shells/newshell/package/contents/ui/mediacenter.qml 11423db
>
> Diff: http://git.reviewboard.kde.org/r/104762/diff/
>
>
> Testing
> -------
>
> Working fine.
>
>
> Thanks,
>
> nancy kumari
>
>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
