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

Team 2 UI - Progress merge #258

Merged
merged 46 commits into from
Oct 15, 2023
Merged

Team 2 UI - Progress merge #258

merged 46 commits into from
Oct 15, 2023

Conversation

The-AhmadAA
Copy link
Contributor

@The-AhmadAA The-AhmadAA commented Oct 14, 2023

This merge brings in improvements to in-game UI and effects, specifically:

  • Screen shake - This is an effect to be used for adding feel to boss encounters, where a boss entering or performing a heavy attack causes a screen shake to be applied to the map grid, background and entities in the game
  • Tile highlighting improvements - tile highlighting now only occurs when a tower is selected for placement, and will use colour to indicate the validity of the action (Green for valid tower placement, red otherwise)
  • Wave timer improvement - The wave timer now only appears between waves and disappears once the next wave has started
  • Pause menu fixes - The pause menu is now implemented as a Window, which prevents the buttons from escaping. Inputs outside the pause menu now do not register. Pause menu positioning is now dependent on screen size and should center automatically.

General UI improvements

  • The skin file has now been updated with an easier to read font that was voted on by the design committee, and team 3's ButtonFactory class has been updated to use this.
  • Buttons now have padding to improve the look and white space around text.
  • Buttons have been updated throughout the project to use this new configuration.
  • Currency display now flashes when a build is attempted with insufficient funds.
  • EngineerCount display and Tower build buttons now have tooltip pop ups

Please feel free to ask questions and provide feedback as necessary, thanks!

AlasdairS4698737 and others added 30 commits October 10, 2023 20:55
…ntity's 4 button components into a single one.
…tead of the edge, so they should stop departing from the pause menu when the screen size changes.
…d in fixing the pause menu screen sizing bug.
removed second declaration of GdxGame
Copy link
Contributor

@BlairCannon97 BlairCannon97 left a comment

Choose a reason for hiding this comment

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

Changes look good, great list of enhancements!

@Mohamad11Dab Mohamad11Dab added this to the Sprint 4 milestone Oct 14, 2023
Copy link
Contributor

@meganroxburgh meganroxburgh left a comment

Choose a reason for hiding this comment

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

All looks good! I agree the game area ui should show the name of the level!

@The-AhmadAA
Copy link
Contributor Author

All looks good! I agree the game area ui should show the name of the level!

Ok, I've added this to my list of To-dos, will implement and should come through in the next PR. Thanks!

@The-AhmadAA The-AhmadAA merged commit dc23cca into main Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sprint 4 team 2
Projects
Development

Successfully merging this pull request may close these issues.

6 participants