Re: Review Request 120634: Added possibility of painting kexi report rect elements in all directions

2014-11-01 Thread Jarosław Staniek
st int KOREPORT_ITEM_RECT_DEFAULT_HEIGHT = 100; in KoReportDesignerItemRectBase.h or so? - Jarosław Staniek On Nov. 1, 2014, 6:10 p.m., Wojciech Kosowicz wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 120634: Added possibility of painting kexi report rect elements in all directions

2014-11-01 Thread Jarosław Staniek
<https://git.reviewboard.kde.org/r/120634/#comment48808> Shouldn't be tempReleasePointY and y() in next line? I't clear when tested with non-square geometries (e.g. barcode). - Jarosław Staniek On Nov. 1, 2014, 6:10 p.m., Wojciec

Review Request 120964: Fix regression: build report chart plugin again

2014-11-03 Thread Jarosław Staniek
elements work Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 120964: Fix regression: build report chart plugin again

2014-11-03 Thread Jarosław Staniek
5d80a023abc3fd9d4c7bc76699b711be0ffbfcc4 Diff: https://git.reviewboard.kde.org/r/120964/diff/ Testing --- plugin loads and elements work Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Review Request 120966: Improve UX of Report Designer

2014-11-03 Thread Jarosław Staniek
-7340fc863e9f__title-focus.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 120940: Added parsing of hashtag comments inside imported csv in kexi

2014-11-04 Thread Jarosław Staniek
i/plugins/importexport/csv/kexicsvimportdialog.cpp <https://git.reviewboard.kde.org/r/120940/#comment48880> S_COMMENT makes sense here and in 1161 too, right? think what state should be set here kexi/plugins/importexport/csv/kexicsvimportdialog.cpp <https://git.reviewboard.kde.o

Re: Review Request 119222: Make labels inline-editable

2014-11-04 Thread Jarosław Staniek
Button.cpp <https://git.reviewboard.kde.org/r/119222/#comment48889> Let's use QPixmap::size() instead of hardcoding - Jarosław Staniek On Nov. 3, 2014, 8:49 p.m., Adam Pigg wrote: > > --- > This is an automatically gener

Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-09 Thread Jarosław Staniek
kexi/main/KexiMenuWidget_p.h 2463b9b30c4f7c55cec061da5f3602af4d78df79 Diff: https://git.reviewboard.kde.org/r/121088/diff/ Testing --- Logo well displayed regardless of the menu height. Thanks, Jarosław Staniek ___ calligra-devel mailing

Re: Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-10 Thread Jarosław Staniek
/KexiMenuWidget_p.h 2463b9b Diff: https://git.reviewboard.kde.org/r/121088/diff/ Testing --- Logo well displayed regardless of the menu height. Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org

Re: Review Request 120966: Improve UX of Report Designer

2014-11-10 Thread Jarosław Staniek
https://git.reviewboard.kde.org/media/uploaded/files/2014/11/04/3c2e503c-50ad-47b3-a751-7340fc863e9f__title-focus.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 119222: Make labels inline-editable

2014-11-13 Thread Jarosław Staniek
ław Staniek On Nov. 12, 2014, 11:51 p.m., Adam Pigg wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119222/ > --- > > (Updated Nov. 12, 2014, 11:51 p.m.) > > > Rev

Re: Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-14 Thread Jarosław Staniek
kexi/main/KexiMenuWidget.cpp 6acd27e kexi/main/KexiMenuWidget_p.h 2463b9b Diff: https://git.reviewboard.kde.org/r/121088/diff/ Testing --- Logo well displayed regardless of the menu height. Thanks, Jarosław Staniek ___ calligra-devel mailing

Re: Review Request 119222: Make labels inline-editable

2014-11-14 Thread Jarosław Staniek
> On July 11, 2014, 12:25 a.m., Jarosław Staniek wrote: > > libs/koreport/items/label/KoReportDesignerItemLabel.cpp, line 189 > > <https://git.reviewboard.kde.org/r/119222/diff/1/?file=289614#file289614line189> > > > > can be called once, move t

Re: Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-16 Thread Jarosław Staniek
- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121088/#review70401 --- On Nov. 16, 2014, 10:34 p.m., Jarosław Staniek wrote: > >

Re: Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-16 Thread Jarosław Staniek
main menu Diffs (updated) - kexi/main/KexiMenuWidget.h f5b418e kexi/main/KexiMenuWidget.cpp 6acd27e kexi/main/KexiMenuWidget_p.h 2463b9b Diff: https://git.reviewboard.kde.org/r/121088/diff/ Testing --- Logo well displayed regardless of the menu height. Thanks, Jarosław

Review Request 121156: Simplify "Save password" checkbox text but add direct what's this button

2014-11-17 Thread Jarosław Staniek
-- The dialog with the What's This box https://git.reviewboard.kde.org/media/uploaded/files/2014/11/17/e666191e-2f5c-4d2b-ae45-45b5fc34c4d7__savepwd.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org

Re: Review Request 119222: Make labels inline-editable

2014-11-17 Thread Jarosław Staniek
Not fixed DEFECT19 Exiting the inline editing mode (actually only possible by clicking the LabelEditButton again) should keep the label element selected. Currently the parent section gets selected (visible in property editor). Fixed! New: DEFECT20: Escape key should undo editing, now it a

Re: Review Request 119222: Make labels inline-editable

2014-11-17 Thread Jarosław Staniek
> On Nov. 13, 2014, 11:36 a.m., Jarosław Staniek wrote: > > libs/koreport/items/label/BoundedTextItem.cpp, line 43 > > <https://git.reviewboard.kde.org/r/119222/diff/6-7/?file=325068#file325068line43> > > > > Why QPalette::Background? It's the same a

Re: Review Request 119222: Make labels inline-editable

2014-11-17 Thread Jarosław Staniek
perty editor). > > > > Fixed! > > > > > > New: DEFECT20: Escape key should undo editing, now it accepts (nonintuitive > > behaviour) > > > > New: DEFECT21: Property editor's Caption property is not updated in real > > time while I

Re: Review Request 121088: Fix position of the Calligra logo based on height of the main menu

2014-11-18 Thread Jarosław Staniek
Diffs - kexi/main/KexiMenuWidget.h f5b418e kexi/main/KexiMenuWidget.cpp 6acd27e kexi/main/KexiMenuWidget_p.h 2463b9b Diff: https://git.reviewboard.kde.org/r/121088/diff/ Testing --- Logo well displayed regardless of the menu height. Thanks, Jarosław Staniek

Review Request 121174: Fix checking for file overwriting on file creation

2014-11-18 Thread Jarosław Staniek
04456990c1404 Diff: https://git.reviewboard.kde.org/r/121174/diff/ Testing --- Creation works as before Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121174: Fix checking for file overwriting on file creation

2014-11-18 Thread Jarosław Staniek
) - kexi/main/startup/KexiStartupFileHandler.cpp ff483e7 kexi/main/startup/KexiStartupFileHandler.h d203bb4 kexi/main/startup/KexiNewProjectAssistant.cpp 633d919 Diff: https://git.reviewboard.kde.org/r/121174/diff/ Testing --- Creation works as before Thank

Re: Review Request 121174: Fix checking for file overwriting on file creation

2014-11-18 Thread Jarosław Staniek
kexi/main/startup/KexiNewProjectAssistant.cpp 633d919 Diff: https://git.reviewboard.kde.org/r/121174/diff/ Testing --- Creation works as before Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.

Re: Review Request 121174: Fix checking for file overwriting on file creation

2014-11-20 Thread Jarosław Staniek
ff483e7 kexi/main/startup/KexiStartupFileHandler.h d203bb4 kexi/main/startup/KexiNewProjectAssistant.cpp 633d919 Diff: https://git.reviewboard.kde.org/r/121174/diff/ Testing --- Creation works as before Thanks, Jarosław Staniek ___ calligr

Review Request 121237: Improve appearance of main tabbed toolbar for GTK+ and Oxygen styles

2014-11-24 Thread Jarosław Staniek
2d7099d430685e250ef8c42f2bd12825f65bacab Diff: https://git.reviewboard.kde.org/r/121237/diff/ Testing --- Run with various Qt styles, looks better Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org

Re: Review Request 120910: Action selection dialog: fix i18n argument, buddys, remove tags

