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

Character Height Slider #45

Closed
wants to merge 12 commits into from
Closed

Character Height Slider #45

wants to merge 12 commits into from

Conversation

DEATHB4DEFEAT
Copy link
Member

@DEATHB4DEFEAT DEATHB4DEFEAT commented Jan 28, 2024

Description

Based partially off of the height "slider" PR from Cosmatic Drift
Adds a slider to the character customization UI that changes eye zoom, fixture size, and sprite size (similarly to the old height adjust traits) with a per-species min/max/default set of values.


Media

Displayed sizes may or may not be accurate to the ones available in-game

2024-01-27_21-46-26.mp4
2024-01-27_21-31-14.mp4
2024-01-27_21-41-27.mp4


Changelog

🆑

  • add: Added a slider to the character customization to change the size of your character

@DEATHB4DEFEAT DEATHB4DEFEAT added Size: 3-Medium For medium issues/PRs Priority: 3-Medium Needs to be resolved at some point labels Jan 28, 2024
@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files labels Jan 28, 2024
@DEATHB4DEFEAT DEATHB4DEFEAT mentioned this pull request Jan 28, 2024
75 tasks
Copy link
Member

@Pspritechologist Pspritechologist left a comment

Choose a reason for hiding this comment

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

Bunch of thoughts on this.

Does this work properly yet? On Nyano, any adjustment to zoom ruined species set zoom, and I didn't like that at all. The shuttle console, for instance, would reset your vision to 1.

I think the mob prototypes for species should probably still be scaled by default just for spawning purposes at times when the adjustment won't take place. At the worst it seems like it probably doesn't need to be gotten rid of.

I don't love small species like Felinids being able to be as tall as a human by default. Perhaps combining this with unusually tall and unusually short traits would be ideal?

@Pspritechologist
Copy link
Member

I'm still of the opinion that height should be vertical, and not horizontal :P
Maybe having that be changeable per species would be a good idea?

@DEATHB4DEFEAT
Copy link
Member Author

DEATHB4DEFEAT commented Feb 4, 2024

The shuttle console, for instance, would reset your vision to 1.

I think it still does that, I'll fix that.

I think the mob prototypes for species should probably still be scaled by default just for spawning purposes at times when the adjustment won't take place. At the worst it seems like it probably doesn't need to be gotten rid of.

I think they get scaled to the default of their speciesProtoype when spawned, need to double check.

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Feb 19, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

This is pretty incredible. I went into this wanting to ask some questions, expecting it to maybe have some of the problems that CD's version was. I was pleasantly surprised that everything I had thought to ask about was already addressed. Even game balance by modifying fixtures, mass, and viewpoint.

DEATHB4DEFEAT added a commit that referenced this pull request Jun 2, 2024
Description copied from my Wizard's Den PR
Resolves Simple-Station/Einstein-Engines#32

# TODO

- [x] CVars
- [x] Server side loadout point validation
- Make tabContainer less bad
- [x] Move head clothes to loadouts from lockers
- [x] Move job starting equipment to loadouts
- [x] Loadout item preview
- [x] Fix loadouts duplicating on server restart
- [x] Make sure everything is localized -- Components and Tags are an
odd thing to display to the player in the whitelist and are very
unrealistic to make localizations for, so.. not doing that, what do I
do?
- [x] Fix all items going into the bag no matter their size
- [x] Add min/cur/max points to the bar
- [x] Use buttons instead of checkboxes
- [x] "Show Unusable" button to hide things your character currently
doesn't match the whitelists for
- [x] Time whitelist option
- [x] Species whitelist option
- [x] Trait whitelist option instead of EntityWhitelist for the sake of
localization
- [ ] More loadouts (filler things while waiting for review)
- [ ] - Golden or themed items for Command roles with an undecided
amount of playtime on their respective jobs
- [x] - Goliath cloak for playing a lot of Salvage
- [x] - Senior items for playing a lot of its respective role
- [ ] - Varying materials of pocket watches for people with varying
overall playtime
- [x] Fix loadout selectors not updating to match current preferences

<details><summary><h1>Media (Click Me!)</h1></summary>
<p>

I need to rerecord these


https://github.com/space-wizards/space-station-14/assets/77995199/59713874-b043-4813-848e-56b2951b6935


https://github.com/space-wizards/space-station-14/assets/77995199/40180aee-bfe3-4f30-9df8-0f628e7e4514

</p>
</details> 

# Changelog

:cl:
- add: Added a new character customization tab for selecting items to
start your shift with, loadouts!
- remove: Removed some default job equipment in favor of loadouts
- remove: Removed several clothing items from Command lockers in favor
of loadouts

---------

Co-authored-by: VMSolidus <[email protected]>
DEATHB4DEFEAT pushed a commit that referenced this pull request Jun 2, 2024
@DEATHB4DEFEAT
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files Priority: 3-Medium Needs to be resolved at some point Size: 3-Medium For medium issues/PRs Status: Merge Conflict FIX YOUR PR AAAGH Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants