-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add front panel keypad to Heathkit H8. Added support for serial console. #12053
base: master
Are you sure you want to change the base?
Conversation
src/mame/layout/h8.lay
Outdated
<element ref="btn_7" inputtag="X0" inputmask="0x80"><bounds x="0.20" y="0.125" width="1.0" height="1.0" /></element> | ||
<element ref="btn_8" inputtag="X1" inputmask="0x01"><bounds x="1.25" y="0.125" width="1.0" height="1.0" /></element> | ||
<element ref="btn_9" inputtag="X1" inputmask="0x02"><bounds x="2.30" y="0.125" width="1.0" height="1.0" /></element> | ||
<element ref="btn_a" inputtag="X1" inputmask="0x04"><bounds x="3.35" y="0.125" width="1.0" height="1.0" /></element> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One of the lines here got indented more than the others. Please use one tab per nesting level for indentation, and don’t indent using spaces.
src/mame/layout/h8.lay
Outdated
<element ref="background"> | ||
<bounds left="1" top="65" right="600" bottom="200" /> | ||
<bounds left="1" top="65" right="900" bottom="400" /> | ||
</element> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doing this will cause space for the keypad to always appear even if the user hides the keypad collection. That’s undesirable. Can you do the layout in a way that will avoid that?
Also, 400 seems excessive extra height when the keypad is 200 units high with only a margin of 5 units at the top.
Pull request has been updated to address review comments. |
@cuavas Please review the revised pull request when you get a chance. |
Made additional changes to support serial console. |
Adds a graphical front panel keypad to the Heathkit H8.