New station screens in PR

Post Reply
robn
Posts: 302
Joined: Mon Jul 01, 2013 1:11 am
Location: Melbourne, Australia

New station screens in PR

Post by robn »

If you're in a compiling and testing kind of mood, I'd love to get some eyeballs on PR #2589, the new station screens.

Note that right now I'm looking for technical issues (missing/broken functionality, crashes, other quirks). I'm not so interested in disagreements about layouts and presentation. Not that that's not important but its usually the case that everyone has different but equal opinions, and I'd rather not hold this back while everyone says their piece. You're quite welcome to hack the code later to produce your dream layout - that's half the reason its all moved out to Lua! :)

Screenshots from the PR:

Image

Image

Image

Image

Image

Image

Image
impaktor
Posts: 995
Joined: Fri Dec 20, 2013 9:54 am
Location: Tellus
Contact:

Re: New station screens in PR

Post by impaktor »

Looks very nice.

As mentioned in #pioneer, the "take off clearance" button is missing.

Also, nice feature to have would be:
- Arrow up / down for scrolling lists.
- F4 cycles through the different tabs in "Lobby, Bulletin Board, etc"
- F3 cycles through the different tabs in "Ship Information, Personal information, etc"
robn
Posts: 302
Joined: Mon Jul 01, 2013 1:11 am
Location: Melbourne, Australia

Re: New station screens in PR

Post by robn »

impaktor wrote:As mentioned in #pioneer, the "take off clearance" button is missing.
That's done now, good catch:

Image
- Arrow up / down for scrolling lists.
Needs some input layer support that I haven't fully thought through yet. Some other time.
- F4 cycles through the different tabs in "Lobby, Bulletin Board, etc"
- F3 cycles through the different tabs in "Ship Information, Personal information, etc"
This gets asked for a bit. I think it could be hacked by adding a shortcut to something for that key, but then that thing has to be clickable or magicked to be invisible. But it is needed. I'm going to throw it in the "input" bucket for now.
FluffyFreak
Posts: 1343
Joined: Tue Jul 02, 2013 1:49 pm
Location: Beeston, Nottinghamshire, GB
Contact:

Re: New station screens in PR

Post by FluffyFreak »

Wow I didn't realise you were doing ALL of the station screens! For some reason I thought you were just doing the trade screen first.
shadmar
Posts: 25
Joined: Mon Jul 01, 2013 7:28 pm

Re: New station screens in PR

Post by shadmar »

That looks great!!
FluffyFreak
Posts: 1343
Joined: Tue Jul 02, 2013 1:49 pm
Location: Beeston, Nottinghamshire, GB
Contact:

Re: New station screens in PR

Post by FluffyFreak »

Found a bit of an issue, it insists that my ship is perfectly intact and doesn't need repairs despite it being almost totally ruined :/

Logged as issue https://github.com/pioneerspacesim/pioneer/issues/2598
walterar
Posts: 95
Joined: Sat Sep 14, 2013 4:48 pm

Re: New station screens in PR

Post by walterar »

I think we need a bigger screen for this:
Image
Or more info. @robn Is there a way to resize the screen?

In other screens, client photos require a better frame or extension.
Image
walterar
Posts: 95
Joined: Sat Sep 14, 2013 4:48 pm

Re: New station screens in PR

Post by walterar »

A doubt. 4:3 is still the standard?
robn
Posts: 302
Joined: Mon Jul 01, 2013 1:11 am
Location: Melbourne, Australia

Re: New station screens in PR

Post by robn »

walterar wrote:I think we need a bigger screen for this:
Or more info. @robn Is there a way to resize the screen?
Yeah, this is kind of crappy. It used to be a variable-sized dialog, but that would cause it to change sizes when the text changed, which made it really hard to use. So I forced it to be fixed, which obviously does this with large quantities of text (and I missed it, annoying).

I think the only other option with this style would be to add scrollbars, which would look terrible too. So the plan is to change the BB layout entirely to something more like this (a mockup by nozmajner):

Image
In other screens, client photos require a better frame or extension.
Yeah, the face widget got a change to report its sizing differently to the UI core so I could do things like the really small ones in the chat forms sanely, but I deliberately didin't change anywhere else because the changeset was already too large. I've got some updates to the info screens coming soon to take advantage of some new UI features made for the station screens, so I'll leave it until then.
robn
Posts: 302
Joined: Mon Jul 01, 2013 1:11 am
Location: Melbourne, Australia

Re: New station screens in PR

Post by robn »

walterar wrote:A doubt. 4:3 is still the standard?
No, you cannot assume any screen size or ratio any more.
Post Reply