[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request 115870: The plasma-media center displays the time in AM between 12 noon and 1pm.
From:       "Mark Gaiser" <markg85 () gmail ! com>
Date:       2014-02-19 14:06:50
Message-ID: 20140219140650.12529.36079 () probe ! kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


> On Feb. 19, 2014, 11:36 a.m., Sebastian Kügler wrote:
> > 

Perhaps this would help: \
http://qt-project.org/doc/qt-4.8/qml-qt.html#formatTime-method

It seems to be providing the features needed to display a given time however you want \
it. In your case the result should probably be something like this: \
headerTimeText.text = Qt.formatTime(new Date(), "h:m AP")

I don't know if javascript's Data object works properly when passed in \
Qt.formatTime.. If it doesn't then you have to take the C++ route instead.


- Mark


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


On Feb. 19, 2014, 11:11 a.m., R.Harish  Navnit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115870/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2014, 11:11 a.m.)
> 
> 
> Review request for Plasma and Sinny Kumari.
> 
> 
> Bugs: 330115 and Bug
> http://bugs.kde.org/show_bug.cgi?id=330115
> http://bugs.kde.org/show_bug.cgi?id=Bug
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> I just happened to run the plasma-mediacenter in the noon (between 12-1) and to my \
> surprise I found the time being displayed as "0:11 AM" . But when I run it now , it \
> displays the time properly. 
> Reproducible: Didn't try
> 
> Steps to Reproduce:
> 1.Run the plasma-mediacenter 
> 2.Check the time being displayed 
> 3.The time displayed will most likely be wrong between 12 noon to 1 pm .
> Actual Results:  
> I just encountered this error once so far . I'll have to wait 24 hours to see if \
> the error persists . 
> Expected Results:  
> Should display the time as 12:11PM instead of 0:11AM.
> 
> The output of "lsb_release -a" : http://pastebin.kde.org/p4v5s1srq
> 
> The output of "uname -a" : http://pastebin.kde.org/pcncndtwb
> 
> 
> Diffs
> -----
> 
> mediaelements/mediawelcome/HomeScreenFooter.qml d2c0eb7 
> 
> Diff: https://git.reviewboard.kde.org/r/115870/diff/
> 
> 
> Testing
> -------
> 
> after creating the patch , the following steps were carried out
> 
> 1.cmake .. -DCMAKE_INSTALL_PREFIX=`kde4-config --prefix`
> 2.sudo make install
> 
> Change the clock to a time in between 12noon and 1pm and run the \
> plasma-mediacenter. 
> The time displays properly ! 
> 
> 
> Thanks,
> 
> R.Harish  Navnit
> 
> 


[Attachment #5 (text/html)]

<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;">  <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/115870/">https://git.reviewboard.kde.org/r/115870/</a>
  </td>
    </tr>
   </table>
   <br />











<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Perhaps this would help: \
http://qt-project.org/doc/qt-4.8/qml-qt.html#formatTime-method

It seems to be providing the features needed to display a given time however you want \
it. In your case the result should probably be something like this: \
headerTimeText.text = Qt.formatTime(new Date(), &quot;h:m AP&quot;)

I don&#39;t know if javascript&#39;s Data object works properly when passed in \
Qt.formatTime.. If it doesn&#39;t then you have to take the C++ route instead.</pre>
<br />










<p>- Mark</p>


<br />
<p>On February 19th, 2014, 11:11 a.m. UTC, R.Harish  Navnit wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;">  <tr>
  <td>

<div>Review request for Plasma and Sinny Kumari.</div>
<div>By R.Harish  Navnit.</div>


<p style="color: grey;"><i>Updated Feb. 19, 2014, 11:11 a.m.</i></p>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=330115">330115</a>, 

 <a href="http://bugs.kde.org/show_bug.cgi?id=Bug">Bug</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-mediacenter
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">I just happened to run the plasma-mediacenter in the noon (between 12-1) \
and to my surprise I found the time being displayed as &quot;0:11 AM&quot; . But when \
I run it now , it displays the time properly.

Reproducible: Didn&#39;t try

Steps to Reproduce:
1.Run the plasma-mediacenter 
2.Check the time being displayed 
3.The time displayed will most likely be wrong between 12 noon to 1 pm .
Actual Results:  
I just encountered this error once so far . I&#39;ll have to wait 24 hours to see if \
the error persists .

Expected Results:  
Should display the time as 12:11PM instead of 0:11AM.

The output of &quot;lsb_release -a&quot; : http://pastebin.kde.org/p4v5s1srq

The output of &quot;uname -a&quot; : http://pastebin.kde.org/pcncndtwb</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">after creating the patch , the following steps were carried out

1.cmake .. -DCMAKE_INSTALL_PREFIX=`kde4-config --prefix`
2.sudo make install

Change the clock to a time in between 12noon and 1pm and run the plasma-mediacenter.

The time displays properly ! </pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>mediaelements/mediawelcome/HomeScreenFooter.qml <span style="color: \
grey">(d2c0eb7)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/115870/diff/" style="margin-left: \
3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>



_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic