Skip to content

Various - Reorganize setting categories#11412

Open
Dystopian wants to merge 1 commit into
acemod:masterfrom
Dystopian:move_settings_to_subsettings
Open

Various - Reorganize setting categories#11412
Dystopian wants to merge 1 commit into
acemod:masterfrom
Dystopian:move_settings_to_subsettings

Conversation

@Dystopian

Copy link
Copy Markdown
Contributor

When merged this pull request will:

  • add Equipment category;
  • move some categories to Equipment, Map, Uncategorized, User Interface, Vehicles, Weapons.

Can move more settings to categories, but not sure if needed/appropriate:
Interaction
Dragging
Sitting

Weapons
Adv Throwing

Units (new category)
Captives
Respawn
Switch Units

Equipment
Scopes
Trenches
Tagging
Wardrobe

Sound (new category)
Hearing
Volume
No Radio
Common - Allow turning down music

User Interface
Common - icon + progress bar + epilepsy settings

Logistics
Pylons

Vehicles
Fast-roping

Effects (new category)
Cook-off
Field Rations ?
Fire
Fragmentation Simulation
G-Forces
Nightvision (Equipment ?)
Hit Reactions
Overheating (Weapons ?)
Overpressure
Pointing ?
View Distance Limiter ?
View Restriction ?
Weather ?
Wind Deflection ?

@SpicyBagpipes

Copy link
Copy Markdown
Contributor

Fan of reducing the number of categories, but, in my opinion:

Would put logistics & cookoff under vehicles

Would keep common as it is (i.e rename user interface to common)

Overheating to weapons for sure

Need a better changelog of what categories are now removed though

@Dystopian

Copy link
Copy Markdown
Contributor Author

Still not sure whether this PR fits ACE. No problem with changelog and moving more settings into categories, but I'd like some confirmation that this change is worth it.

It's also worth noting that we can't move many settings into a single category because of performance issues. Some addons already have their own subcategories (e.g. Map Gestures - Colors), so those settings don't seem movable either.

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.

2 participants