ReactiveDrop / reactivedrop_public_src

Alien Swarm: Reactive Drop game source code and issue tracker.
https://reactivedrop.com
109 stars 35 forks source link

Namings of newly added inventory items #517

Open Duke-M-commits opened 1 year ago

Duke-M-commits commented 1 year ago

After adding the new inventory items to my profile I realized the popup doesn't show tags: image As opposed to the item description in the inventory (marked in red): image Thus, I propose to implement the items rarity into the items display name (marked in green).

Also: Since the items include medals, stickers as also fun items (like the toilet) I would propose to name them Collectibles instead of Medals. In this case it would be named Common Collectible.

Here's an example of how that looks for an cs:go item. image

I personally see the ingame promotions more fitting to the name medals as they are given to the commander for playing for a longer time and getting more skilled. I say that just for clarification, I don't think there should be renamings in that part of the game.

BenLubar commented 1 year ago

The reason they say "Medal" (or some variant thereof) is that that's the slot you can equip them in. I'm open to changing any of the display_type values as long as it's still easy to tell where the item can be equipped.

We can also rename the medal equip slot to collectable or trinket or something and keep the internal name as medal.

Duke-M-commits commented 1 year ago

The reason they say "Medal" (or some variant thereof) is that that's the slot you can equip them in. I'm open to changing any of the display_type values as long as it's still easy to tell where the item can be equipped.

We can also rename the medal equip slot to collectable or trinket or something and keep the internal name as medal.

I'm not sure who "they" are. Also I can't seem to find a place ingame which really gives the slots a name. I believe with slots you mean the 2 small places in the lobby which show the selected medal from the collections-menu and the current promotion/level of the player. Please clarify.

BenLubar commented 1 year ago

I'll hope this clarifies:

The reason the display_type of most items with item_slot set to medal is some variant of "Medal", "Community Medal", etc. is that the equip slot is medal ( rd_equipped_medal, see rd_inventory_shared.cpp for where they're defined)

Currently medal is the only equip slot that affects what you see in the game in any meaningful way. Weapon equips change the name of a weapon when you try to pick it up to include your Steam name, but nobody has a weapon item because the rest isn't ready for testing yet.

The slot name is shown on the collections screen tab. We can definitely rename the slot to "Collectibles" or "Trinkets" or something else like that (it'll need to keep its internal name of medal to not break stuff due to our system of having two active versions of the game, just like how Mender is called asw_shaman internally).

Duke-M-commits commented 1 year ago

Thanks for the clarification. I hope I find the time until next monday to implement the following renamings in the UI:

Regarding weapon names, giving weapons player names could also be realized as name-tags like this: image

BenLubar commented 1 year ago

What I mean by "player name" is that the message would show "[E] Pick up Ben Lubar's Rifle" rather than just "[E] Pick up Rifle"

Duke-M-commits commented 1 year ago

Reference #752