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

List:       kde-panel-devel
Subject:    Re: Review Request: Allow hiding of items in the Kate Sessions
From:       "Yuen Hoe Lim" <yuenhoe86 () gmail ! com>
Date:       2010-09-05 15:48:29
Message-ID: 20100905154829.13165.93875 () vidsolbach ! de
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/4264/
-----------------------------------------------------------

(Updated 2010-09-05 15:48:29.104994)


Review request for Kate and Plasma.


Changes
-------

Implemented Aaron's suggestions. Sorry I left this one behind for so long -=
 figured that there was still time before 4.6 so focused on finishing my GS=
oC first :)

New config UI screenshot included.


Summary
-------

Allows the user to specify which items to show or hide in the Kate Sessions=
 Plasmoid using a configuration interface. Two possible use-cases for this:

1) Allow users who just want quick access to their projects to remove the d=
efault three items (Start Kate, New Session etc), which can take up a lot o=
f space since the applet is usually quite small.

2) Allow multiple applets displaying different lists, for example having ap=
plets in different activities each displaying only sessions related to that=
 activity.

The Kate Sessions Plasmoid currently does not have a configuration interfac=
e so I thought this is probably no harm :) I have attached a screenshot of =
the configuration UI. I'm somewhat inexperienced with QtDesigner so feedbac=
k on how I could improve the patch code or the config UI are most welcome :)


Diffs (updated)
-----

  /trunk/KDE/kdesdk/kate/plasma/session/katesessionConfig.ui PRE-CREATION =

  /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.h 1171815 =

  /trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.cpp 1171815 =


Diff: http://svn.reviewboard.kde.org/r/4264/diff


Testing
-------

Tested on trunk on my computer. Works as far as I can tell.


Screenshots
-----------

Config UI Screenshot
  http://svn.reviewboard.kde.org/r/4264/s/428/
Updated config UI screenshot
  http://svn.reviewboard.kde.org/r/4264/s/497/


Thanks,

Yuen Hoe


[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="http://svn.reviewboard.kde.org/r/4264/">http://svn.reviewboard.kde.org/r/4264/</a>
  </td>
    </tr>
   </table>
   <br />


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

<div>Review request for Kate and Plasma.</div>
<div>By Yuen Hoe Lim.</div>


<p style="color: grey;"><i>Updated 2010-09-05 15:48:29.104994</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Implemented Aaron&#39;s suggestions. Sorry I left this one behind for so \
long - figured that there was still time before 4.6 so focused on finishing my GSoC \
first :)

New config UI screenshot included.</pre>
  </td>
 </tr>
</table>


<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;">Allows the user to specify which items to show or hide in the Kate \
Sessions Plasmoid using a configuration interface. Two possible use-cases for this:

1) Allow users who just want quick access to their projects to remove the default \
three items (Start Kate, New Session etc), which can take up a lot of space since the \
applet is usually quite small.

2) Allow multiple applets displaying different lists, for example having applets in \
different activities each displaying only sessions related to that activity.

The Kate Sessions Plasmoid currently does not have a configuration interface so I \
thought this is probably no harm :) I have attached a screenshot of the configuration \
UI. I&#39;m somewhat inexperienced with QtDesigner so feedback on how I could improve \
the patch code or the config UI are most welcome :)</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;">Tested on trunk on my computer. Works as far as I can tell.</pre>  </td>
 </tr>
</table>




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

 <li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionConfig.ui <span style="color: \
grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.h <span style="color: \
grey">(1171815)</span></li>

 <li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.cpp <span style="color: \
grey">(1171815)</span></li>

</ul>

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



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

<div>

 <a href="http://svn.reviewboard.kde.org/r/4264/s/428/"><img \
src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/06/09/katesessionconfig_400x100.jpeg" \
style="border: 1px black solid;" alt="Config UI Screenshot" /></a>

 <a href="http://svn.reviewboard.kde.org/r/4264/s/497/"><img \
src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/09/05/katesessionfixedscreeny_400x100.jpeg" \
style="border: 1px black solid;" alt="Updated config UI screenshot" /></a>

</div>


  </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