Skip to content

Conversation

impaktor
Copy link
Member

Do we want this?
I've added it because I can never remember how these buttons work, which might be an indication there's a deeper UI issue.

Anyway, if we want it I can clean this up and add it to localization. Looks like so now:

tooltip.mp4

ping @bszlrd & @sturnclaw

@fluffyfreak
Copy link
Contributor

More tooltips is generally more betterer 😉I think it's helpful

@bszlrd
Copy link
Contributor

bszlrd commented Jul 16, 2025

I agree with Fluffy, the more tooltips, the better.

@sturnclaw
Copy link
Member

Grug say "many tooltip, more knowledge".

(The tooltip should probably say "Pump X t of fuel to/from cargo".)

@impaktor
Copy link
Member Author

OK, done.
Ready for review.

tooltip.mp4

@impaktor impaktor marked this pull request as ready for review July 16, 2025 18:14
@Bodasey
Copy link

Bodasey commented Jul 19, 2025

Any other open source user here who can not watch these videos? (file corrupt)

I've added it because I can never remember how these buttons work, which might be an indication there's a deeper UI issue.

Yes, these arrows are contra-intuitive...some suggestions how it might be better? These arrows always indicate the direction of fuel flow:

Screenshot_20250719_164720 fuelflow2

I have added the signs +- as suggestion in the second draft to have the same look as in the commodity market, but then, the text was better "from Cargo bay".

@craigo-
Copy link
Contributor

craigo- commented Jul 19, 2025

I like the numbers on the buttons, I hope they remain.

Suggestion:

  • Place a single "<" (or another existing arrow equivalent) before the word "Refuel" to indicate direction
  • Place a single ">" (or another existing arrow equivalent) before the phrase "Pump down" to indicate direction
  • Keep the numbers on the buttons, but remove the negative sign for the "Pump down" options

@impaktor impaktor merged commit 9dfb47e into pioneerspacesim:master Aug 23, 2025
@impaktor impaktor deleted the tooltips branch August 23, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants