Re: Review Request 108570: This patch add support for bulk operations in systemtray applet settings.

2013-01-30 Thread Sandro Andrade

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

(Updated Jan. 30, 2013, 10:48 a.m.)


Review request for kde-workspace.


Changes
---

Alternative layout for bulk operations with no use of a QTableWidget for bulk 
operations and removal of bulk shortcut clean button.


Description
---

In page 'entries', user can select/deselect a specific or all entries and then 
change visibility and/or reset shortcut for all of them with a single combo 
selection/clear button click. Individually selecting all entries causes 
header's checkbox to automatically be checked. After selecting all entries 
(individually or by clicking in header's checkbox), a single entry deselection 
causes header's checkbox to automatically be unchecked. General suggestions 
and, in particular about UI usability, are appreciated.


Diffs (updated)
-

  plasma/generic/applets/systemtray/ui/applet.h 0b4a869 
  plasma/generic/applets/systemtray/ui/applet.cpp 09482d7 
  plasma/generic/applets/systemtray/ui/autohide.ui 3b6efff 

Diff: http://git.reviewboard.kde.org/r/108570/diff/


Testing
---


File Attachments


Initial setup
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions1.png
Setting all visibilities to 'Auto'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions2.png
All visibilities adjusted to 'Auto'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions3.png
Setting two specific item visibilities to 'Always Visible'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions4.png


Thanks,

Sandro Andrade



Re: Review Request 108570: This patch add support for bulk operations in systemtray applet settings.

2013-01-30 Thread Sandro Andrade

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

(Updated Jan. 30, 2013, 10:50 a.m.)


Review request for kde-workspace.


Changes
---

Adding screenshots for alternative bulk layout


Description
---

In page 'entries', user can select/deselect a specific or all entries and then 
change visibility and/or reset shortcut for all of them with a single combo 
selection/clear button click. Individually selecting all entries causes 
header's checkbox to automatically be checked. After selecting all entries 
(individually or by clicking in header's checkbox), a single entry deselection 
causes header's checkbox to automatically be unchecked. General suggestions 
and, in particular about UI usability, are appreciated.


Diffs
-

  plasma/generic/applets/systemtray/ui/applet.h 0b4a869 
  plasma/generic/applets/systemtray/ui/applet.cpp 09482d7 
  plasma/generic/applets/systemtray/ui/autohide.ui 3b6efff 

Diff: http://git.reviewboard.kde.org/r/108570/diff/


Testing
---


File Attachments (updated)


Initial setup
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions1.png
Setting all visibilities to 'Auto'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions2.png
All visibilities adjusted to 'Auto'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions3.png
Setting two specific item visibilities to 'Always Visible'
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions4.png
Bulk operations (2nd layout) - 1
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/30/systray-bulk-tableless1.png
Bulk operations (2nd layout) - 2
  
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/30/systray-bulk-tableless2.png


Thanks,

Sandro Andrade



Review Request 108675: kjs: Implement Function.prototype.bind

2013-01-30 Thread Bernd Buschinski

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

Review request for kdelibs.


Description
---

kjs: Implement Function.prototype.bind
following ECMAScript Edition 5.1r6 15.3.4.5

The only tiny difference is that the thisArg can not be null or undefined.
As we call use null or undefined as thisObject in call.
So, like in Function.prototype.call, I use the global object instead.

NOTE: this patch also depends on Object.defineOwnProperty


Diffs
-

  kjs/function.h 5dbbfb2 
  kjs/function.cpp 6263b36 
  kjs/function_object.h ebcb937 
  kjs/function_object.cpp 76f55cb 

Diff: http://git.reviewboard.kde.org/r/108675/diff/


Testing
---

passes all Function.prototype.bind ecmascript testcases and does not introduce 
regressions.


Thanks,

Bernd Buschinski



Re: KDEREVIEW: share like connect and plasmate

2013-01-30 Thread Ben Cooksley
On Wed, Jan 30, 2013 at 11:23 PM, Giorgos Tsiapaliokas
terie...@gmail.com wrote:
 Hello,

Hi Giorgos,


 as it has been mentioned plasmate is ready to go into the extragear.
 Can you move it to the extragear?

Where precisely in Extragear is Plasmate and Share-Like-Connect headed?
They are both overdue to be moved from KDE Review.

Note: Moves into and out of KDE Review must *always* be CC'ed to kde-core-devel.

On that note: any last objections?


 Regards,
 Giorgos

Regards,
Ben Cooksley
KDE Sysadmin



 --
 Giorgos Tsiapaliokas (terietor)
 KDE Developer

 terietor.gr

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



Re: KDEREVIEW: share like connect and plasmate

2013-01-30 Thread Aaron J. Seigo
On Thursday, January 31, 2013 10:43:29 Ben Cooksley wrote:
  as it has been mentioned plasmate is ready to go into the extragear.
  Can you move it to the extragear?
 
 Where precisely in Extragear is Plasmate

SDK

 and Share-Like-Connect headed?

Base

thanks :)

-- 
Aaron J. Seigo

signature.asc
Description: This is a digitally signed message part.