On Thu, 2010-10-07 at 11:16 +0200, [email protected] 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 <[email protected]> _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
