Add "jettison all" buttons to cargo sidebar menu#5867
Add "jettison all" buttons to cargo sidebar menu#5867sturnclaw merged 1 commit intopioneerspacesim:masterfrom
Conversation
|
OK, forced pushed to use same tooltip as other buttons. |
I am not sure why the original tooltips were passed in but makes sense a bit for re-usability. mt purpose of adding the new tooltips was so that people could disable these buttons. ie
|
Because it leaves open the possibility for more specific tooltips like "increase scan frequency" or "shrink garbage compactor". Anything that's not passed as an argument is hardcoded to a specific value. |
Closes #5860 feature request by @syonfox (I was going to close it anyway, as we don't do feature requests on the issue tracker, but figured I might as well implement it, low hanging fruit).
Note: just appending "all" to the tooltip hardcoded. Dunno if we keep it, and put it in ui-core, or remove. Not obvious to me why the
tooltipsare passed as arguments to this function?