2014-11-24 Thread Jarosław Staniek
reply, visit: https://git.reviewboard.kde.org/r/120910/#review70875 ------- On Oct. 31, 2014, 12:25 a.m., Jarosław Staniek wrote: > > --- > This is an automatically

Re: Review Request 120910: Action selection dialog: fix i18n argument, buddys, remove tags

2014-11-24 Thread Jarosław Staniek
/kexiactionselectiondialog.cpp 1527152252fa3a344f5197ff03fd9b0cf67b9af3 kexi/formeditor/kexiactionselectiondialog_p.h 161dcb00785f92d871fafdd7b97d709904d515ad Diff: https://git.reviewboard.kde.org/r/120910/diff/ Testing --- looks better Thanks, Jarosław Staniek

Re: Review Request 121156: Simplify "Save password" checkbox text but add direct what's this button

2014-11-24 Thread Jarosław Staniek
same > > > File Attachments > > > The dialog with the What's This box > > https://git.reviewboard.kde.org/media/uploaded/files/2014/11/17/e666191e-2f5c-4d2b-ae45-45b5fc34c4d7__savepwd.png > > > Thanks, > > Jarosław Staniek > > ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121156: Simplify "Save password" checkbox text but add direct what's this button

2014-11-24 Thread Jarosław Staniek
g/r/121156/diff/ Testing --- Dialog improved, overall features the same File Attachments The dialog with the What's This box https://git.reviewboard.kde.org/media/uploaded/files/2014/11/17/e666191e-2f5c-4d2b-ae45-45b5fc34c4d7__savepwd.png Thank

Re: Review Request 121180: Kexi - msghandler.cpp. Wrong function called.

2014-11-25 Thread Jarosław Staniek
r you. - Jarosław Staniek On Nov. 20, 2014, 12:06 a.m., Stephen Leibowitz wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard

Re: Review Request 119168: cleanup: COMPILING_TESTS is inherited in subdirs so it needs to be set once

2014-11-27 Thread Jarosław Staniek
--- tested by building with tests Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 119222: Make labels inline-editable

2014-11-27 Thread Jarosław Staniek
ttps://git.reviewboard.kde.org/r/119222/#comment49597> Better, now we're catching clicks on the section's surface. But what if we clicked on another item, ie. itemAt(e->scenePos()) != 0? Then the previous editing isn't cancelled: http://i.imgur.com/ZdyVaV1.pn

Re: Review Request 121237: Improve appearance of main tabbed toolbar for GTK+ and Oxygen styles

2014-11-28 Thread Jarosław Staniek
can assume appearance improvements fixe bug 341150 too Diffs - kexi/main/KexiMainWindow_p.h 2d7099d430685e250ef8c42f2bd12825f65bacab Diff: https://git.reviewboard.kde.org/r/121237/diff/ Testing --- Run with various Qt styles, looks better Thanks, Jarosław Staniek

Re: Review Request 119222: Make labels inline-editable

2014-11-29 Thread Jarosław Staniek
convenient since we're only implementing them in one element so far. - Jarosław Staniek On Nov. 29, 2014, 7:04 p.m., Adam Pigg wrote: > > --- > This is an automatically

Re: Review Request 119222: Make labels inline-editable

2014-11-29 Thread Jarosław Staniek
- > > (Updated Nov. 29, 2014, 7:04 p.m.) > > > Review request for Calligra and Jarosław Staniek. > > > Bugs: 336825 > http://bugs.kde.org/show_bug.cgi?id=336825 > > > Repository: calligra > > > Description > --- >

Re: Review Request 121281: When printing reports use High DPI

2014-11-29 Thread Jarosław Staniek
avoid multiplication everywhere? libs/koreport/renderer/KoReportPrintRenderer.cpp <https://git.reviewboard.kde.org/r/121281/#comment49661> 1. For better portability: double -> qreal 2. Move this line out of the loop. - Jarosław Staniek On Nov. 28, 2014, 8:52 p.m., A

Re: Review Request 119222: Make labels inline-editable

