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

Unified GUI updates, part 4 #244

Merged
merged 13 commits into from
Aug 25, 2023
Merged

Unified GUI updates, part 4 #244

merged 13 commits into from
Aug 25, 2023

Conversation

Lowie375
Copy link
Member

@Lowie375 Lowie375 commented Aug 19, 2023

Implements many of the remaining outstanding features in #231 and #234, patches some known bugs, and makes some GUI-related quality of life improvements

* Update skybox mod
* Add realm helpers for applying/saving skyboxes
* Add realm skybox getter
* Prevent unnecessary spawn updates when editing realms
* Groups are current saved per-player - I am currently considering adding a way to export/share groups for convenience
* Move list swap button textures from mc_tutorial to mc_teacher
* Fix crash when using the bring action
* Players can now only be students, teachers, or administrators
* Add additional player list refresh calls
* Update priv requirements for using student tools
  * Student notebook: no privs
  * Tutorial book: interact
* Add new placeholder icon to player list in teacher tool
* Add getter function for realm owners + owner string converter
* Add [SPAWN] prefix to the spawn realm
* Show owners of private realms dynamically instead of appending to name on realm creation
  * This will not change the names of private realms created before this change - their names will need to be edited manually
* Adapted from actions.lua in rubenwardy's classroom mod (https://gitlab.com/rubenwardy/classroom/-/blob/master/actions.lua)
* Abstract general realm joinability checks into helper function
* Move sounds to sounds/music so that "teleport" isn't treated as music
* Add new helpers for getting/setting music
@Lowie375 Lowie375 marked this pull request as ready for review August 23, 2023 20:54
@Lowie375
Copy link
Member Author

Opening for review to merge critical patches and complete current GUI implementation. The remaining features in #234 will be added in a future PR

@Lowie375 Lowie375 requested a review from pauldpickell August 23, 2023 20:55
@pauldpickell pauldpickell merged commit ec49d78 into develop Aug 25, 2023
@pauldpickell pauldpickell deleted the features/unified_gui branch August 25, 2023 16:13
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