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

Rework UI #102

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Rework UI #102

wants to merge 2 commits into from

Conversation

Deewarz
Copy link
Contributor

@Deewarz Deewarz commented Mar 15, 2024

Global

  • Add (MafiaMP)UIBase (virtual extends from Framework UIBase)
  • Fix control locks increment issues (low level locking functions are only called if necessary (to avoid incrementing the internal game counter))
  • Add a way to bypass control locks when locked and show a message to unlock/relock them (use F1 key)
  • Fix duplicate control unlock when exiting main menu
  • Remove DevFeatures as friend class of Application and use getters instead

Devs UIs

  • Extends from (MafiaMP)UIBase
  • Move devs UIs to a dedicated folder ui/devs/...
  • Rename debug UIs, use debug_ as prefix (eg. audio_debug => debug_audio) to group them
  • Use UIBase custom window

Console

  • Extends from (MafiaMP)UIBase
  • Rename MafiaConsole to Console

Chat

  • Extends from (MafiaMP)UIBase
  • Remove window background and title bar
  • Disable window move and resize
  • Hide scrollbar when unfocused
  • Add font shadow

@Deewarz Deewarz self-assigned this Mar 15, 2024
@Deewarz Deewarz marked this pull request as ready for review March 15, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Rethink debug UIs
1 participant