wizzomafizzo / mrext

Collection of extensions and utilities for the MiSTer system.
GNU General Public License v3.0
173 stars 11 forks source link

NFC Menu #66

Closed sigboe closed 1 year ago

sigboe commented 1 year ago

This pull request contains a gamepad friendly NFC Menu that includes but is not limited to:

main

This script is close to being ready for wider testing. I can't test directly on real hardware I don't have an NFC reader to do testing with. In this Pull Request, it doesn't need to last so long that the menu gets feature complete, or 100% bug free, or completely finished. I invite anyone to review code, test the code, give their opinions, explain their use cases, come with feature requests, and help out with the tasks below. This PR will be ready to be merged after the following tasks are completed:

When these are either completed, or made a deliberate decision to postpone them to after wider testing, this can be merged. Keep in mind, regard all english language texts as "programmer art"

If you are not accustomed to using Git or GitHub but would like to review examples of the English language used for a dialog box, follow these steps:

  1. Click on "Files changed."
  2. You will see a "diff" of the files changed, and since this mainly adds one large file, you will see the entire file with a green background.
  3. Scroll down and find any English language text enclosed in quotes, as seen in the example on line 467.
  4. Click the plus button to add a comment. This will create a discussion thread here on the main page of this pull request, making it easy for us to track and resolve any comments made in this manner.
sigboe commented 1 year ago

I revamped the settings menu, giving you the ability to set all documented settings image The connection string setting has presets for all documented hardware, or gives you the ability to write in a setting via the keyboard. @wizzomafizzo Feel free to give me an arbitrary number of new presets for this menu, as its trivial to just add more image

These text descriptions should probably me shorter, suggestions welcome

sigboe commented 1 year ago

This small update to the PR, is mainly fixing bugs in the Mapping database editor, and improving an error in the command palette. There is one main new feature, actually giving you the ability to delete an entry from the Mappings database! I also made the menu slightly easier to read

sigboe commented 1 year ago

This small update to the PR, is mainly fixing bugs in the Mapping database editor, There is one main new feature, now displays matches in the Mappings database when reading a tag. So you know if an override will occur!