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

List:       kde-commits
Subject:    branches/koffice/1.6/koffice/krita
From:       Bart Coppens <kde () bartcoppens ! be>
Date:       2006-12-30 15:28:00
Message-ID: 1167492480.597925.13382.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 617918 by coppens:

Strange that this didn't work, but the new way is more consistent and actually works. \
Also changed the previously mentioned clearSelection on the adjustment layers to \
something sensible: Select All -> apply adjustment everywhere, Deselect -> apply \
nowhere. BUG:137638
CCBUG:137708

 M  +1 -1      core/kis_adjustment_layer.cc  
 M  +9 -2      ui/kis_selection_manager.cc  


--- branches/koffice/1.6/koffice/krita/core/kis_adjustment_layer.cc #617917:617918
@@ -125,7 +125,7 @@
 
     QRect bounds = extent();
     bounds |= image()->bounds();
-    gc.fillRect(bounds, KisColor(Qt::white, cs), MAX_SELECTED);
+    gc.fillRect(bounds, KisColor(Qt::white, cs), MIN_SELECTED);
     gc.end();
 }
 
--- branches/koffice/1.6/koffice/krita/ui/kis_selection_manager.cc #617917:617918
@@ -508,8 +508,15 @@
     if (img->undo()) t = new KisSelectedTransaction(i18n("Select All"), dev);
     Q_CHECK_PTR(t);
 
-    dev->selection()->clear();
-    dev->selection()->invert();
+    // Make adjustment layers behave better
+    KisAdjustmentLayer* adj = \
dynamic_cast<KisAdjustmentLayer*>(img->activeLayer().data()); +    if (adj) {
+        adj->clearSelection();
+        adj->selection()->invert();
+    } else {
+        dev->selection()->clear();
+        dev->selection()->invert();
+    }
     dev->setDirty();
     dev->emitSelectionChanged();
 


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