2014-11-29 Thread Jarosław Staniek
org/r/119222/ > --- > > (Updated Nov. 29, 2014, 11:37 p.m.) > > > Review request for Calligra and Jarosław Staniek. > > > Bugs: 336825 > http://bugs.kde.org/show_bug.cgi?id=336825 > > > Repository: calligra > > > Description > --- > > Usage:

Re: Review Request 121281: When printing reports use High DPI

2014-12-01 Thread Jarosław Staniek
e use QPainter::scale() transformation in one place to avoid "* scale" multiplication everywhere? I thought the QPainter::scale() setting also affects the line width, what's what we want too. Are you willing to experiment? - Jarosław Staniek On Nov. 30, 2014, 9:05 p

Re: Review Request 121281: When printing reports use High DPI

2014-12-01 Thread Jarosław Staniek
... - Jarosław Staniek On Nov. 30, 2014, 9:05 p.m., Adam Pigg wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 121237: Improve appearance of main tabbed toolbar for GTK+ and Oxygen styles

2014-12-01 Thread Jarosław Staniek
ly generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121237/#review71218 --- On Nov. 28, 2014, 10:07 p.m., Jarosław Staniek wrote: > > --- > This is an automat

Re: Review Request 121237: Improve appearance of main tabbed toolbar for GTK+ and Oxygen styles

2014-12-01 Thread Jarosław Staniek
better Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Review Request 121321: Newly created project shouldn't use window of previously opened project

2014-12-01 Thread Jarosław Staniek
w.cpp d61f955b340091952212da7bc1f58af3fde3b380 kexi/main/KexiMainWindow.h 13532b8fd234be71f76069f95bb8bfb2162d9de7 Diff: https://git.reviewboard.kde.org/r/121321/diff/ Testing --- Works OK for any combination of file/server projects Thanks, Jarosław St

Re: Review Request 121321: Newly created project shouldn't use window of previously opened project

2014-12-01 Thread Jarosław Staniek
r any combination of file/server projects Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 120898: Make side panes lighter by removing frames

2014-12-07 Thread Jarosław Staniek
> On Oct. 30, 2014, 8:11 p.m., Adam Pigg wrote: > > What does it look like with a scroll bar? Where are the button actions > > gone from the top? > > Jarosław Staniek wrote: > http://wstaw.org/m/2014/10/30/plasma-desktopxB1834.png > > These are k

Re: Review Request 120898: Make side panes lighter by removing frames

2014-12-07 Thread Jarosław Staniek
://git.reviewboard.kde.org/media/uploaded/files/2014/10/30/fabd15f3-00eb-4b83-bd28-b26c2fcce5cc__2.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 120033: Multiple map elements render in reports.

2014-12-08 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120033/#review71537 --- What's new with this Radek? - Jarosław Staniek On Se

Re: Review Request 117035: Kexi quickform QML plugin

2014-12-08 Thread Jarosław Staniek
tasks for Kexi Mobile with Qt Quick 2 and Qt Quick Controls. Then entire GUI would be made of the Controls. - Jarosław Staniek On March 24, 2014, 10:44 p.m., Adam Pigg wrote: > > --- > This is an automatically generated e-mail.

Review Request 121389: Newly created project shouldn't use window of previously opened project

2014-12-08 Thread Jarosław Staniek
ctAssistant.cpp 78a3e08c6248a18a2c1d150676fe0bcf1118846e Diff: https://git.reviewboard.kde.org/r/121389/diff/ Testing --- Works OK for any combination of file/server projects Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https

Re: Review Request 121389: Newly created project shouldn't use window of previously opened project

2014-12-08 Thread Jarosław Staniek
d62cf2f5bf kexi/main/startup/KexiNewProjectAssistant.cpp 78a3e08c6248a18a2c1d150676fe0bcf1118846e Diff: https://git.reviewboard.kde.org/r/121389/diff/ Testing --- Works OK for any combination of file/server projects Thanks, Jarosław Staniek

Review Request 121420: Make the project navigator more modern - towards Plasma 5

2014-12-09 Thread Jarosław Staniek
/KexiPropertyPaneViewBase.cpp bd63b721f3681aef2b89e7c8aa67b4291493ce46 kexi/core/KexiView.cpp 64615f51a4ca9bd6d47f9cd9d277ef4c8a2dd5ec Diff: https://git.reviewboard.kde.org/r/121420/diff/ Testing --- All previous functionality still present Thanks, Jarosław Staniek

Re: Review Request 121420: Make the project navigator more modern - towards Plasma 5

2014-12-11 Thread Jarosław Staniek
://git.reviewboard.kde.org/r/121420/diff/ Testing --- All previous functionality still present Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121509: CMake module for detecting marble with Marble marble version.

2014-12-15 Thread Jarosław Staniek
ail. To reply, visit: > https://git.reviewboard.kde.org/r/121509/ > --- > > (Updated Dec. 14, 2014, 4:28 p.m.) > > > Review request for Calligra, Adam Pigg and Jarosław Staniek. > > > Repository: calligra > > >

Re: Review Request 121509: CMake module for detecting marble with Marble marble version.

2014-12-15 Thread Jarosław Staniek
the PATH changes. - Jarosław Staniek On Dec. 14, 2014, 4:28 p.m., Radosław Wicik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewb

Re: Review Request 121509: CMake module for detecting marble with Marble marble version.

2014-12-17 Thread Jarosław Staniek
* variables? What's minimal version accordint to your test? - Jarosław Staniek On Dec. 16, 2014, 11:26 p.m., Radosław Wicik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard

Re: Review Request 121509: CMake module for detecting marble with Marble marble version.

2014-12-20 Thread Jarosław Staniek
org/r/121509/ > --- > > (Updated Dec. 19, 2014, 11:12 p.m.) > > > Review request for Calligra, Adam Pigg and Jarosław Staniek. > > > Repository: calligra > > > Description > --- > > CMake module for detecting Marble with versi

Review Request 121625: Port Kexi Table View to Qt 4

2014-12-21 Thread Jarosław Staniek
Diffs - Diff: https://git.reviewboard.kde.org/r/121625/diff/ Testing --- Tested on tables and query designer: functional Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121625: Port Kexi Table View to Qt 4

2014-12-21 Thread Jarosław Staniek
irecordnavigator.cpp dd47b2c libs/db/tableviewcolumn.h f2370bc kexi/plugins/tables/kexitabledesigner_dataview.h ea9aba0 kexi/CMakeLists.txt 55eac5b Diff: https://git.reviewboard.kde.org/r/121625/diff/ Testing --- Tested on tables and query designer: functional

Re: Review Request 121625: Port Kexi Table View to Qt 4

2014-12-22 Thread Jarosław Staniek
://community.kde.org/Kexi/Porting_to_Qt%26KDE_4#Table_View - Jarosław Staniek On Dec. 21, 2014, 11:27 p.m., Jarosław Staniek wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 120935: Add 'Value' property to report items which can contain static data if not bound to a field

2014-12-23 Thread Jarosław Staniek
de.org/r/120935/#comment50510> Per convention, QString() is enough instead of "" plugins/reporting/barcode/KoReportDesignerItemBarcode.cpp <https://git.reviewboard.kde.org/r/120935/#comment50511> Per convention, QString() is enough instead of "" plugin

Review Request 121655: Add overloads (non-raw queries and params) for convenience KexiDB functions

2014-12-23 Thread Jarosław Staniek
-- Tested with large .kexi files; these methods are used there and Kexi still works OK Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121625: Port Kexi Table View to Qt 4

2014-12-29 Thread Jarosław Staniek
/121625/#review72566 ------- On Dec. 29, 2014, 10:28 p.m., Jarosław Staniek wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kd

Re: Review Request 121625: Port Kexi Table View to Qt 4

2014-12-29 Thread Jarosław Staniek
esigner: functional Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121389: Newly created project shouldn't use window of previously opened project

2014-12-29 Thread Jarosław Staniek
jectAssistant.h 25b4705b303966e14c1e5db7cc3126d62cf2f5bf kexi/main/startup/KexiNewProjectAssistant.cpp 78a3e08c6248a18a2c1d150676fe0bcf1118846e Diff: https://git.reviewboard.kde.org/r/121389/diff/ Testing --- Works OK for any combination of file/server projects Thanks, J

Re: Review Request 121420: Make the project navigator more modern - towards Plasma 5

