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

Allow disabling of systems by right clicking #111

Merged
merged 11 commits into from
Aug 15, 2024

Conversation

Nidoxs
Copy link
Contributor

@Nidoxs Nidoxs commented Aug 14, 2024

Proposed Changes

Allowed disabling of systems by right clicking them in the system list. Additionally, I have also added a tooltip for when a system's name is truncated.

Related issues

Closes #110.

@Nidoxs
Copy link
Contributor Author

Nidoxs commented Aug 14, 2024

image

@LastTalon
Copy link
Member

Is this ready for review at this stage?

@Nidoxs
Copy link
Contributor Author

Nidoxs commented Aug 15, 2024

Is this ready for review at this stage?

Yeah, ready for review! Feel free to checkout the branch and test on your end if you wish to see it more closely. 😄

@LastTalon LastTalon requested a review from a team August 15, 2024 18:53
Ukendio
Ukendio previously approved these changes Aug 15, 2024
@Nidoxs Nidoxs dismissed Ukendio’s stale review August 15, 2024 20:57

The merge-base changed after approval.

Ukendio
Ukendio previously approved these changes Aug 15, 2024
@Nidoxs Nidoxs dismissed Ukendio’s stale review August 15, 2024 22:45

The merge-base changed after approval.

Ukendio
Ukendio previously approved these changes Aug 15, 2024
@Nidoxs Nidoxs dismissed Ukendio’s stale review August 15, 2024 22:51

The merge-base changed after approval.

@LastTalon LastTalon merged commit 9abd461 into matter-ecs:main Aug 15, 2024
6 checks passed
@Nidoxs Nidoxs deleted the right-click-system-toggle branch August 15, 2024 22:53
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.

Right click to disable hovered system in system list
3 participants