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

C4: Reworked UI #1734

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

C4: Reworked UI #1734

wants to merge 3 commits into from

Conversation

TimGoll
Copy link
Member

@TimGoll TimGoll commented Jan 21, 2025

Currently done:

  • basic UI element
  • press&hold buttons to replace the old weird 2 stage button to destroy the entity

fixes #1096

@TimGoll TimGoll added the type/enhancement Enhancement or simple change to existing functionality label Jan 21, 2025
@TimGoll
Copy link
Member Author

TimGoll commented Jan 29, 2025

image

some progress with the UI. While it is far from done, my general idea is already there:

  1. A text box on top that explains some things that can be ignored once you know how it works
  2. The time slider is replaced by a preset of 5(?) predefined times. Makes it easier to select and define differences
  3. the area below will show the effects of the selected time on the explosion in a visible pleasing manner; right now my idea was to have the following effects:
    • volume of C4 beeper
    • amount of wires that lead to instant explosion on defusal attempt
    • amount of fire spawned around C4
    • C4 killradius
  4. These values should be balanced in such a way that no C4 is inherintly better than the other, only situational differences

@TimGoll TimGoll mentioned this pull request Jan 29, 2025
@MrXonte
Copy link
Contributor

MrXonte commented Jan 29, 2025

  1. While i like removing the clonky slider, it did allow you to set the timer close to important round progress points like roundtime and overtime end. And depending on your server config there can be huge differences in max/min time available.
    Would it maybe make sense to instead have a Time with a -30s and +30s button next to it? This is a granular enough increment i think and you can still clearly show the effects from the increase 🤔

  2. Great Idea! Its probably a huge pain to do, but it would be amazing if the kill radius would be shown visually, like this:
    image
    So showing a person as a size reference, and the a half circle for the radius? (bigger radius = smaller reference person)

  3. What exactly do you mean here? I assume something like more time -> range++, defuse difficulty++, offset by beep volume++ and the increased time to find it? or did you have a more complex interaction in mind?

@TimGoll
Copy link
Member Author

TimGoll commented Jan 29, 2025

  1. While we can make the times dynamic, your +/- 30s remove the possibility to define a handful scenarios. Allowing for dynamic time changes defeats this purpose
  2. I honestly wanted to make this waaay simpler. Something like "Radius: 20m" in a neatly rendered way
  3. yeah, something along those lines, I have not yet figured this out

@Crashington
Copy link

  1. Hmm, personally, I am not sure about the complete removal of the slider as an option. I see where you're coming from but personally greatly appreciate the exact control over the timer.
  2. I like it just being displayed as a readable distance in irl units. Is an F1 menu toggle to switch between meters and feet needed :D?
  3. A longer C4 should be quieter, but should it maybe scale higher? I can see arguements for basically all implementations of volume scaling/levels i can think of.

I think personally I would make it so there are only two or three discrete levels. the normal beeps, which are only audible within the same room. Then the beeps withing the last minute or so, where they should be audible in a few rooms over. and finally the the last 30 seconds or so, where it gets as loud as it gets currently. I think putting it in discrete stages like this makes it easier for (trained) innocents to understand where they are at, and how to react, while it should at the same time make it a bit more tense when players immediatly notice "oh we are already in the endgame"

I am not sure how to scale this to the fuse time though. if this should all be based on percents of the chosen of time, or if shorter bombs should immediatly start in the later levels or what

@TimGoll
Copy link
Member Author

TimGoll commented Jan 29, 2025

  1. I personally hated the slider because nobody really bothers to set an exact time anyway (I watched quite a few videos and everybody just either leaves it at 0:45 or removes it to a random position without any second thoughts; therefore I want a quick to use UI). And probably nobody noticed the correlation between the defuse wires and the time. Having a fixed amount of presets makes that clear. Also I don't get why you have to set it to 2:06 instead of 2 minutes - is there really a difference? Because I'm always trying to achieve to things: simplify the options and explain more.
  2. Currently we only support metric, and I'm not really interested in supporting anything else

@Crashington
Copy link

  1. I personally hated the slider because nobody really bothers to set an exact time anyway

awww, i definitly do that haha. but i get what you mean. I've got to say, I did not entirely get the problem with having discrete 30 second or so steps to choose from. It feels like a great compromise to me. these can still be discretely defined, it would be more a matter of displaying them so that there arent like 10 different options to choose from but you instead cycle through them by pressing the "time up" or "time down" buttons

@TimGoll
Copy link
Member Author

TimGoll commented Jan 29, 2025

There is no real problem, maybe your solution is even the best. My point was: Nobody wants to fine tune their C4, nobody has the time. I want to provide 3 to 5 discrete time stamps that have their own strategic benefits. When having a +-30s button, you have a slider, but worse :D It is even slower than a slider, while providing less granularity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhancement or simple change to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] C4
3 participants