2014-12-31 Thread Jarosław Staniek
previous functionality still present Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121655: Add overloads (non-raw queries and params) for convenience KexiDB functions

2014-12-31 Thread Jarosław Staniek
bs/db/utils.cpp 923a401f7f4265d19c699db3d35046f21914747f Diff: https://git.reviewboard.kde.org/r/121655/diff/ Testing --- Tested with large .kexi files; these methods are used there and Kexi still works OK Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-deve

Re: Review Request 121625: Port Kexi Table View to Qt 4

2015-01-01 Thread Jarosław Staniek
org/r/121625/#review72874 --- On Dec. 29, 2014, 10:28 p.m., Jarosław Staniek wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > htt

Re: Review Request 121625: Port Kexi Table View to Qt 4

2015-01-01 Thread Jarosław Staniek
> On Jan. 1, 2015, 8:18 p.m., Friedrich W. H. Kossebau wrote: > > Added some more defects I found on the wiki page. Looking into header not > > resizing currently. > > Jarosław Staniek wrote: > Thanks, categorized them now to major/minor/wishes. Do you thin

Re: Review Request 121625: Port Kexi Table View to Qt 4

2015-01-02 Thread Jarosław Staniek
kexi/pics/tableview_pen.png f91eacf63e604300ca563a48bd749cde5d33cb6c kexi/CMakeLists.txt 55eac5b kexi/formeditor/connectiondialog.h 627499e kexi/formeditor/connectiondialog.cpp 469e78a Diff: https://git.reviewboard.kde.org/r/121625/diff/ Testing --- Tested on tables and query designer: functional

Review Request 121793: Fix: if window gets focus back, view jumps to ensure current cell is visible

2015-01-02 Thread Jarosław Staniek
/dataviewcommon/kexidataawareobjectiface.h 24cfa546324e3fa65fb8d5b36bf3e26f66bd47c3 Diff: https://git.reviewboard.kde.org/r/121793/diff/ Testing --- The issue disappeared in data table views Thanks, Jarosław Staniek ___ calligra-devel mailing list

Re: Review Request 121868: Remove uncompiled files that include Q3 or K3 headers

2015-01-05 Thread Jarosław Staniek
message please list all classes removed in case we'll want to find them for reference later. kexi/plugins/forms/CMakeLists.txt <https://git.reviewboard.kde.org/r/121868/#comment50916> extra tabs not needed - Jarosław Staniek On Jan. 5, 2015, 9:38 p.m., Adam

Review Request 121874: Improve major icons and introduce KexiIcon

2015-01-05 Thread Jarosław Staniek
and a few other themes, plus all relevant GUIs File Attachments Welcome Screen with refreshed icons https://git.reviewboard.kde.org/media/uploaded/files/2015/01/05/252a3b9a-02cd-40d2-9584-143d401c2b71__kexi-2.9-breeze-icons.png Thanks, J

Re: Review Request 121793: Fix: if window gets focus back, view jumps to ensure current cell is visible

2015-01-07 Thread Jarosław Staniek
rd to shorten it :) Maybe one day... - Jarosław --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121793/#review73413 --- On Jan. 2, 2015,

Re: Review Request 121793: Fix: if window gets focus back, view jumps to ensure current cell is visible

2015-01-07 Thread Jarosław Staniek
, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121874: Improve major icons and introduce KexiIcon

2015-01-07 Thread Jarosław Staniek
g headers appear OK soon. Thanks so much for the review! - Jarosław --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121874/#review73429 --------

Re: Review Request 121874: Improve major icons and introduce KexiIcon

2015-01-07 Thread Jarosław Staniek
-02cd-40d2-9584-143d401c2b71__kexi-2.9-breeze-icons.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Review Request 121938: Properly merge geometry record navigator with horizontal scrollbar

2015-01-08 Thread Jarosław Staniek
sting --- Checked with tables/queries/forms/reports and various styles File Attachments The navigator https://git.reviewboard.kde.org/media/uploaded/files/2015/01/09/806686d1-7680-434a-b2be-52a3b03a1275__d7AloPW.png Thanks, Jarosław St

Re: Review Request 121938: Properly merge record navigator with horizontal scrollbar

