Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't open menu when set "Enable embedded menu bar" #331

Closed
samo79 opened this issue Mar 21, 2021 · 7 comments
Closed

Can't open menu when set "Enable embedded menu bar" #331

samo79 opened this issue Mar 21, 2021 · 7 comments

Comments

@samo79
Copy link

samo79 commented Mar 21, 2021

I've originally reported it on the iGame repo

MrZammler/iGame#144

Menu does not seems to work anymore in iGame if you set "Enable embedded menu bar" from MUI prefs ... options are showed in the main window but they can't be opened Tested with iGame 2.1 Beta 1 under AmigaOS 4.1 FE (latest MUI 5.0)

@tboeckel
Copy link
Contributor

The menu might be built from Menu objects and Menuitem objects, but Menudisplay checked MUIA_Menu_Enabled only to be true. For Menuitem object this of course is never true and hence the item was considered to be disabled. Now the class is checked before the corresponding enabled state is checked.

Menudisplay_21_27_os3.zip
Menudisplay_21_27_os4.zip

@samo79
Copy link
Author

samo79 commented Mar 22, 2021

Ok tested, but got a crash

Crashlog_iGame.OS4_2021-03-22_17-51-19.txt

@samo79
Copy link
Author

samo79 commented Mar 23, 2021

Crash is reproducible in all MUI applications, i've revert for now waiting for a proper implementation :-)

@tboeckel
Copy link
Contributor

This crash seems to be related to some internal changes since 5.0-2020R3. The location of the crash is before the latest change...

@tboeckel
Copy link
Contributor

We really need to get the nightly build up and running again :(

@tboeckel
Copy link
Contributor

MUI-5.0-20210324-0d266f2-os4.zip

Please try this private nightly build for AmigaOS4.

@samo79
Copy link
Author

samo79 commented Mar 24, 2021

Yeah seems you fixed the issue at 100%, bravo :-)
Ticket can be closed now..

About MUI development, frankly I was felt better with the old system .. apart from not being able to use the Amiga to report bugs (github does not work with Odyssey) it is not clear how we betatesters can follow its progress in details .... once the switch will be completed I hope that at least the periodically updated changelog file could be publicly available for reading/consulting ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants