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

Migrate UI to a real UI framework #402

Closed
19 of 20 tasks
chreden opened this issue Nov 29, 2018 · 2 comments · Fixed by #883
Closed
19 of 20 tasks

Migrate UI to a real UI framework #402

chreden opened this issue Nov 29, 2018 · 2 comments · Fixed by #883
Assignees
Milestone

Comments

@chreden
Copy link
Owner

chreden commented Nov 29, 2018

Migrate to ImGui

Tasks:

Remaining tests:

  • Camera Controls #886
  • Camera Position
  • Console
  • Go To
  • Level Info
  • Items Window
  • Rooms Window
  • Route Window
  • Triggers Window
@chreden chreden added the Needs Review Needs refinement label Dec 9, 2019
@chreden
Copy link
Owner Author

chreden commented Jan 18, 2022

This might be better and is far smaller: https://github.com/ocornut/imgui

@chreden chreden added ui in progress and removed Needs Review Needs refinement labels Jan 18, 2022
@chreden chreden self-assigned this Jan 18, 2022
@chreden
Copy link
Owner Author

chreden commented Jan 18, 2022

Might be useful: ocornut/imgui#269

chreden added a commit that referenced this issue Jan 19, 2022
To experiment with as a UI replacement
#402
chreden added a commit that referenced this issue Jan 19, 2022
chreden added a commit that referenced this issue Jan 19, 2022
chreden added a commit that referenced this issue Jan 21, 2022
chreden added a commit that referenced this issue Jan 21, 2022
chreden added a commit that referenced this issue Jan 21, 2022
chreden added a commit that referenced this issue Jan 21, 2022
chreden added a commit that referenced this issue Jan 21, 2022
chreden added a commit that referenced this issue Jan 22, 2022
Also put nav selection into triggered by.
#402
chreden added a commit that referenced this issue Jan 22, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
Add docking branch of ImGui from commit 3e0d6ec895483e33a663d3cfc1c58f1cafbe978c
#402
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Jan 23, 2022
chreden added a commit that referenced this issue Feb 16, 2022
chreden added a commit that referenced this issue Feb 16, 2022
chreden added a commit that referenced this issue Feb 17, 2022
Finish go to room tests
Split input into click, type, press
#402
chreden added a commit that referenced this issue Feb 17, 2022
chreden added a commit that referenced this issue Feb 18, 2022
chreden added a commit that referenced this issue Feb 18, 2022
chreden added a commit that referenced this issue Feb 18, 2022
chreden added a commit that referenced this issue Feb 18, 2022
chreden added a commit that referenced this issue Feb 19, 2022
chreden added a commit that referenced this issue Feb 20, 2022
chreden added a commit that referenced this issue Feb 20, 2022
Route window tests
Also remove old hack in application di
#402
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 21, 2022
chreden added a commit that referenced this issue Feb 22, 2022
Previously only one of the values was being converted to radians.
#402
chreden added a commit that referenced this issue Feb 22, 2022
This PR removes the existing homemade trview UI system and replaces it with Dear ImGui which was added in a previous PR. Almost all functionality was retained and a few extra bits have come for free. This should hopefully make new features be made faster as there won't be as much fiddling around with the UI.

Changes:
- Remove `trview.ui`, `trview.ui.render`, `trview.ui.tests` and all contained files
- Convert all UI elements to use ImGui
- Create something resembling a testing framework for ImGui - this lets the tests click on element and check some of their properties. This uses the ImGui testing hooks plus a couple of others that I added.
- Convert all tests in `trview.app.tests` to test ImGui using the system mentioned previously.

Closes #402
@chreden chreden added this to the Next milestone Mar 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant