chart2/source/controller/dialogs/res_Trendline.cxx | 53 +++++++++------------ chart2/source/controller/dialogs/res_Trendline.hxx | 4 - chart2/uiconfig/ui/tp_Trendline.ui | 9 --- 3 files changed, 27 insertions(+), 39 deletions(-)
New commits: commit 3c5dd9812b1028188b186f070874bdb6ecdcd4ba Author: Laurent Balland-Poirier <laurent.balland-poir...@laposte.net> Date: Fri Dec 13 07:25:20 2013 +0100 fdo#40316 Remove SpinButton in UI for extrapolate trendline UI Trendline: Spinbutton are not adapted for real value of x-values used in extrapolate For/Back-ward. Replaced with FormattedFiled like Intercept value Change-Id: Idc49f842a0f76906eb3c027e15a493e625c3d535 Reviewed-on: https://gerrit.libreoffice.org/7067 Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> Tested-by: Tomaž Vajngerl <qui...@gmail.com> diff --git a/chart2/source/controller/dialogs/res_Trendline.cxx b/chart2/source/controller/dialogs/res_Trendline.cxx index c8bba7e..91552e4 100644 --- a/chart2/source/controller/dialogs/res_Trendline.cxx +++ b/chart2/source/controller/dialogs/res_Trendline.cxx @@ -54,8 +54,8 @@ TrendlineResources::TrendlineResources( Window * pParent, const SfxItemSet& rInA pTabPage->get(m_pNF_Degree,"degree"); pTabPage->get(m_pNF_Period,"period"); pTabPage->get(m_pEE_Name,"entry_name"); - pTabPage->get(m_pNF_ExtrapolateForward,"extrapolateForward"); - pTabPage->get(m_pNF_ExtrapolateBackward,"extrapolateBackward"); + pTabPage->get(m_pFmtFld_ExtrapolateForward,"extrapolateForward"); + pTabPage->get(m_pFmtFld_ExtrapolateBackward,"extrapolateBackward"); pTabPage->get(m_pCB_SetIntercept,"setIntercept"); pTabPage->get(m_pFmtFld_InterceptValue,"interceptValue"); pTabPage->get(m_pCB_ShowEquation,"showEquation"); @@ -81,11 +81,6 @@ TrendlineResources::TrendlineResources( Window * pParent, const SfxItemSet& rInA m_pNF_Period->SetModifyHdl( aLink ); m_pFmtFld_InterceptValue->SetModifyHdl( aLink ); - m_pNF_ExtrapolateForward->SetMin( SAL_MIN_INT64 ); - m_pNF_ExtrapolateForward->SetMax( SAL_MAX_INT64 ); - m_pNF_ExtrapolateBackward->SetMin( SAL_MIN_INT64 ); - m_pNF_ExtrapolateBackward->SetMax( SAL_MAX_INT64 ); - Reset( rInAttrs ); UpdateControlStates(); } @@ -159,27 +154,21 @@ void TrendlineResources::Reset( const SfxItemSet& rInAttrs ) m_pNF_Period->SetValue( 2 ); } + double nValue = 0.0; if( rInAttrs.GetItemState( SCHATTR_REGRESSION_EXTRAPOLATE_FORWARD, sal_True, &pPoolItem ) == SFX_ITEM_SET ) { - double nValue = static_cast< const SvxDoubleItem * >( pPoolItem )->GetValue() * 100; - m_pNF_ExtrapolateForward->SetValue( (sal_Int64) nValue ); - } - else - { - m_pNF_ExtrapolateForward->SetValue( 0 ); + nValue = ((const SvxDoubleItem*)pPoolItem)->GetValue() ; } + lcl_setValue( *m_pFmtFld_ExtrapolateForward, nValue ); + nValue = 0.0; if( rInAttrs.GetItemState( SCHATTR_REGRESSION_EXTRAPOLATE_BACKWARD, sal_True, &pPoolItem ) == SFX_ITEM_SET ) { - double nValue = static_cast< const SvxDoubleItem * >( pPoolItem )->GetValue() * 100; - m_pNF_ExtrapolateBackward->SetValue( (sal_Int64) nValue ); - } - else - { - m_pNF_ExtrapolateBackward->SetValue( 0 ); + nValue = ((const SvxDoubleItem*)pPoolItem)->GetValue() ; } + lcl_setValue( *m_pFmtFld_ExtrapolateBackward, nValue ); - double nValue = 0.0;; + nValue = 0.0; if( rInAttrs.GetItemState( SCHATTR_REGRESSION_INTERCEPT_VALUE, sal_True, &pPoolItem ) == SFX_ITEM_SET ) { nValue = ((const SvxDoubleItem*)pPoolItem)->GetValue() ; @@ -273,19 +262,21 @@ sal_Bool TrendlineResources::FillItemSet(SfxItemSet& rOutAttrs) const sal_Int32 aPeriod = m_pNF_Period->GetValue(); rOutAttrs.Put(SfxInt32Item( SCHATTR_REGRESSION_PERIOD, aPeriod ) ); - double aExtrapolateForwardValue = m_pNF_ExtrapolateForward->GetValue() / 100.0; - rOutAttrs.Put(SvxDoubleItem( aExtrapolateForwardValue, SCHATTR_REGRESSION_EXTRAPOLATE_FORWARD ) ); + sal_uInt32 nIndex = 0; + double aValue = 0.0; + m_pNumFormatter->IsNumberFormat(m_pFmtFld_ExtrapolateForward->GetText(),nIndex,aValue); + rOutAttrs.Put(SvxDoubleItem( aValue, SCHATTR_REGRESSION_EXTRAPOLATE_FORWARD ) ); - double aExtrapolateBackwardValue = m_pNF_ExtrapolateBackward->GetValue() / 100.0; - rOutAttrs.Put(SvxDoubleItem( aExtrapolateBackwardValue, SCHATTR_REGRESSION_EXTRAPOLATE_BACKWARD ) ); + aValue = 0.0; + m_pNumFormatter->IsNumberFormat(m_pFmtFld_ExtrapolateBackward->GetText(),nIndex,aValue); + rOutAttrs.Put(SvxDoubleItem( aValue, SCHATTR_REGRESSION_EXTRAPOLATE_BACKWARD ) ); if( m_pCB_SetIntercept->GetState() != STATE_DONTKNOW ) rOutAttrs.Put( SfxBoolItem( SCHATTR_REGRESSION_SET_INTERCEPT, m_pCB_SetIntercept->IsChecked() )); - double aInterceptValue = 0.0; - sal_uInt32 nIndex = 0; - m_pNumFormatter->IsNumberFormat(m_pFmtFld_InterceptValue->GetText(),nIndex,aInterceptValue); - rOutAttrs.Put(SvxDoubleItem( aInterceptValue, SCHATTR_REGRESSION_INTERCEPT_VALUE ) ); + aValue = 0.0; + m_pNumFormatter->IsNumberFormat(m_pFmtFld_InterceptValue->GetText(),nIndex,aValue); + rOutAttrs.Put(SvxDoubleItem( aValue, SCHATTR_REGRESSION_INTERCEPT_VALUE ) ); return sal_True; } @@ -304,8 +295,8 @@ void TrendlineResources::UpdateControlStates() { bool bMovingAverage = ( m_eTrendLineType == CHREGRESS_MOVING_AVERAGE ); bool bInterceptAvailable = ( m_eTrendLineType == CHREGRESS_LINEAR ) || ( m_eTrendLineType == CHREGRESS_POLYNOMIAL ); - m_pNF_ExtrapolateForward->Enable(!bMovingAverage); - m_pNF_ExtrapolateBackward->Enable(!bMovingAverage); + m_pFmtFld_ExtrapolateForward->Enable(!bMovingAverage); + m_pFmtFld_ExtrapolateBackward->Enable(!bMovingAverage); m_pCB_SetIntercept->Enable( bInterceptAvailable ); m_pFmtFld_InterceptValue->Enable( bInterceptAvailable ); if(bMovingAverage) @@ -348,6 +339,8 @@ IMPL_LINK( TrendlineResources, ChangeValue, void *, pNumericField) void TrendlineResources::SetNumFormatter( SvNumberFormatter* pFormatter ) { m_pNumFormatter = pFormatter; + m_pFmtFld_ExtrapolateForward->SetFormatter( m_pNumFormatter ); + m_pFmtFld_ExtrapolateBackward->SetFormatter( m_pNumFormatter ); m_pFmtFld_InterceptValue->SetFormatter( m_pNumFormatter ); } diff --git a/chart2/source/controller/dialogs/res_Trendline.hxx b/chart2/source/controller/dialogs/res_Trendline.hxx index 37f25b9..6057505 100644 --- a/chart2/source/controller/dialogs/res_Trendline.hxx +++ b/chart2/source/controller/dialogs/res_Trendline.hxx @@ -63,8 +63,8 @@ private: NumericField* m_pNF_Degree; NumericField* m_pNF_Period; Edit* m_pEE_Name; - NumericField* m_pNF_ExtrapolateForward; - NumericField* m_pNF_ExtrapolateBackward; + FormattedField* m_pFmtFld_ExtrapolateForward; + FormattedField* m_pFmtFld_ExtrapolateBackward; CheckBox* m_pCB_SetIntercept; FormattedField* m_pFmtFld_InterceptValue; CheckBox* m_pCB_ShowEquation; diff --git a/chart2/uiconfig/ui/tp_Trendline.ui b/chart2/uiconfig/ui/tp_Trendline.ui index b5a0fad..e9ce8ef 100644 --- a/chart2/uiconfig/ui/tp_Trendline.ui +++ b/chart2/uiconfig/ui/tp_Trendline.ui @@ -398,15 +398,12 @@ </packing> </child> <child> - <object class="GtkSpinButton" id="extrapolateForward"> + <object class="svtlo-FormattedField" id="extrapolateForward"> <property name="visible">True</property> <property name="can_focus">True</property> <property name="invisible_char">â</property> <property name="caps_lock_warning">False</property> <property name="input_purpose">number</property> - <property name="adjustment">adjustmentForwardBacward</property> - <property name="digits">2</property> - <property name="numeric">True</property> </object> <packing> <property name="left_attach">1</property> @@ -416,12 +413,10 @@ </packing> </child> <child> - <object class="GtkSpinButton" id="extrapolateBackward"> + <object class="svtlo-FormattedField" id="extrapolateBackward"> <property name="visible">True</property> <property name="can_focus">True</property> <property name="invisible_char">â</property> - <property name="adjustment">adjustmentForwardBacward</property> - <property name="digits">2</property> </object> <packing> <property name="left_attach">1</property>
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits