dMZX Forums: ALT+D works outside of the palette editor -> Archived MegaZeux Bugs -> Tracker

As noted in the feature request thread, ALT+D is a palette editor hotkey that works outside of the palette editor and does not request confirmation. There are four things that could be done about this:

1) Nothing (BAD ENDING GAME OVER)
2) Make it request confirmation
3) Since it's not noted anywhere outside of the palette editor IIRC, and there's little use for it outside of the palette editor, simply disable it outside of the palette editor
4) zzo has mentioned that ALT+D had a completely different function before the port, related to built-in placement colors. As a final option, this functionality could be restored (zzo will post this version of the fix).

1) No
2) Bare Minimum
3) Feasible, though I could possibly maybe theoretically see the inability making a difference later (e.g. when viewing the whole board). I don't think it'd matter much.
4) This Yet Again. Well, anyway, it should be trivial to put that back in, and bound to something else if need be.

...
static void thing_menu(World *mzx_world, int menu_number, mzx_thing *new_id,
 int *new_color, int *new_param, Robot *copy_robot, Scroll *copy_scroll,
 Sensor *copy_sensor, int x, int y, int uses_default_colors)
...
    if(menu_number<8) { // normal kinds
      id = tmenu_thing_ids[menu_number][chosen];

      if(def_colors[id] && uses_default_colors)
      {
        color = def_colors[id];
      }
      else
      {
        color = *new_color;
      }
...
static void __edit_world(World *mzx_world)
{
...
  int uses_default_colors=1;
...
      case IKEY_d:
      {
        if(draw_mode != 2)
        {
          if(get_alt_status(keycode_internal))
          {
            if(mzx_world->editor_conf.old_altd) {
              uses_default_colors=!uses_default_colors;
            } else {
              default_palette();
              update_palette();
            }
          }
...

This is an approximate way to correct it; you will need to change it according to how you have MegaZeux currently coded. (The "// normal kinds" is about a PZX feature; you can omit that part.)

Updating status to: Awaiting Feedback

Terryn, can you explain your reply to #3?

Updating status to: Fixed
Updating severity to: 2 - Fair
Updating version to: 2.83

Issue fixed in: 2.84


Yeah whatever, I just went with #2 since I doubt anyone really misses the original functionality.

Moving to: Archived MegaZeux Bugs