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

List:       kde-panel-devel
Subject:    Re: [KDE] New public review on Improvements and enhancements for Plasma Media Center (Akshay Ratan)
From:       Akshay Ratan <akshayratan () gmail ! com>
Date:       2013-04-27 17:48:23
Message-ID: CAKTp-8T1UvJm+h_swJO4YTZ5nnUC28CcbJ4__DsRSMXe0qHx=Q () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


> Hi,

>   Shantanu Tushar has left the following public review at
> http://www.google-melange.com/gsoc/proposal/review/google/gsoc2013/akshay_ratan/1#c5001:
>
>
> Good proposal overall, here are few things to fix-
>

Thanks a lot for the review ! :)

>
>    - "Add the feature of subtitles in the video player of PMC" is
>    misleading, better "Improve the subtitles feature in the video player of
>    PMC"
>
>
+ 1 ! Thanks for pointing this out. Yes, indeed it was seemingly
misleading. Improve is the better term.

>
>    - In "Improving playlist search", I get a feeling you want to improve
>    search overall, not just playlist. If its true, make the title not specific
>    to playlist.
>
>
+1

>
>    - The link and phrase "Plugin support example:https .." isn't clear.
>    What are you trying to show?
>
>
What I was trying to show is the existing backend support for Picasa in the
PMC. Polishing is needed in this as well as new dropbox extension would be
on similar lines using Accounts and SSO Framework.

>
>    - For Keyboard navigation, note that most TV remotes won't have both
>    Escape and Backspace keys. Most of them will have a "back" button that maps
>    to Escape. So try using only Escape.
>
>
Okay. Thank you for bringing this point. It might need a discussion before
finalizing. What I thought ( also in discussion with Sinny) was to make
each and every component of PMC keyboard-usable. For this, the best option
I thought was of using multiple keyboard keys which are common in other
applications. For instance, backspace key for going one level up. I didnt
really think in terms of TV Remotes. One naive doubt- from where can I find
the API to map TV Remote keys in Qt ? Does Qt::Key contains a key mapping
for remote ? (http://qt-project.org/doc/qt-4.8/qt.html#Key-enum ) . I dont
have experience for tv remotes or other related mapping in Qt. Nevertheless
I am ready to study it if PMC demands it. Please guide me further on this
if possible.

>
>    - In Keyboard navigation, you have screenshots of existing not-so-good
>    behavior that we want to improve. We will need mockups  which will show the
>    solution/new workflow as well
>
>
In the second link of Mock test case in my idea of Keyboard navigation
plan, I have shown the solution in the form of images. Even in first , PMC
divisions have been shown in the image to clarify the divisions in which
TAB needs to be implemented and flow made. Which kind of further solution
mockups are needed ? I l try and add more workflows soon. Just 2 more exams
left ! Kindly tell me something more so that I can improve on this part.

Reference images-

*

https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Images/PMC_1_tab.png


https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Images/Escape%20Key%20effect%20image.jpg
*

>
>    - Clarification about documentation, the media center is supposed to
>    have an intuitive UI so that it will never need a user's manual or
>    something. However, you must keep in mind to keep your code documented
>    always.
>
> Yes, sure. Ideally there should not be any need for users manual or
something. What I really meant was to keep my code documented, and also
provide a certain level of documentation for new users on Computers and TV
if they wish to have one. Nevertheless, I will keep blogging about it
anyways. What else can be done on this > Any suggestions ?

>
>    - IMO you should swap the first two tasks of coding phase 1 and 2
>    because the ones in 2 are easier to get started with. This will help you
>    gain confidence if you complete these things earlier during the project.
>
>
I thought the same about the difficulty levels of the two phases . But my
intention was that since there are college holidays till end of July, I l
be in a better position to tackle problems and do intensive work in summer
holidays much more. Then after college reopens, I can work on a relatively
easier feature. I thought this initially though. Do you still recommend
switching the coding phases ?

>
>    - Another note, while you will develop separate features in branches.
>    You should merge a feature branch to master once that feature is complete
>    and reviewed. Do *NOT* wait for the end of the GSoC program to merge - its
>    very, very difficult for both the student and other devs who will review.
>
>
Okay, this thing was new to me :) Thanks ! Yes I will merge the individual
features on the way itself and not wait till the end.

Thanks a lot once again !

Looking forward for a valuable response :)

Cheers
*Akshay Ratan*
(akshay_r #plasma #kde-in )

[Attachment #5 (text/html)]

<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div></div></blockquote>Hi,<br><div \
class="gmail_quote"> <blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>
  <p>
	
</p><p>
Shantanu Tushar has left the following public review at
<a href="http://www.google-melange.com/gsoc/proposal/review/google/gsoc2013/akshay_ratan/1#c5001" \
target="_blank">http://www.google-melange.com/gsoc/proposal/review/google/gsoc2013/akshay_ratan/1#c5001</a>:
 </p>
<div>
<p>
</p><p>Good proposal overall, here are few things to \
fix-</p></div></div></blockquote><div><br></div><div>Thanks a lot for the review ! :) \
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> <div><div>
<ul>
<li>&quot;Add the feature of subtitles in the video player of PMC&quot; is \
misleading, better &quot;Improve the subtitles feature in the video player of \
PMC&quot;</li></ul></div></div></blockquote><div><br></div><div>+ 1 ! Thanks for \
pointing this out. Yes, indeed it was seemingly misleading. Improve is the better \
term.</div> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><div><div><ul> <li>In &quot;Improving playlist \
search&quot;, I get a feeling you want to improve search overall, not just playlist. \
If its true, make the title not specific to \
playlist.</li></ul></div></div></blockquote><div><br></div> <div>+1  \
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div><div><ul> <li>The link and phrase &quot;Plugin support \
example:https ..&quot; isn&#39;t clear. What are you trying to \
show?</li></ul></div></div></blockquote><div><br></div><div>What I was trying to show \
is the existing backend support for Picasa in the PMC. Polishing is needed in this as \
well as new dropbox extension would be on similar lines using Accounts and SSO \
Framework. </div> <blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><ul> <li>For Keyboard \
navigation, note that most TV remotes won&#39;t have both Escape and Backspace keys. \
Most of them will have a &quot;back&quot; button that maps to Escape. So try using \
only Escape.</li></ul></div></div></blockquote> <div><br></div><div>Okay. Thank you \
for bringing this point. It might need a discussion before finalizing. What I thought \
( also in discussion with Sinny) was to make each and every component of PMC \
keyboard-usable. For this, the best option I thought was of using multiple keyboard \
keys which are common in other applications. For instance, backspace key for going \
one level up. I didnt really think in terms of TV Remotes. One naive doubt- from \
where can I find the API to map TV Remote keys in Qt ? Does Qt::Key contains a key \
mapping for remote ? (<a \
href="http://qt-project.org/doc/qt-4.8/qt.html#Key-enum">http://qt-project.org/doc/qt-4.8/qt.html#Key-enum</a> \
) . I dont have experience for tv remotes or other related mapping in Qt. \
Nevertheless I am ready to study it if PMC demands it. Please guide me further on \
this if possible.</div> <blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><ul> <li>In Keyboard \
navigation, you have screenshots of existing not-so-good behavior that we want to \
improve. We will need mockups  which will show the solution/new workflow as \
well</li></ul></div></div></blockquote><div><br> </div><div>In the second link of \
Mock test case in my idea of Keyboard navigation plan, I have shown the solution in \
the form of images. Even in first , PMC divisions have been shown in the image to \
clarify the divisions in which TAB needs to be implemented and flow made. Which kind \
of further solution mockups are needed ? I l try and add more workflows soon. Just 2 \
more exams left ! Kindly tell me something more so that I can improve on this part. \
</div> <div><br></div><div>Reference images-</div><div><br></div><div><em \
style="margin:0px;padding:0px;border:0px;outline:0px;font-size:13px;vertical-align:bas \
eline;background-color:rgb(246,246,246);font-style:normal;font-family:Arial,&#39;Helvetica \
Neue&#39;,Helvetica,sans-serif;line-height:19.5px"><em \
style="margin:0px;padding:0px;border:0px;outline:0px;vertical-align:baseline;background-color:transparent;font-style:normal"><p \
dir="ltr" style="margin:0pt 0px 0pt \
36pt;padding:0px;border:0px;outline:0px;font-size:12px;vertical-align:baseline;background-color:transparent;line-height:1.15;background-repeat:initial \
initial"> <span style="margin:0px;padding:0px;border:0px;outline:0px;vertical-align:ba \
seline;background-color:transparent;color:rgb(17,85,204);text-decoration:none;font-size:13px;font-family:Arial;font-weight:bold;white-space:pre-wrap"><a \
href="https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Images/PMC_1_tab.png" \
style="margin:0px;padding:0px;border:0px;outline:0px;vertical-align:baseline;backgroun \
d-color:transparent;color:rgb(0,83,139);text-decoration:none">https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Images/PMC_1_tab.png</a></span></p>
 <p dir="ltr" style="margin:0pt 0px 0pt \
36pt;padding:0px;border:0px;outline:0px;font-size:12px;vertical-align:baseline;background-color:transparent;line-height:1.15;background-repeat:initial \
initial"><br></p><p dir="ltr" style="margin:0pt 0px 0pt \
36pt;padding:0px;border:0px;outline:0px;font-size:12px;vertical-align:baseline;background-color:transparent;line-height:1.15;background-repeat:initial \
initial"> <a href="https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Ima \
ges/Escape%20Key%20effect%20image.jpg">https://dl.dropboxusercontent.com/u/107169565/GSOC%20Proposal%20Images/Escape%20Key%20effect%20image.jpg</a></p>
 </em></em></div><blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><ul> <li>Clarification \
about documentation, the media center is supposed to have an intuitive UI so that it \
will never need a user&#39;s manual or something. However, you must keep in mind to \
keep your code documented always.</li> </ul></div></div></blockquote><div>Yes, sure. \
Ideally there should not be any need for users manual or something. What I really \
meant was to keep my code documented, and also provide a certain level of \
documentation for new users on Computers and TV if they wish to have one. \
Nevertheless, I will keep blogging about it anyways. What else can be done on this \
&gt; Any suggestions ?</div> <blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><ul> <li>IMO you should \
swap the first two tasks of coding phase 1 and 2 because the ones in 2 are easier to \
get started with. This will help you gain confidence if you complete these things \
earlier during the project.</li></ul> </div></div></blockquote><div><br></div><div>I \
thought the same about the difficulty levels of the two phases . But my intention was \
that since there are college holidays till end of July, I l be in a better position \
to tackle problems and do intensive work in summer holidays much more. Then after \
college reopens, I can work on a relatively easier feature. I thought this initially \
though. Do you still recommend switching the coding phases ? </div> <blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div><div><ul> <li>Another note, while you will develop \
separate features in branches. You should merge a feature branch to master once that \
feature is complete and reviewed. Do *NOT* wait for the end of the GSoC program to \
merge - its very, very difficult for both the student and other devs who will \
review.</li> </ul></div></div></blockquote><div><br></div><div>Okay, this thing was \
new to me :) Thanks ! Yes I will merge the individual features on the way itself and \
not wait till the end. </div><div><br></div><div>Thanks a lot once again ! </div> \
</div><div><br></div>Looking forward for a valuable response :)<br \
clear="all"><div><br></div>Cheers<br><b>Akshay Ratan</b><div>(akshay_r #plasma \
#kde-in )</div>



_______________________________________________
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