Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


On Thu, 2010-10-07 at 11:16 +0200, camille.moulin@free.fr wrote:
 So, apparently, from a technical point of view, when you add a
feature, you have to add extra code to specify that the feature should
not be accessible in ReadOnly mode.

Yup, you are right on here. :-)

IIRC there is a call similar to IsReadOnly() (the name may be slightly
different... I forget), which we need to call on those actions that need
disabled when the document is "read only".

If that's the case, it would be a good example of unnecessary
additional complexity to the code and one more good reason to change
the behaviour to something simpler.

No disagreement there.  I would be among the first ones to support
removal of this cruft.

Having said that, once we remove this, it won't be easy to add it back.
So I'd personally like to be a bit careful on making decision here.  For
example, can anybody provide any arguments for keeping the current
behavior?

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida@novell.com>


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.