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

Rename obj.supressButtonReaction to obj.suppressButtonReaction #547

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MasterBel2
Copy link
Contributor

obj.supressButtonReaction will still work for the time being, so this can be merged ASAP without breaking code. A warning will be listed once in the log.

Please remove this concession ASAP.

`obj.supressButtonReaction` will still work for the time being, so this can be merged ASAP without breaking code. A warning will be listed once in the log.

Please remove this concession ASAP.
MasterBel2 added a commit to MasterBel2/BYAR-Chobby that referenced this pull request Jun 26, 2022
- Decrease button & font sizes to give ready / start buttons more prominence
- Use i18n lib for tooltips
- Change tooltip depending on button state

- Add `SetButtonStatePlaying()` and `SetButtonStateSpectating()` to avoid duplication of constants throughout the code.
- Tooltip/caption values set after button init with `SetButtonStatePlaying()`

-  Move `button_highlight` style colours to mirror `option_button` colours

- Makes use of `suppressButtonReaction` to remove hints that the play/spectating buttons are pressable when they aren't (This won't work until Spring-Chobby/Chobby#547 is merged, due to a typo in the source.)
Beherith pushed a commit to beyond-all-reason/BYAR-Chobby that referenced this pull request Jul 1, 2022
* Ready button default to unready; dec font size for ready & start buttons

- Fixes "Not Ready" outsizing ready button on default scaling on 1080p displays
- Fixes ready button being grey with "Button" caption by default

* Update Play / Spectating button styling

- Decrease button & font sizes to give ready / start buttons more prominence
- Use i18n lib for tooltips
- Change tooltip depending on button state

- Add `SetButtonStatePlaying()` and `SetButtonStateSpectating()` to avoid duplication of constants throughout the code.
- Tooltip/caption values set after button init with `SetButtonStatePlaying()`

-  Move `button_highlight` style colours to mirror `option_button` colours

- Makes use of `suppressButtonReaction` to remove hints that the play/spectating buttons are pressable when they aren't (This won't work until Spring-Chobby/Chobby#547 is merged, due to a typo in the source.)
@Beherith
Copy link
Contributor

Beherith commented Jan 9, 2023

Does this one still need a merge? I dunno of other games on chobby base wont suffer from this

FIr3baL added a commit to FIr3baL/BYAR-Chobby that referenced this pull request Apr 30, 2023
leaned on original PR from MasterBel2 for Spring-Chobby
Spring-Chobby/Chobby#547
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