Skip to content

Conversation

@MoonPadUSer
Copy link
Contributor

The veto button was forgotten in our new map selector menu, thus I added it back in:

first

second

third
No worries, the screenshot was just taken at a bad moment, the Force button is actually blinking like it did before

Comment on lines +779 to +782
show_force_button = 0
if (&& (getclientpriv $getclientnum $vetolock) (=s (getmastermode 1) "veto")) [
show_force_button = 1
]
Copy link
Contributor

Choose a reason for hiding this comment

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

If you are showing the force button, then you probably shouldn't have the normal vote button which also forces. Perhaps have a button to force a new vote on the selection screen and a button to force your current vote on the votes screen, or a similar arrangement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

but... the normal Vote button doesn't force?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, it does, if mastermode is set to veto and you have veto privs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh... well that smth we could change, Imo you should still be able to normally vote, right?

Copy link
Member

Choose a reason for hiding this comment

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

No, why should you? veto mode is something rarely used anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the question is: why should we NOT do it

@TheAssassin TheAssassin modified the milestones: 1.6.1, 1.7.0 Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants