-
Notifications
You must be signed in to change notification settings - Fork 116
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
feat: add 'Crosshair' plugin to DB #745
Conversation
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.
Please remove assets, backend, defaults and py_modules folders as you do not appear to be making use of them. If you are using the existing license for the plugin-template please update it's contents to reflect this. If not, please include the original license info at the bottom and the new license above it (in the same file). Please also remove the root and debug flags from plugin.json.
Once you have made these changes, the PR can proceed.
I am currently experimenting with a concept for testing plugin submissions and plugin updates. I would request that you submit testing reports for at least 2 other plugin submissions/updates (preferably the oldest PRs still active). Then if no other plugin author (as I will be encouraging all others to do the same) or tester submits a testing report I will request that a member of the SDH team tests your plugin update/submission according to the testing guidelines.
@TrainDoctor all adjustments made as requested, and I will do the testing reports for other plugins here shortly! |
@TrainDoctor reviewed 2 plugins; Pause games and web browser. Please let me know if theres anything else needed! |
Please make sure you're reports follow the format specified in the link below: I greatly appreciate you taking the time to make those plugin testing reports. Just need to make sure they include the actual information that's needed for a report to be considered valid. |
Make sure to push your changes to the submodule in your fork of the plugin database, otherwise they won't be picked up properly. |
@TrainDoctor had a kerfuffle, should be good now! |
Testing ReportVersionsDecky: v3.1.2-pre1 SummaryMajor blocking issues
Minor issuesNone Currently installed plugins
Yay/nay on to send to the production store or if it requires more testing.
|
@Tormak9970 if user adjusts stats overlay level at all from QAM it will overwrite crosshairs, even if plugin is uninstalled. Still its not the best user experience, will investigate a remove button |
Testing ReportCurrently installed plugins
Version
Major blocking issue
Minor issue
SummaryYay/nay on to send to the production store or if it requires more testing.Overall, it needs more tweaking before send to the production store:
|
Added remove crosshairs button. Currently I dont have a great way to let performance overlay remain when crosshairs are active, but can look into explaining to the user clearly within the interface |
@wynn1212 added remove button and information within the plugin interface to drive additional clarity |
@Tormak9970, @wynn1212, I'd like to know if the inability for performance overlays to coexist with the crosshairs is a blocking issue in your opinion. Ultimately the plugin author will be deferred to on design opinions but I want to know if you feel that this would be confusing/frustrating for users. Thank you both for your feedback. |
Personally, I think that there should be a disclaimer (maybe at the top of the plugin's QAM menu), indicating what happens if you apply the crosshair. A user could easily get confused if they applied the crosshair and their overlay just dissappears. @xXJSONDeruloXx if the user has a custom overlay does that get restored after the crosshair is disabled or is it lost perminantly? |
@Tormak9970 i added a disclaimer in the plugin ui from your earlier comments. And if the user manually made a config for one of the 4 presets and toggle to it it will use that custom config, as the 'mangohud.xyz' temp file essentially overwrites the crosshair config in favor of "preset = 2" which references config defined elsewhere that crosshair itself does not touch. Crosshair only hijacks the temp file that typically defines the preset, not the config itself Content of disclaimer, open to suggestions for the wording:
|
Ok perfect. In that case @TrainDoctor I think its good to go |
From my perspective, as long as the description and disclaimer are clear, I think it would be acceptable. Without them, users might be confused about why their performance HUD suddenly disappears. |
@Tormak9970 Here’s another suggestion to ensure users are fully aware of the limitation: display a one-time dialog when the user presses the "Make Crosshair" button for the first time. This would help, as the disclaimer is located below the button on an 800p Steam Deck screen and might not be immediately noticeable. |
I agree, that's a good idea, or alternatively move the notice above the buttons and make it more concise. Either way the user wouldn't miss it |
@xXJSONDeruloXx please implement one of the above suggested solutions or something that you feel ensures the user is aware of the interplay of adding the crosshair and the performance overlays (including custom overlays etc) and then this should be good to go imo. |
@TrainDoctor let me know if the merge commit causes issues, I am still new to working with submodules, thanks for all your assistance |
Implemented, thanks! |
The implementation looks good to me. Re-testing now... (EDIT: No major/minor issue found on latest SteamOS 3.7_20250114.1001 [Main], and SteamOS 3.6.20_20241030.1 [Stable], the list of currently installed plugins remains unchanged, even though only this plugin has been updated to the latest version from the testing store. ) |
Thank you for the review! |
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.
Changes look good according to testers. Good for production.
Crosshair Plugin
This plugin overlays a customizable crosshair on the screen for use in games on the Steam Deck. It provides predefined crosshairs for 800p (Steam Deck) and 1080p (external displays, other handhelds) and allows users to adjust the crosshair position dynamically via the plugin UI.
Checklist:
Developer Checklist
Plugin Checklist
Plugin Backend Checklist
Testing
Tested on SteamOS Stable/Beta Update Channel.
Tested on SteamOS Preview Update Channel.