[Commits] [wesnoth/wesnoth] 03841c: Editor: use emplace_back for palette item_group se...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 03841cb1a0c47f4b13a9c4163a69e1427b426036
      https://github.com/wesnoth/wesnoth/commit/03841cb1a0c47f4b13a9c4163a69e1427b426036
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2017-05-02 (Tue, 02 May 2017)

  Changed paths:
    M src/editor/palette/item_palette.cpp
    M src/editor/palette/terrain_palettes.cpp
    M src/editor/palette/unit_palette.cpp

  Log Message:
  -----------
  Editor: use emplace_back for palette item_group setup


  Commit: 1000d9684ab77f47976a6ab1cc6dc452a29472c4
      https://github.com/wesnoth/wesnoth/commit/1000d9684ab77f47976a6ab1cc6dc452a29472c4
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2017-05-02 (Tue, 02 May 2017)

  Changed paths:
    M src/widgets/button.cpp

  Log Message:
  -----------
  GUI1: allow buttons to accept "" as a valid image path

This allows the editor to clear the palette group menu button overlay by passing "".


  Commit: e64727271b09444fc3f658659b95b9d8c94c1db0
      https://github.com/wesnoth/wesnoth/commit/e64727271b09444fc3f658659b95b9d8c94c1db0
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2017-05-02 (Tue, 02 May 2017)

  Changed paths:
    M src/editor/palette/palette_manager.cpp

  Log Message:
  -----------
  Editor: 'unhide' palette when drawing contents

I'm not sure why, but this allows the enabled state of the to be set properly when switching to a tool
with an empty palette. It seems this function is called after editor_toolkit::hotkey_set_mouse_action
is fired, which already seems to do that, so I'm not sure why it doesn't take effect there, but meh.

Also includes a fixup for when palette_menu_button is fetched. It's a menu button not an action button,
but it probably not havig worked before seems not to have affected anything.


  Commit: a29f3bb244327087ef3b08543365fb1008a29d1e
      https://github.com/wesnoth/wesnoth/commit/a29f3bb244327087ef3b08543365fb1008a29d1e
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2017-05-02 (Tue, 02 May 2017)

  Changed paths:
    M src/editor/palette/empty_palette.hpp
    M src/editor/palette/location_palette.hpp

  Log Message:
  -----------
  Editor: remove 'Change Palette Group' entry from empty and location palettes

This is a dummy entry that's supposed to be expanded to the toolkit's palette groups.
Doesn't really matter now, since the button will be hidden, but it's good to do.


  Commit: b04a9728a0a4387f715259297fdc97fef6640d3c
      https://github.com/wesnoth/wesnoth/commit/b04a9728a0a4387f715259297fdc97fef6640d3c
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2017-05-02 (Tue, 02 May 2017)

  Changed paths:
    M src/editor/palette/empty_palette.hpp

  Log Message:
  -----------
  Editor: keep empty palette group list as static


Compare: https://github.com/wesnoth/wesnoth/compare/0f0a245c5c55...b04a9728a0a4