Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-07 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/#review24882 --- i didn't see this review before i went in and just fixed it. th

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/#review24858 --- Ship it! I would leave the label, maybe remove the parenthesis

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-05 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/ --- (Updated Jan. 5, 2013, 4:18 p.m.) Review request for Plasma. Description

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-05 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/ --- (Updated Jan. 5, 2013, 4:17 p.m.) Review request for Plasma. Changes ---

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/#review24634 --- plasmate/savesystem/commitdialog.cpp

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-03 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/ --- (Updated Jan. 3, 2013, 8:13 p.m.) Review request for Plasma. Changes ---

Re: Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/#review24574 --- And why would you do that? Why are coordinates set statically?

Review Request: Plasmate: Replace QDialogButtonBox with QVBoxLayout

2013-01-03 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108115/ --- Review request for Plasma. Description --- Hello, In this patch i am