https://bugs.documentfoundation.org/show_bug.cgi?id=97087
Muhammet Kara <muhammet.k...@pardus.org.tr> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |muhammet.k...@pardus.org.tr
--- Comment #7 from Muhammet Kara <muhammet.k...@pardus.org.tr> ---
(In reply to Michael Meeks from comment #5)
> Hi Muhammet - the strings are understandable, but not that unique eg.
>
> + mpOnlineSpellingIdle = new Idle("OnlineSpelling");
>
> I would prefer "OnlineSpelling (Draw)" or something =) since there will be
> similar functionality for spell checking in many places. Similarly:
>
> + pPageImpl->pLoadIdle = new Idle("DelayedLoad");
>
> could be "Background Tab Page - delayed load".
>
> It is best if the strings are unique and this grep-able in the code-base =)
>
> Otherwise - good stuff.
I forgot to add myself to cc. That's why I am reading your comment now. Sorry
about that, and for the noise. :)
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
