Review Request: Introduce Author into the Calligra family

[Gopalakrishna Bhat]

> On Aug. 18, 2012, 8:24 a.m., Jarosław Staniek wrote:
> > words/part/author/main.cpp, line 29
> > <http://git.reviewboard.kde.org/r/106064/diff/2/?file=78322#file78322line29>
> >
> >     Why to hardcode it here? No other app, despote potential gains, do that in Calligra. You may want to put an advice (to use command line switch for calligraauthor) on the community.kde.org for Author if there's specific need like this. 
> >

Calligra Words uses it currently. There was a discussion on the mailing list about this http://www.mail-archive.com/calligra-devel@kde.org/msg00234.html


- Gopalakrishna


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


On Aug. 17, 2012, 8:46 p.m., Gopalakrishna Bhat wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106064/
> -----------------------------------------------------------
> 
> (Updated Aug. 17, 2012, 8:46 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> This is the first step in realizing [1]. Currently author resides in words/part and will eventually moved out to a directory of its own. This patch is currently a work in progress and to evaluate that we are on the right path.
> 
> Currently there is a known issue that various options of the text tool are not shown for the first time when the application is launched(see attached snapshot). Once I explicitly activate the tool by switching to a different tool and back it works fine. Any ideas what can be wrong? 
> 
> 
> [1] http://www.calligra.org/news/calligra-announces-author/
> 
> 
> Diffs
> -----
> 
>   words/part/CMakeLists.txt cdeca2b 
>   words/part/KWPart.h 5b6621b 
>   words/part/KWPart.cpp f94354a 
>   words/part/author/CAuAboutData.h PRE-CREATION 
>   words/part/author/CAuFactory.h PRE-CREATION 
>   words/part/author/CAuFactory.cpp PRE-CREATION 
>   words/part/author/CAuFactoryInit.cpp PRE-CREATION 
>   words/part/author/CAuPart.h PRE-CREATION 
>   words/part/author/CAuPart.cpp PRE-CREATION 
>   words/part/author/CAuView.h PRE-CREATION 
>   words/part/author/CAuView.cpp PRE-CREATION 
>   words/part/author/author.desktop PRE-CREATION 
>   words/part/author/author.rc PRE-CREATION 
>   words/part/author/author_readonly.rc PRE-CREATION 
>   words/part/author/authorpart.desktop PRE-CREATION 
>   words/part/author/authorrc PRE-CREATION 
>   words/part/author/main.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/106064/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Text Tool
>   http://git.reviewboard.kde.org/r/106064/s/678/
> 
> 
> Thanks,
> 
> Gopalakrishna Bhat
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120819/b55c9b3f/attachment.htm>