2015-01-08 Thread Jarosław Staniek
media/uploaded/files/2015/01/09/806686d1-7680-434a-b2be-52a3b03a1275__d7AloPW.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 121938: Properly merge record navigator with horizontal scrollbar

2015-01-08 Thread Jarosław Staniek
W.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 120935: Add 'Value' property to report items which can contain static data if not bound to a field

2015-01-10 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120935/#review73658 --- Ship it! Good Job! - Jarosław Staniek On Jan. 4, 2015, 10

Re: Review Request 121938: Properly merge record navigator with horizontal scrollbar

2015-01-10 Thread Jarosław Staniek
files/2015/01/09/806686d1-7680-434a-b2be-52a3b03a1275__d7AloPW.png Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Review Request 122021: Kexi: Clicking "Go to new record" in tableview moves to first reasonable column

2015-01-12 Thread Jarosław Staniek
bility fix) Diffs - kexi/widget/dataviewcommon/kexidataawareobjectiface.cpp aeaf7175feed04568d55b6f0410c66d431a27060 Diff: https://git.reviewboard.kde.org/r/122021/diff/ Testing --- Editable table views work Thanks, Jarosław Staniek __

Re: Review Request 122021: Kexi: Clicking "Go to new record" in tableview moves to first reasonable column

2015-01-12 Thread Jarosław Staniek
<https://git.reviewboard.kde.org/r/122021/#comment51371> int columnToSelect = 0; does the trick, selects 1st column. - Jarosław Staniek On Jan. 12, 2015, 10:59 p.m., Jarosław Staniek wrote: > > --- > This is an automatica

Re: Review Request 122021: Kexi: Clicking "Go to new record" in tableview moves to first reasonable column

2015-01-12 Thread Jarosław Staniek
the active column. (this is a usability fix) Diffs (updated) - kexi/widget/dataviewcommon/kexidataawareobjectiface.cpp aeaf717 Diff: https://git.reviewboard.kde.org/r/122021/diff/ Testing --- Editable table views work Thanks, Jarosł

Re: Review Request 122042: Fix crash after attempt to change alignment in a text field

2015-01-13 Thread Jarosław Staniek
g/r/122042/#comment51395> The original code at 2242 looks like command compression. Could you explain what happens after your change? - Jarosław Staniek On Jan. 13, 2015, 9:06 p.m., Roman Shtemberko wrote: > > ---

Re: Review Request 122021: Kexi: Clicking "Go to new record" in tableview moves to first reasonable column

2015-01-13 Thread Jarosław Staniek
olumn that has no autoincrement flag set. Before it was the active column. (this is a usability fix) Diffs - kexi/widget/dataviewcommon/kexidataawareobjectiface.cpp aeaf717 Diff: https://git.reviewboard.kde.org/r/122021/diff/ Testing --- Editable table views work Thanks, Jarosł

Re: Review Request 120940: Added parsing of hashtag comments inside imported csv in kexi

2015-01-13 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120940/#review73989 --- Let's finish it! - Jarosław Staniek On Nov. 1, 2014,

Re: Review Request 122036: Fix crash during elements deletion in forms

2015-01-15 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122036/#review74067 --- Ship it! Perfect! - Jarosław Staniek On Jan. 13, 2015, 3

Re: Review Request 122042: Fix crash after attempt to change alignment in a text field

2015-01-15 Thread Jarosław Staniek
. - Jarosław Staniek On Jan. 14, 2015, 9:55 a.m., Roman Shtemberko wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 122042: Fix crash after attempt to change alignment in a text field

2015-01-15 Thread Jarosław Staniek
il. To reply, visit: > https://git.reviewboard.kde.org/r/122042/ > --- > > (Updated Jan. 14, 2015, 9:55 a.m.) > > > Review request for Calligra, Adam Pigg and Jarosław Staniek. > > > Bugs: 342371 > http://bug

Re: Review Request 122051: Add Ctrl+F5 shortcut for Check Query of SQL query in Query Design view SQL

2015-01-15 Thread Jarosław Staniek
<https://git.reviewboard.kde.org/r/122051/#comment51448> Good catch. I wonder if we can add this line just to KexiView::setViewActions() for each action. - Jarosław Staniek On Jan. 14, 2015, 12:21 p.m., Roman Shtemberko

