From b5832c1759ea79952d7b7a451047387446995a31 Mon Sep 17 00:00:00 2001 From: Adrien Destugues Date: Mon, 5 Jan 2015 15:51:41 +0100 Subject: [PATCH] Appearance: don't send kMsgUpdate on ATTRIBUTE_CHOSEN As discussed in #11698, selecting an item in the list needs an update of the selected color, but doesn't need a check for changes and status of the "revert" button. --- src/preferences/appearance/APRView.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/preferences/appearance/APRView.cpp b/src/preferences/appearance/APRView.cpp index 4ffcc8d1a0..401fa162b7 100644 --- a/src/preferences/appearance/APRView.cpp +++ b/src/preferences/appearance/APRView.cpp @@ -141,6 +141,8 @@ APRView::MessageReceived(BMessage *msg) if (msg->FindData("RGBColor", (type_code)'RGBC', (const void**)&color, &size) == B_OK) { _SetCurrentColor(*color); + + Window()->PostMessage(kMsgUpdate); } } @@ -150,6 +152,8 @@ APRView::MessageReceived(BMessage *msg) // Received from the color fPicker when its color changes rgb_color color = fPicker->ValueAsColor(); _SetCurrentColor(color); + + Window()->PostMessage(kMsgUpdate); break; } @@ -262,8 +266,6 @@ APRView::_SetCurrentColor(rgb_color color) fPicker->SetValue(color); fColorPreview->SetColor(color); fColorPreview->Invalidate(); - - Window()->PostMessage(kMsgUpdate); }