From 27485b7ff68f51ebaeeb1d2e349b6bc9e4e1e913 Mon Sep 17 00:00:00 2001 From: David Capello Date: Thu, 21 May 2015 16:25:41 -0300 Subject: [PATCH] Fix problem selecting transparent color from the ColorBar with middle button When we move the mouse with the middle mouse button, we must trigger onPaletteViewIndexChange() delegate in any case (the middle button doesn't change the m_currentEntry). --- src/app/ui/palette_view.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/app/ui/palette_view.cpp b/src/app/ui/palette_view.cpp index 0b3ab5786..b4c58735b 100644 --- a/src/app/ui/palette_view.cpp +++ b/src/app/ui/palette_view.cpp @@ -316,10 +316,11 @@ bool PaletteView::onProcessMessage(Message* msg) StatusBar::instance()->showColor(0, "", app::Color::fromIndex(idx), 255); + MouseButtons buttons = mouseMsg->buttons(); - if (hasCapture() && (idx != m_currentEntry || msg->type() == kMouseDownMessage)) { - MouseButtons buttons = mouseMsg->buttons(); - + if (hasCapture() && ((idx != m_currentEntry) || + (msg->type() == kMouseDownMessage) || + ((buttons & kButtonMiddle) == kButtonMiddle))) { if ((buttons & kButtonMiddle) == 0) { if (!msg->ctrlPressed()) deselect();