dMZX Forums: Bad behavior clicking the X to close window -> Archived MegaZeux Bugs -> Tracker

Jump to content

Report ID 616 Title Bad behavior clicking the X to close window
Product Archived MegaZeux Bugs Status Fixed (Severity 3 - Medium)
Version GIT Fixed in GIT
Introduced In Version GITOperating System Windows 7

Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Report ID #616: Bad behavior clicking the X to close window

#1 User is offline  
Terryn 

  • ******
  • Group: DigiStaff
  • Posts: 2,966
  • Joined: 12-October 00
  • Gender:Male

Posted 30 March 2015 - 05:21 AM

On Windows, clicking the X to initiate a closing action for a MegaZeux window fails during certain situations, such as during display of file menus, ASK boxes, and the F11 screen. Trying to do so on these screens will prevent any typical mouse or key input on these screens, but pressing Esc will remove them and things will progress as normal from there.

This seems to be a consequence of commit 4cd886b (12/29/12). I haven't tested its action on OSX or Linux.
angelic stream - shed sanguine - ill-adapt - avis - para/lyser - renaissance - dead tangent - phosphene blur - birth breeds death - ________ - painted glass - lagniappe

<Exophase> HES STEALING MAH AIRSHIP!!!!!!11111111


Page 1 of 1  
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Replies (1 - 6)

#2 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,949
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 19 June 2015 - 07:01 PM

Updating status to: Confirmed
Updating severity to: 3 - Medium

I've noticed this too.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#3 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,949
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 18 May 2017 - 10:23 PM

The X button and CTRL+C (Windows, Git Bash) both generate an escape press that is never released. Normally it wouldn't have mattered that this sticks around, since games can't use escape anyway, but in this case it keeps the release check waiting for a release that will never arrive:

event_sdl.c, line 178:
    case SDL_QUIT:
    {
      // Stuff an escape
      status->key = IKEY_ESCAPE;
      status->keymap[IKEY_ESCAPE] = 1;
      status->keypress_time = get_ticks();
      break;
    }

"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#4 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,949
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 18 May 2017 - 11:24 PM

Updating status to: Fixed
Issue fixed in: GIT

Fixed in GIT d00cd5e. I reimplemented the key repeat protection from GIT 4cd886b using the tools already available from event.c because my old version was embarrassing. The protection doesn't lock up MZX anymore; it just ignores certain key and mouse presses that have been held.

A side note/separate issue: KEY1 is set to 1 when holding escape after closing a dialog, or after pressing the X or using CTRL+C until escape is pressed again.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#5 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,949
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 18 May 2017 - 11:40 PM

Updating status to: Confirmed

Moving this back to Confirmed so I don't forget about the KEY1 issue.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#6 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,949
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 19 May 2017 - 04:06 AM

Updating status to: Fixed

Fixed the KEY1 issue in GIT b5aa3bd. The SDL_QUIT event does not trigger an escape key press anymore, but will still act the same as it.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#7 User is offline  
Terryn 

  • ******
  • Group: DigiStaff
  • Posts: 2,966
  • Joined: 12-October 00
  • Gender:Male

Posted 29 June 2017 - 07:07 AM

Moving to: Archived MegaZeux Bugs


Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

0 User(s) are reading this issue
0 Guests and 0 Anonymous Users


Powered by IP.Tracker 1.3.2 © 2025  IPS, Inc.