Re: Review Request 122053: Fix Ctl+Shift+Insert shortcut for Table/Query Design Views

2015-01-15 Thread Jarosław Staniek
AwareObject->isSortingEnabled()); ...and it works - both in table and query designer. However we wouldn't need to explicitly call addAction() this way. Let's leave this unresolved for a while so I'll look how to improve the overall API without workarounds. Looks like this would be

Re: Review Request 122053: Fix Ctl+Shift+Insert shortcut for Table/Query Design Views

2015-01-24 Thread Jarosław Staniek
> On Jan. 15, 2015, 5:27 p.m., Jarosław Staniek wrote: > > Your solution is a workaround - 1. we would need add the shortcut by hand > > for every use of the table view, and 2. we wouldn't have global shortcut > > available for configuration by users. > > &g

Re: Review Request 122197: Fixes to .ods import

2015-01-24 Thread Jarosław Staniek
<https://git.reviewboard.kde.org/r/122197/#comment51768> object -> table kexi/widget/KexiConnectionSelectorWidget.cpp <https://git.reviewboard.kde.org/r/122197/#comment51770> This looks good! - Jarosław Staniek On Jan. 22, 2015, 9:58 a.m., Roman Shtemberko

Review Request 122240: Improve Kexi data-aware cell editing API

2015-01-24 Thread Jarosław Staniek
ews remain unchanged Thanks, Jarosław Staniek ___ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel

Re: Review Request 122240: Improve Kexi data-aware cell editing API

2015-01-25 Thread Jarosław Staniek
exi/widget/tableview/KexiTableScrollArea.h 836f5cc kexi/widget/tableview/KexiTableScrollArea.cpp 0a06fc9 Diff: https://git.reviewboard.kde.org/r/122240/diff/ Testing --- functionality of table views remain unchanged Thanks, Jarosław Staniek __

Re: Review Request 122240: Improve Kexi data-aware cell editing API

2015-01-27 Thread Jarosław Staniek
/tableview/KexiTableScrollArea.h 836f5cc kexi/widget/tableview/KexiTableScrollArea.cpp 0a06fc9 Diff: https://git.reviewboard.kde.org/r/122240/diff/ Testing --- functionality of table views remain unchanged Thanks, Jarosław Staniek ___ calligra-devel ma

Re: Review Request 122042: Fix crash after attempt to change alignment in a text field

2015-02-01 Thread Jarosław Staniek
tps://git.reviewboard.kde.org/r/122042/#comment52001> "geomtery" is a typo. How this fixes anything? :) > Other than that we will need to add some handler for "geometry" > property in Form::slotPropertyChanged(). Works as expected on my side. Interesting so could you please explain? Ca

Review Request 122380: Improve undo/redo in Kexi form designer

2015-02-02 Thread Jarosław Staniek
itor/form_p.cpp a6a277a583476e4c9a24d40d3124efc367d4babd kexi/formeditor/resizehandle.cpp f4e8b1e7c74f31cb6613caa1e042ada559389f62 Diff: https://git.reviewboard.kde.org/r/122380/diff/ Testing --- Tested improvements with various widgets Thanks, Jarosław Staniek _

Re: Review Request 122045: Calligra filters: refactor chart handling and make it sane

2015-02-02 Thread Jarosław Staniek
.reviewboard.kde.org/r/122045/#comment52021> -- indent filters/words/docx/import/CMakeLists.txt <https://git.reviewboard.kde.org/r/122045/#comment52022> Will these three files be moved to a shared lib instead one day? - Jarosław Staniek On Jan. 18, 2015, 4:26 a.m., Inge Wal

Re: Review Request 122035: Calligra filters: Enhance the odf reader library significantly

2015-02-02 Thread Jarosław Staniek
comment52032> --explicit filters/libodfreader/OdtReaderBackend.h <https://git.reviewboard.kde.org/r/122035/#comment52029> --explicit - Jarosław Staniek On Jan. 13, 2015, 3:30 p.m., Inge Wallin wrote: > > --- &

  1   2   3   4   5   6   7   8   9   10   >