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

Nicer routes #36

Merged
merged 15 commits into from
Nov 10, 2024
Merged

Nicer routes #36

merged 15 commits into from
Nov 10, 2024

Conversation

Froggy618157725
Copy link
Collaborator

Routes now use heirarchies, letting them use all the other nice things with those.
Added a lot more heirarchy support to other things
More hotkeys :D

Copy link

semanticdiff-com bot commented Jul 25, 2024

Review changes with SemanticDiff.

Analyzed 18 of 19 files.

Overall, the semantic diff is 24% smaller than the GitHub diff.

3 files do not contain logic changes.

Filename Status
CHANGELOG.md Unsupported file format
✔️ crates/alkahest-renderer/src/ecs/common.rs Analyzed
✔️ crates/alkahest-renderer/src/ecs/tags.rs No logic changes found
✔️ crates/alkahest-renderer/src/ecs/utility.rs 43.97% smaller
✔️ crates/alkahest/src/app.rs 1.64% smaller
✔️ crates/alkahest/src/maplist.rs 18.97% smaller
✔️ crates/alkahest/src/util/action.rs 11.91% smaller
✔️ crates/alkahest/src/gui/activity_select.rs No logic changes found
✔️ crates/alkahest/src/gui/configuration.rs No logic changes found
✔️ crates/alkahest/src/gui/console.rs Analyzed
✔️ crates/alkahest/src/gui/crosshair.rs 99.6% smaller
✔️ crates/alkahest/src/gui/hotkeys.rs 24.29% smaller
✔️ crates/alkahest/src/gui/node_gizmos.rs 51.13% smaller
✔️ crates/alkahest/src/gui/menu/mod.rs 14.78% smaller
✔️ crates/alkahest/src/gui/menu/utility.rs 21.3% smaller
✔️ crates/alkahest/src/gui/inspector/decorator.rs Analyzed
✔️ crates/alkahest/src/gui/inspector/light.rs Analyzed
✔️ crates/alkahest/src/gui/inspector/mod.rs 37.31% smaller
✔️ crates/alkahest/src/gui/inspector/util.rs 5.74% smaller

Labels can now have an offset associated with them. This offset is done in
camera coordinates. Utility Objects have been updated Accordingly.
Include nondefault labels as named entities.
Allow for deletion of mutable enitity heirarchies, and support globals
with heirarchies.
This scatters Routes into the winds of the world. Instead of being self contained,
they're now built piecemeal out of other objects so they can take advantage of common
infrastructure, like the handy movement gizmos, and label selection.
You can hit up for parent, down for child, and left/right to go through siblings
There's a few areas this is done, and it's pretty error prone,
so generating them in the same place makes sense. We could do the
same for all Utility Shapes, but I'm starting with this in case we
find a better way to set this up.
crates/alkahest/src/gui/inspector/mod.rs Outdated Show resolved Hide resolved
crates/alkahest/src/maplist.rs Outdated Show resolved Hide resolved
crates/alkahest/src/gui/hotkeys.rs Outdated Show resolved Hide resolved
crates/alkahest-renderer/src/ecs/utility.rs Outdated Show resolved Hide resolved
crates/alkahest-renderer/src/ecs/utility.rs Outdated Show resolved Hide resolved
crates/alkahest-renderer/src/ecs/utility.rs Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
crates/alkahest-renderer/src/ecs/utility.rs Outdated Show resolved Hide resolved
crates/alkahest-renderer/src/ecs/utility.rs Outdated Show resolved Hide resolved
crates/alkahest/src/util/action.rs Outdated Show resolved Hide resolved
crates/alkahest/src/util/action.rs Outdated Show resolved Hide resolved
crates/alkahest/src/util/action.rs Outdated Show resolved Hide resolved
crates/alkahest/src/util/action.rs Outdated Show resolved Hide resolved
@cohaereo cohaereo mentioned this pull request Nov 8, 2024
@cohaereo cohaereo merged commit 81988bc into cohaereo:main Nov 10, 2024
1 check passed
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