Remove discard functionality from the ChemMaster #34092
Open
+4
−89
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the PR
The ChemMaster no longer has the ability to discard chemicals.
Why / Balance
This duplicates other methods of discarding chemicals, including the drain conventionally given to chemistry labs.
Any problems it solves are miniscule compared to the severe UI error that is having:
In plainer words, the way discard in the UI works encourages irreversible mistakes via a bad UI design, and in enough ways that it would require a restructuring of how the ChemMaster UI works to eliminate these mistakes.
Instead of adding a band-aid solution such as a "Are you Sure?" dialogue or attempting to improve the UI, I took a step back and realised that this functionality doesn't solve many problems for chemists - unlike other features of the ChemMaster that are mostly unique to it at round-start, the discard functionality has a litany of alternatives, including but not limited to:
Recommended Reading On UI Design
As always, the Nielsen-Norman group is the gold standard for introductory articles to these concepts:
Bruce Tognazzini also has some good elaboration on guidelines and examinations of specific examples:
If you're more into published books, Don Norman has an entire chapter on design and user errors in his seminal work The Design Of Everyday Things titled Human Error? No, Bad Design, which opens with:
Technical details
Removed all of the UI, functionality, and translation strings associated with the mode functionality in the ChemMaster.
Media
Requirements
Breaking changes
Changelog
🆑