Re: Review Request 114328: re-add customstyleelement suite to kstyle

2013-12-08 Thread Commit Hook

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


This review has been submitted with commit 
278f2ed5630e21bba28c3433579405d372b7c89b by Hugo Pereira Da Costa to branch 
frameworks.

- Commit Hook


On Dec. 6, 2013, 2:43 p.m., Hugo Pereira Da Costa wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/114328/
 ---
 
 (Updated Dec. 6, 2013, 2:43 p.m.)
 
 
 Review request for KDE Frameworks.
 
 
 Repository: kdelibs
 
 
 Description
 ---
 
 re-add customstyleelement suite to kstyle
 
 
 Diffs
 -
 
   tier4/frameworkintegration/src/kstyle/kstyle.h 8a881f5 
   tier4/frameworkintegration/src/kstyle/kstyle.cpp 27d407e 
 
 Diff: http://git.reviewboard.kde.org/r/114328/diff/
 
 
 Testing
 ---
 
 compiles, works, fix kde-workspace build
 also: will be used when moving oxygen from qcommonstyle back to kstyle (right 
 now we have a fork of some of the said methods)
 
 
 Thanks,
 
 Hugo Pereira Da Costa
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 114328: re-add customstyleelement suite to kstyle

2013-12-08 Thread Hugo Pereira Da Costa

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

(Updated Dec. 8, 2013, 10:24 a.m.)


Status
--

This change has been marked as submitted.


Review request for KDE Frameworks.


Repository: kdelibs


Description
---

re-add customstyleelement suite to kstyle


Diffs
-

  tier4/frameworkintegration/src/kstyle/kstyle.h 8a881f5 
  tier4/frameworkintegration/src/kstyle/kstyle.cpp 27d407e 

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


Testing
---

compiles, works, fix kde-workspace build
also: will be used when moving oxygen from qcommonstyle back to kstyle (right 
now we have a fork of some of the said methods)


Thanks,

Hugo Pereira Da Costa

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 114328: re-add customstyleelement suite to kstyle

2013-12-07 Thread Àlex Fiestas

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

Ship it!


Code looks good, you are the master of QStyle if you think those methods are 
useful, please ship it!

- Àlex Fiestas


On Dec. 6, 2013, 2:43 p.m., Hugo Pereira Da Costa wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/114328/
 ---
 
 (Updated Dec. 6, 2013, 2:43 p.m.)
 
 
 Review request for KDE Frameworks.
 
 
 Repository: kdelibs
 
 
 Description
 ---
 
 re-add customstyleelement suite to kstyle
 
 
 Diffs
 -
 
   tier4/frameworkintegration/src/kstyle/kstyle.h 8a881f5 
   tier4/frameworkintegration/src/kstyle/kstyle.cpp 27d407e 
 
 Diff: http://git.reviewboard.kde.org/r/114328/diff/
 
 
 Testing
 ---
 
 compiles, works, fix kde-workspace build
 also: will be used when moving oxygen from qcommonstyle back to kstyle (right 
 now we have a fork of some of the said methods)
 
 
 Thanks,
 
 Hugo Pereira Da Costa
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 114328: re-add customstyleelement suite to kstyle

2013-12-06 Thread Hugo Pereira Da Costa

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

(Updated Dec. 6, 2013, 2:43 p.m.)


Review request for KDE Frameworks.


Changes
---

changed groups.


Repository: kdelibs


Description
---

re-add customstyleelement suite to kstyle


Diffs
-

  tier4/frameworkintegration/src/kstyle/kstyle.h 8a881f5 
  tier4/frameworkintegration/src/kstyle/kstyle.cpp 27d407e 

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


Testing
---

compiles, works, fix kde-workspace build
also: will be used when moving oxygen from qcommonstyle back to kstyle (right 
now we have a fork of some of the said methods)


Thanks,

Hugo Pereira Da Costa

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel