Don't create PalettePopup when the program starts

This commit is contained in:
David Capello 2015-05-20 16:51:32 -03:00
parent e730b90958
commit ca16580b09
2 changed files with 10 additions and 6 deletions

View File

@ -29,6 +29,7 @@
#include "app/ui/color_spectrum.h"
#include "app/ui/editor/editor.h"
#include "app/ui/input_chain.h"
#include "app/ui/palette_popup.h"
#include "app/ui/skin/skin_theme.h"
#include "app/ui/status_bar.h"
#include "app/ui_context.h"
@ -322,15 +323,18 @@ void ColorBar::onPaletteButtonClick()
}
case PalButton::PRESETS: {
if (!m_palettePopup.isVisible()) {
if (!m_palettePopup)
m_palettePopup.reset(new PalettePopup());
if (!m_palettePopup->isVisible()) {
gfx::Rect bounds = m_buttons.getItem(item)->getBounds();
m_palettePopup.showPopup(
m_palettePopup->showPopup(
gfx::Rect(bounds.x, bounds.y+bounds.h,
ui::display_w()/2, ui::display_h()/2));
}
else {
m_palettePopup.closeWindow(NULL);
m_palettePopup->closeWindow(NULL);
}
break;
}

View File

@ -13,7 +13,6 @@
#include "app/ui/button_set.h"
#include "app/ui/color_button.h"
#include "app/ui/input_chain_element.h"
#include "app/ui/palette_popup.h"
#include "app/ui/palette_view.h"
#include "base/connection.h"
#include "base/signal.h"
@ -28,8 +27,9 @@
namespace app {
class ColorButton;
class Command;
class PalettesLoader;
class PaletteIndexChangeEvent;
class PalettePopup;
class PalettesLoader;
class ColorBar : public ui::Box
, public PaletteViewDelegate
@ -106,7 +106,7 @@ namespace app {
};
ButtonSet m_buttons;
PalettePopup m_palettePopup;
base::UniquePtr<PalettePopup> m_palettePopup;
ScrollableView m_scrollableView;
PaletteView m_paletteView;
ui::Button m_remapButton;