Re: Review Request: Changed Form Design Toolbar Icons order and set to have text under icon

2011-10-17 Thread Jarosław Staniek
> On Oct. 17, 2011, 11:02 p.m., Jarosław Staniek wrote: > > Ship It! Congratulations, first reviewboard patch! - Jarosław --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102902/#review745

Re: Review Request: Changed Form Design Toolbar Icons order and set to have text under icon

2011-10-17 Thread Dimitrios Tanis
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102902/ --- (Updated Oct. 17, 2011, 11:02 p.m.) Review request for Calligra and Jarosł

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102871/#review7451 --- Ship it! Ship It! - Jarosław Staniek On Oct. 15, 2011, 4:49

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102871/#review7450 --- libs/flake/KoToolFactoryBase.h

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Jarosław Staniek
> On Oct. 17, 2011, 7:17 p.m., Jarosław Staniek wrote: > > Screenshot: screentshot of the new ui > > > > > > Related: > > If we look at radical changes, New and Open actions are not among most > > frequently used. I removed them from Kexi and have only in

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread C. Boemann
> On Oct. 17, 2011, 7:17 p.m., Jarosław Staniek wrote: > > Screenshot: screentshot of the new ui > > > > > > Hiding whole sidebar could be enough, the only action. No need to hide > > individual panes. This is the same limitation 'by design' as not allowing

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Jarosław Staniek
> On Oct. 17, 2011, 1:33 p.m., Thomas Pfeiffer wrote: > > This is a very good idea for tabs with more than one element in them (e.g. > > Text editing, References, Review). > > > > What it does not make sense for are those tabs that contain only one or two > > buttons (e.g. zoom and pan). Tabs

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102871/#review7446 --- Screenshot: screentshot of the new ui

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Thomas Pfeiffer
> On Oct. 17, 2011, 1:33 p.m., Thomas Pfeiffer wrote: > > This is a very good idea for tabs with more than one element in them (e.g. > > Text editing, References, Review). > > > > What it does not make sense for are those tabs that contain only one or two > > buttons (e.g. zoom and pan). Tabs

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Thomas Pfeiffer
> On Oct. 16, 2011, 6:08 a.m., Thorsten Zachmann wrote: > > My biggest concern is about constiency between the different applications > > changes with this patch. Would it not be better if that would be a common > > option the user can select so all applications would behave like that? > > Cyr

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread C. Boemann
> On Oct. 17, 2011, 2:06 p.m., Markus Meik Slopianka wrote: > > In the first screenshot, there are two drop-down lists (under Character and > > Paragraph) that don't indiciate via an arrow that they are actually > > drop-down lists. > > > > Aside from that in the first and in the Karbon screen

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Markus Meik Slopianka
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102871/#review7426 --- In the first screenshot, there are two drop-down lists (under Ch

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread C. Boemann
> On Oct. 17, 2011, 1:33 p.m., Thomas Pfeiffer wrote: > > This is a very good idea for tabs with more than one element in them (e.g. > > Text editing, References, Review). > > > > What it does not make sense for are those tabs that contain only one or two > > buttons (e.g. zoom and pan). Tabs

Re: Review Request: New UI for words, some hopefully insignificat changes to other apps

2011-10-17 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102871/#review7424 --- This is a very good idea for tabs with more than one element in

Re: Review Request: Fix loading/saving clone layers

2011-10-17 Thread Dmitry Kazakov
> On Oct. 16, 2011, 6:04 p.m., Dmitry Kazakov wrote: > > Hi! > > > > It's a really cool idea to use uuids here! I think the patch is very good, > > but i have one small concern. Is it possible to remove old code like > > copyFromName(), setCopyFromName() and findNodeByName()? Probably, map the