https://bugs.documentfoundation.org/show_bug.cgi?id=97087
Muhammet Kara <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #7 from Muhammet Kara <[email protected]> --- (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 [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
