summaryrefslogtreecommitdiff
path: root/src/gui/buydialog.cpp
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2012-10-11 02:48:10 +0300
committerAndrei Karas <akaras@inbox.ru>2012-10-11 02:48:10 +0300
commit0b20f61d590d82468d14e564da14444a80ab2c76 (patch)
treee9f549a57ee2a83d5643388f24f5352208a1c2fd /src/gui/buydialog.cpp
parent62a7ad8a4f8cc297c8a0eb6468fb27086de368d0 (diff)
downloadmv-0b20f61d590d82468d14e564da14444a80ab2c76.tar.gz
mv-0b20f61d590d82468d14e564da14444a80ab2c76.tar.bz2
mv-0b20f61d590d82468d14e564da14444a80ab2c76.tar.xz
mv-0b20f61d590d82468d14e564da14444a80ab2c76.zip
Add support for double click in buy and sell dialog.
Diffstat (limited to 'src/gui/buydialog.cpp')
-rw-r--r--src/gui/buydialog.cpp27
1 files changed, 14 insertions, 13 deletions
diff --git a/src/gui/buydialog.cpp b/src/gui/buydialog.cpp
index 30eed9ae2..fd2e3a5f3 100644
--- a/src/gui/buydialog.cpp
+++ b/src/gui/buydialog.cpp
@@ -119,6 +119,10 @@ void BuyDialog::init()
mSlider->setActionEventId("slider");
mSlider->addActionListener(this);
+
+ mShopItemList->setDistributeMousePressed(false);
+ mShopItemList->setActionEventId("buy");
+ mShopItemList->addActionListener(this);
mShopItemList->addSelectionListener(this);
ContainerPlacer placer;
@@ -184,49 +188,47 @@ void BuyDialog::addItem(const int id, const unsigned char color,
void BuyDialog::action(const gcn::ActionEvent &event)
{
- if (event.getId() == "quit")
+ const std::string &eventId = event.getId();
+
+ if (eventId == "quit")
{
close();
return;
}
-
const int selectedItem = mShopItemList->getSelected();
// The following actions require a valid selection
- if (selectedItem < 0 ||
- selectedItem >= mShopItems->getNumberOfElements())
- {
+ if (selectedItem < 0 || selectedItem >= mShopItems->getNumberOfElements())
return;
- }
- if (event.getId() == "slider")
+ if (eventId == "slider")
{
mAmountItems = static_cast<int>(mSlider->getValue());
mAmountField->setValue(mAmountItems);
updateButtonsAndLabels();
}
- else if (event.getId() == "inc" && mAmountItems < mMaxItems)
+ else if (eventId == "inc" && mAmountItems < mMaxItems)
{
mAmountItems++;
mSlider->setValue(mAmountItems);
mAmountField->setValue(mAmountItems);
updateButtonsAndLabels();
}
- else if (event.getId() == "dec" && mAmountItems > 1)
+ else if (eventId == "dec" && mAmountItems > 1)
{
mAmountItems--;
mSlider->setValue(mAmountItems);
mAmountField->setValue(mAmountItems);
updateButtonsAndLabels();
}
- else if (event.getId() == "max")
+ else if (eventId == "max")
{
mAmountItems = mMaxItems;
mSlider->setValue(mAmountItems);
mAmountField->setValue(mAmountItems);
updateButtonsAndLabels();
}
- else if (event.getId() == "amount")
+ else if (eventId == "amount")
{
mAmountItems = mAmountField->getValue();
mSlider->setValue(mAmountItems);
@@ -235,8 +237,7 @@ void BuyDialog::action(const gcn::ActionEvent &event)
// TODO: Actually we'd have a bug elsewhere if this check for the number
// of items to be bought ever fails, Bertram removed the assertions, is
// there a better way to ensure this fails in an _obvious_ way in C++?
- else if (event.getId() == "buy" && mAmountItems > 0 &&
- mAmountItems <= mMaxItems)
+ else if (eventId == "buy" && mAmountItems > 0 && mAmountItems <= mMaxItems)
{
if (mNpcId != -1)
{