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