> On July 11, 2011, 4 a.m., Thorsten Zachmann wrote: > > Some things I noticed when testing: > > > > The custom slide show name is not fully readable when the name is to long > > for the combobox. The combo box should use a bigger width if available (not > > the full width that is available) so that the full name is readable. At the > > moment it is just to small. > > > > It is not possible to add the same slide to a custom slide show multiple > > time. That should be possible as the user does not need to create copies of > > the slide if he wants to use it multiple times. > > > > The animation that is show when a custom slide show gets selected should be > > faster. At the moment it takes quite some time. > > > > The old way of adding custom slide shows should be removed the same time > > the new custom slide shows are added. To make the feature easier to find > > the menu entry in Edit menu should bring you to the new stuff. > > > > If a slide is deleted (e.g. in the normal slide sorter) that is part of a > > custom slide show it is not readadded to the custom slide show when the > > delete of the slide is undone. > > Thorsten Zachmann wrote: > Forget the most important thing. This all looks very nice and I like it > very much. Good work. > > Paul Mendez wrote: > Thanks, I haven't notice the combobox length problem, because when you > work with a document that already have some custom shows, the combobox takes > the length of the longest name. So now, I will fix it. > I have to explore more how to add the same slide multiple times, because > I could easily add it to the custom show list, but when you go to > presentation mode it shows only one slide and in some occasions I got some > extrange behaviour like infinite loops, or things like that. > The animation is fast in my computer, but I think there isn't problem on > reduce the animation time. > I was thinking to work in the integration with application menus this > week. > I started working in the undo command for deleted slides last week but I > couldn't yet find a way to recover the delete cmd and then concatenate it > with a new delete command, or other way to make the two commands work at the > same time. >
I will fix the other comments. - Paul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101820/#review4541 ----------------------------------------------------------- On July 8, 2011, 8:02 p.m., Paul Mendez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101820/ > ----------------------------------------------------------- > > (Updated July 8, 2011, 8:02 p.m.) > > > Review request for Calligra. > > > Summary > ------- > > (preliminary) new GUI for manage custom slide shows. > Note: As some GUI integration is still missing, it's only a review request, > it isn't for merge with master branch. > (part of GSoC project) > > > Diffs > ----- > > kpresenter/part/CMakeLists.txt 613ff8d3de84a19776df1651c0bd3604fc395099 > kpresenter/part/KPrCustomSlideShows.h > 7d8a91b67e14e32312fd2706dcede6efac6e1aa7 > kpresenter/part/KPrCustomSlideShows.cpp > 4088a7446d2992d3d56059bec4ad646b4bbfc77f > kpresenter/part/KPrCustomSlideShowsModel.h PRE-CREATION > kpresenter/part/KPrCustomSlideShowsModel.cpp PRE-CREATION > kpresenter/part/KPrSlidesManagerView.h > 6f345e049cdf746e00fdfd9e3e7644668ff9e96e > kpresenter/part/KPrSlidesManagerView.cpp > 39e80b1b770584f97e6bfcbcebc06bd107bc1a68 > kpresenter/part/KPrSlidesSorterDocumentModel.h > a5b65a4d87f8af6a51b4e1992c2ac2a3d23aed37 > kpresenter/part/KPrSlidesSorterDocumentModel.cpp > 4938e3696bed3fb90c017aba5997c40268ce3e99 > kpresenter/part/KPrSlidesSorterItemDelegate.h PRE-CREATION > kpresenter/part/KPrSlidesSorterItemDelegate.cpp PRE-CREATION > kpresenter/part/KPrViewModeSlidesSorter.h > 240bd9e0c6b4158ff0726f2e72e4c79982a9ea18 > kpresenter/part/KPrViewModeSlidesSorter.cpp > 61866f8a1344e1999224444d71a46a1a21a1864e > kpresenter/part/commands/KPrAddCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/commands/KPrAddCustomSlideShowCommand.cpp PRE-CREATION > kpresenter/part/commands/KPrDelCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/commands/KPrDelCustomSlideShowCommand.cpp PRE-CREATION > kpresenter/part/commands/KPrEditCustomSlideShowsCommand.h PRE-CREATION > kpresenter/part/commands/KPrEditCustomSlideShowsCommand.cpp PRE-CREATION > kpresenter/part/commands/KPrRenameCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/commands/KPrRenameCustomSlideShowCommand.cpp PRE-CREATION > kpresenter/part/tests/CMakeLists.txt > f9b56a6ed55ebe610ae30cd21437975c00c317d1 > kpresenter/part/tests/PAMock.h PRE-CREATION > kpresenter/part/tests/TestAddCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/tests/TestAddCustomSlideShowCommand.cpp PRE-CREATION > kpresenter/part/tests/TestDelCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/tests/TestDelCustomSlideShowCommand.cpp PRE-CREATION > kpresenter/part/tests/TestEditCustomSlideShowsCommand.h PRE-CREATION > kpresenter/part/tests/TestEditCustomSlideShowsCommand.cpp PRE-CREATION > kpresenter/part/tests/TestRenameCustomSlideShowCommand.h PRE-CREATION > kpresenter/part/tests/TestRenameCustomSlideShowCommand.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/101820/diff > > > Testing > ------- > > build and run > > > Thanks, > > Paul > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
