[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdelibs/kdeprint
From:       Michael Goffioul <goffioul () imec ! be>
Date:       2004-03-10 15:13:12
Message-ID: 20040310151312.AC052999B () office ! kde ! org
[Download RAW message or body]

CVS commit by goffioul: 

Add the plugin changer in the expand/collapse feature of print dialog. This setting \
is indeed rarely used (bug #77126)


  M +10 -4     kprintdialog.cpp   1.78


--- kdelibs/kdeprint/kprintdialog.cpp  #1.77:1.78
@@ -845,7 +845,10 @@ void KPrintDialog::expandDialog(bool on)
         if (on)
         {
-                sz.setHeight(sz.height()+d->m_dummy->minimumSize().height()+layout()->spacing());
                
-                if (isVisible() || !d->m_dummy->isVisible())
+                sz.setHeight(sz.height()+d->m_dummy->minimumSize().height()+d->m_plugin->minimumSize().height()+2*layout()->spacing());
 +                if (isVisible() || !d->m_dummy->isVisible() || \
!d->m_plugin->isVisible()) +                {
                         d->m_dummy->show();
+                        d->m_plugin->show();
+                }
                 d->m_extbtn->setIconSet(SmallIconSet("up"));
                 d->m_extbtn->setText(i18n("Collaps&e"));
@@ -854,7 +857,10 @@ void KPrintDialog::expandDialog(bool on)
         else
         {
-                sz.setHeight(sz.height()-d->m_dummy->height()-layout()->spacing());
-                if (!isVisible() || d->m_dummy->isVisible())
+                sz.setHeight(sz.height()-d->m_dummy->height()-d->m_plugin->height()-2*layout()->spacing());
 +                if (!isVisible() || d->m_dummy->isVisible() || \
d->m_plugin->isVisible()) +                {
                         d->m_dummy->hide();
+                        d->m_plugin->hide();
+                }
                 d->m_extbtn->setIconSet(SmallIconSet("down"));
                 d->m_extbtn->setText(i18n("&Expand"));


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic