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

🚧Replace all UI controls with IMGUI #860

Merged
merged 47 commits into from
Jan 15, 2024

Conversation

CalvinWilkinson
Copy link
Member

@CalvinWilkinson CalvinWilkinson commented Jan 3, 2024

Description:

To allow this pull request to be merged, please make sure that the following items of this pull request are complete.

This pull request closes #859

✅The pull request head branch must be a feature branch with the syntax 'feature/-sync-testing'.
✅The pull request base branch must be a 'main' or 'preview' branch..
✅The pull request head branch contains a valid issue number.
✅The pull request title matches the linked issue title exactly.
✅The pull request assignees match the assignees of the issue.
✅The pull request labels match the labels of the issue.
✅The pull request organizational projects match the organizational projects of the issue.
✅The pull requests milestone matches the milestone of the issue.


Additional Info:

  1. Pull requests are automatically synced with the associated issue upon creation by one of the KinsonDigital workflow bots.
  2. The associated issue is the issue number that is embedded in the pull request head branch.
  3. The list above will be automatically updated as the pull request's various settings match or do not match the associated issue.
  4. To manually sync the pull request to the issue, create a comment with the [run-sync] command.

    Note You must be an admin member of the organization to use this command.


  • Sync with the issue. Use this to enable or disable this pull request from syncing with its associated issue.

@CalvinWilkinson CalvinWilkinson changed the title new pr 🚧Replace all UI controls with IMGUI Jan 3, 2024
@CalvinWilkinson CalvinWilkinson added tech-debt Code refactoring or cleanup / tech debt reduction high-priority High Priority labels Jan 3, 2024
@CalvinWilkinson CalvinWilkinson self-assigned this Jan 3, 2024
@CalvinWilkinson CalvinWilkinson added the 🧨breaking-changes Feature/changes contains breaking changes label Jan 9, 2024
@CalvinWilkinson
Copy link
Member Author

CalvinWilkinson commented Jan 9, 2024

Warning

Introduced the following https://github.com/KinsonDigital/Velaptor/labels/%F0%9F%A7%A8breaking%20changes

  1. Set the maximum size of fonts to be 100. This is technically a breaking behavioral change, but it is anticipated that most people will never render a font with such a large size so the amount of people is expected is to be very small.
  2. Removed the AddControl and RemoveControl methods from the IScene interface and SceneBase class.

@CalvinWilkinson CalvinWilkinson added the ✨new-feature New feature that does not exist label Jan 9, 2024
@CalvinWilkinson
Copy link
Member Author

CalvinWilkinson commented Jan 9, 2024

Important

Added the following https://github.com/KinsonDigital/Velaptor/labels/%E2%9C%A8new%20feature 's

  1. Changed the following CornerRadius properties from readonly to readonly init which will allow the use of the with keyboard:
    • TopLeft
    • TopRight
    • BottomRight
    • BottomLeft
  2. Integrated a runtime stats window into the core of Velaptor. This window can be shown by using the keyboard key combination Ctrl + Shift + Alt + S.

@CalvinWilkinson CalvinWilkinson force-pushed the feature/859-replace-controls branch from 1052c3a to 99b8c17 Compare January 11, 2024 17:44
@CalvinWilkinson CalvinWilkinson force-pushed the feature/859-replace-controls branch from 99b8c17 to 36fb661 Compare January 14, 2024 16:42
@CalvinWilkinson CalvinWilkinson force-pushed the feature/859-replace-controls branch from 36fb661 to a6bdcbd Compare January 14, 2024 22:20
@CalvinWilkinson CalvinWilkinson added this to the v1.0.0-preview.32 milestone Jan 14, 2024
@CalvinWilkinson CalvinWilkinson marked this pull request as ready for review January 14, 2024 22:21
@CalvinWilkinson CalvinWilkinson merged commit d5f119d into preview Jan 15, 2024
26 checks passed
@CalvinWilkinson CalvinWilkinson deleted the feature/859-replace-controls branch January 15, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧨breaking-changes Feature/changes contains breaking changes high-priority High Priority ✨new-feature New feature that does not exist tech-debt Code refactoring or cleanup / tech debt reduction
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

🚧Replace all UI controls with IMGUI
2 participants