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.
This commit is contained in:
@@ -141,6 +141,8 @@ APRView::MessageReceived(BMessage *msg)
|
|||||||
if (msg->FindData("RGBColor", (type_code)'RGBC', (const void**)&color,
|
if (msg->FindData("RGBColor", (type_code)'RGBC', (const void**)&color,
|
||||||
&size) == B_OK) {
|
&size) == B_OK) {
|
||||||
_SetCurrentColor(*color);
|
_SetCurrentColor(*color);
|
||||||
|
|
||||||
|
Window()->PostMessage(kMsgUpdate);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -150,6 +152,8 @@ APRView::MessageReceived(BMessage *msg)
|
|||||||
// Received from the color fPicker when its color changes
|
// Received from the color fPicker when its color changes
|
||||||
rgb_color color = fPicker->ValueAsColor();
|
rgb_color color = fPicker->ValueAsColor();
|
||||||
_SetCurrentColor(color);
|
_SetCurrentColor(color);
|
||||||
|
|
||||||
|
Window()->PostMessage(kMsgUpdate);
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -262,8 +266,6 @@ APRView::_SetCurrentColor(rgb_color color)
|
|||||||
fPicker->SetValue(color);
|
fPicker->SetValue(color);
|
||||||
fColorPreview->SetColor(color);
|
fColorPreview->SetColor(color);
|
||||||
fColorPreview->Invalidate();
|
fColorPreview->Invalidate();
|
||||||
|
|
||||||
Window()->PostMessage(kMsgUpdate);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user