To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=88521





------- Additional comments from [EMAIL PROTECTED] Fri May  9 14:40:01 +0000 
2008 -------
Some comments on the patch:

In Window::Paint, the rectangle is the area that was invalidated, which can be
only a part of the window. So you must not position the elements based on that
rectangle. This is different from status bar controls.

In ScPreviewShell::Execute, you can't call ModifyStyleSheet directly with the
item from the controller, because the page style contains a different item.

The status notification (ScZoomSliderControl::StateChanged) doesn't work because
SvxZoomSliderItem implementation is incomplete. It doesn't set
mnMinZoom/mnMaxZoom in the copy constructor, and it doesn't read/write them in
QueryValue/PutValue.

Also note that our C++ source files must contain 7-bit ASCII characters only,
including comments.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to