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

List:       kde-commits
Subject:    [calligra/tables-uirefactor-mek] sheets/ui: only enable the cell editor docker if the cell tool is a
From:       Marijn Kruisselbrink <mkruisselbrink () kde ! org>
Date:       2012-06-01 0:25:40
Message-ID: 20120601002540.39D17A60BB () git ! kde ! org
[Download RAW message or body]

Git commit a671cf2af8f255cf90c94bf6edb520f1aa41dcfc by Marijn Kruisselbrink.
Committed on 01/06/2012 at 02:24.
Pushed by mkruisselbrink into branch 'tables-uirefactor-mek'.

only enable the cell editor docker if the cell tool is active; better would be to \
always allow editing of cells regardless of active tool, but this is easier for a \
quick fix. Hopefully I'll get the other way implemented before 2.5 is released though \
:)

M  +4    -1    sheets/ui/CellEditorDocker.cpp

http://commits.kde.org/calligra/a671cf2af8f255cf90c94bf6edb520f1aa41dcfc

diff --git a/sheets/ui/CellEditorDocker.cpp b/sheets/ui/CellEditorDocker.cpp
index e32ab47..052833f 100644
--- a/sheets/ui/CellEditorDocker.cpp
+++ b/sheets/ui/CellEditorDocker.cpp
@@ -159,7 +159,10 @@ void CellEditorDocker::toolChanged(const QString &toolId)
 {
     kDebug() << "tool changed to" << toolId;
 
-    if (toolId == QLatin1String("KSpreadCellToolId")) {
+    const bool isCellTool = toolId == QLatin1String("KSpreadCellToolId");
+    setEnabled(isCellTool);
+
+    if (isCellTool) {
         KoToolBase* tool = KoToolManager::instance()->toolById(d->canvas, toolId);
         d->cellTool = qobject_cast<CellToolBase*>(tool);
         Q_ASSERT(d->cellTool);


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