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


User mmeeks changed the following:

                What    |Old value                 |New value
================================================================================
                      CC|'cd'                      |'cd,pl'
--------------------------------------------------------------------------------




------- Additional comments from [EMAIL PROTECTED] Wed Aug 29 12:08:13 +0000 
2007 -------
Thanks for the review Philipp - Carsten & others have already reviewed the
properties work in toolkit/ which I guess leaves only the (rather
non-controversial) connection of the virtual methods to VCL.

Having said that - I would like to push down some of the work inside toolkit/ to
get the sizing 'right' down into the VCL GetOptimalSize method impl.s eg.

::com::sun::star::awt::Size VCLXEdit::getMinimumSize( sal_Int16 nCols, sal_Int16
) throw(::com::sun::star::uno::RuntimeException)
{
        ::vos::OGuard aGuard( GetMutex() );

        Size aSz;
        Edit* pEdit = (Edit*) GetWindow();
        if ( pEdit )
        {
                if ( nCols )
                        aSz = pEdit->CalcSize( nCols );
                else
                        aSz = pEdit->CalcMinimumSize();
        }

or: vclxwindow.cxx's:

                        case WINDOW_TIMEBOX:
                        case WINDOW_LONGCURRENCYBOX:
                                aSz.Width() = GetWindow()->GetTextWidth( 
GetWindow()->GetText() )+2*2;
                                aSz.Height() = GetWindow()->GetTextHeight()+2*2;
                        break;

But basically some small changes.

Thanks :-)

---------------------------------------------------------------------
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