endless-sky icon indicating copy to clipboard operation
endless-sky copied to clipboard

Make the Ship Info cargo column scrollable, or add as a separate tab with columns

Open opusforlife2 opened this issue 1 year ago • 12 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Describe the bug

~~If there are too many kinds of cargo items on a ship, scanning it causes the popup to go out of bounds of the display. For me, the maximum number of viewable items is 31. I assume that with larger or smaller display/resolution combos, this threshold could be different.~~

~~I encountered this for an outfit scan too quite a while ago. The memory is vague, but it might have been the Heron, not sure. It's somewhat easier to encounter this because of the headers separating the different categories.~~

~~While testing, I realised that~~ the Ship Info cargo list just straight-up omits items if they can't fit in the given space.

Steps to Reproduce

Scan ship with really large cargo/outfit list, or, more reliably:

  1. Have a really-tiny-cargo-space flagship, and a massive cargo escort.
  2. Put 1 of as many different kinds of cargo as you can in it.
  3. Scan it.

Also, open Ship Info screen and view the cargo column. Note the missing items.

Expected Behavior

~~The popup list should be scrollable. Or, the list should occupy 2 or more columns if it overruns a set percentage of display height.~~

I don't think there's anything else that can be done for the Ship Info cargo list except making it scrollable. Or, maybe, the entire cargo list could be shifted into a 3rd tab, separate from Player Info and Ship Info. This would allow multiple columns here too.

Screenshots

~~I tried taking a screenshot of the popup, but apparently the game has a 'security feature' where the screenshot always turns out to be of the pause menu. ಠ_ರೃ~~ see https://github.com/endless-sky/endless-sky/issues/10598#issuecomment-2416040679

Link to save file

No response

Operating System

W10

Game Source

GitHub Releases

Game Version

0.10.8

Additional Information

No response

opusforlife2 avatar Oct 15 '24 05:10 opusforlife2

Dialogs are scrollable in 0.10.9, including the scan dialogs.

warp-core avatar Oct 15 '24 08:10 warp-core

New scroll bar for various UI elements.

Damn it. xD

Edit: What about the Ship Info cargo list?

opusforlife2 avatar Oct 15 '24 19:10 opusforlife2

Edit: What about the Ship Info cargo list?

You're right, that doesn't have scrolling yet.

warp-core avatar Oct 16 '24 00:10 warp-core

the screenshot always turns out to be of the pause menu

Any idea what's up with this? Should I open a new issue for this?

opusforlife2 avatar Oct 16 '24 00:10 opusforlife2

It's not intentional, and I never had this happen. We don't even show the pause screen if you tab out, so I'm not sure what can cause this. It should only show up when you press Escape (and you can't do that on the player info panel directly, I think?)

tibetiroka avatar Oct 16 '24 04:10 tibetiroka

So what I was doing was: PrntScr on the scan popup for the screenshot, then pause, Alt+Tab to another software like Paint or an image viewer, and paste.

Kept pasting the Pause screen every time. Do these steps work correctly for you?

opusforlife2 avatar Oct 16 '24 04:10 opusforlife2

I'm on linux, but it works correctly here.

tibetiroka avatar Oct 16 '24 05:10 tibetiroka

Thanks. I'll see if I'm able to debug this.

opusforlife2 avatar Oct 16 '24 05:10 opusforlife2

Works alright on a Mac.

TheGiraffe3 avatar Oct 16 '24 06:10 TheGiraffe3

related: #7868

So what I was doing was: PrntScr on the scan popup for the screenshot, then pause, Alt+Tab to another software like Paint or an image viewer, and paste.

Kept pasting the Pause screen every time. Do these steps work correctly for you?

When you alt-tab back to the game, does it show the menu, or is it only on the screenshot?

TomGoodIdea avatar Oct 16 '24 08:10 TomGoodIdea

If I leave the game at the Pause menu, it's at the menu. If I leave it at some other screen, and switch back, it stays on that screen as expected.

However, the screenshot is still of the Pause menu somehow, despite never switching to it. Basically, the Pause menu is screenshot-bombing me.

Edit: Figured it out with some reading over at the MS answers forum. As it turns out, when Windows switches to the game in full screen, that exact moment is what gets captured in the screenshot that follows, even if it is taken a long time later. So what you have to do is: ensure that the exact thing you want to screenshot is on display in front of you, switch away from and back to the game once, and then take a screenshot.

opusforlife2 avatar Oct 16 '24 08:10 opusforlife2

~~sounds like something windows could do~~

TomGoodIdea avatar Oct 16 '24 09:10 TomGoodIdea