TTT-2 / TTT2

Trouble in Terrorist Town 2 for Garry's Mod (gmod)
https://steamcommunity.com/sharedfiles/filedetails/?id=1357204556
178 stars 74 forks source link

Missing tooltips on some weapons #1151

Closed ghost closed 8 months ago

ghost commented 10 months ago

Your version of TTT2 (mandatory)

Describe the bug (mandatory)

There are weapons that don't have a tooltip while holding them

To reproduce

Steps to reproduce the behaviour:

  1. Pull out crowbar
  2. See tooltip
  3. Pull out Magneto-stick
  4. No tooltip

Expected behaviour

Weapons/equipment should all have tooltips

Context (please provide as much as you can)

Screenshot from 2023-12-07 20-09-20 Screenshot from 2023-12-07 20-09-31

Other weapons that don't have, or are not fully tooltipped:

Only weapons that currently have a file in the ttt2

Implmented weapons

Had a look at adding base ttt weapons and adding the tooltips

https://github.com/TTT-2/TTT2/commit/8e4ba8d43eab594f4e4042a19ecef89f938cbf79 should resolve for the C4 image

TimGoll commented 10 months ago

Thanks, this is a really good issue. A few notes

TimGoll commented 10 months ago

RE C4: probably "Throw C4" and "Stick C4 to surface" would be better suited

ghost commented 10 months ago
* if primary and secondary do the same thing we only show the primary to hide redundancy

would it be possible to have an icon that indicates both?

TimGoll commented 10 months ago
* if primary and secondary do the same thing we only show the primary to hide redundancy

would it be possible to have an icon that indicates both?

probably? I can try to make one, but it is probably not that easy to visualize

ghost commented 10 months ago
* if primary and secondary do the same thing we only show the primary to hide redundancy

would it be possible to have an icon that indicates both?

probably? I can try to make one, but it is probably not that easy to visualize

could something like this work? if you made them red? 288708111-13890931-a0d3-4bdc-baca-4442b4107bc6

TimGoll commented 10 months ago

No, we don't want color in these icons

saibotk commented 10 months ago

Just chiming in to say: Very nice issues and PR descriptions with screenshots! Really appreciate this, makes it 10x easier to talk about, thank you!

ghost commented 10 months ago

Whatever happened to the magneto-stick tooltips from the screenshot in this (https://github.com/TTT-2/TTT2/pull/1103) pr? And also support for extra keys? like 'R' in the magneto-stick screenshot?

TimGoll commented 10 months ago

Whatever happened to the magneto-stick tooltips from the screenshot in this (#1103) pr? And also support for extra keys? like 'R' in the magneto-stick screenshot?

This is the "advanced magneto stick" addon

Histalek commented 8 months ago

This was fixed in #1162