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]