Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Station Police view has tabs: 1. crime records, + new: 2. rules & regulations #5863

Merged
merged 4 commits into from
Aug 23, 2024

Conversation

impaktor
Copy link
Member

@impaktor impaktor commented Jul 12, 2024

I found this 4 year old branch. I figured I could fix it up and PR it.

This aims to show more clearly what rules the player need to be weary of. Changes:

  • Put existing police stuff in "Crime record" tab
  • New tab, lists legal offenses, and the fine at the station

(Never mind my Evarchart facemod)
2024-07-12-113745_1295x659_scrot
2024-07-12-113757_1280x648_scrot

@impaktor
Copy link
Member Author

impaktor commented Jul 12, 2024

thoughts:

  • I'm not sure fontsize is correct.
  • Could also add illegal goods as well.

@impaktor
Copy link
Member Author

Like so
2024-07-12-143455_1286x658_scrot

@Web-eWorks
Copy link
Member

Will review this after the UI theme PR is merged. I like what I'm seeing here, but some areas could have improved formatting - I'll see if I can address that.

@zonkmachine
Copy link
Member

Looks nice! I would consider sorting the lists.

@impaktor
Copy link
Member Author

rebased to new theme on master. No changes beyond that....
(policeman have not mouth.... and he must scream)
2024-08-22-132058_1372x698_scrot

@Web-eWorks Web-eWorks self-requested a review August 23, 2024 03:42
Copy link
Member

@Web-eWorks Web-eWorks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to be cheeky and approve my own changes here too - made stylistic and textual changes as needed for a uniform standard of quality across the entire tab. This PR is definitely a good change, but the whole station view UI is desperately in need of a redesign in the long term.

Final(ish) appearance:

image

image

image

@Web-eWorks
Copy link
Member

Web-eWorks commented Aug 23, 2024

And yes, the faces are different in each screenshot because every time you do a debug reload of the police tab, the station gets a different constable 😄

impaktor and others added 4 commits August 23, 2024 09:48
- If a transformer function returned a modified key, that key was accidentally fed back into the next call to the underlying iterator
- Fix occurrences of two-space indentation / mis-indentation
- Render crime lists as tables
- Use ui.tabBarFont() for consistent styling of the tab bar
- Show fine cost on payment button and change state based on player monetary state
- Sort criminal records by count of crimes committed
- Sort list of illegal commodities alphabetically
- Add introductory text to illegal commodity list
- Tweak rules/regulations text to mention that the distance pertains to laws enforced within the station's sensor coverage.
@impaktor impaktor merged commit a819636 into pioneerspacesim:master Aug 23, 2024
@impaktor impaktor deleted the rules_explicit branch August 23, 2024 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants