-
Notifications
You must be signed in to change notification settings - Fork 48
feat(gridworks): Quick navigation on config and mission view #4154
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
Conversation
Eckhardt-D
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some self-review items
Removes unused dangling prop, idiomatic config loop. Better sizing of select, make EditableNode more generic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One last thing to clean up, I'll approve now to speed up the process, feel free to merge when ready
gridworks/src/app/cogames/missions/view/[name]/MissionTitle.tsx
Outdated
Show resolved
Hide resolved
berekuk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought I approved last time, my bad
Adds ability to edit the config title, which opens a list of all other configs, when typing starts configs are filtered based on whether it contains the typed text. - Doesn't immediately filter on the active name to see full list of other configs - Resets on blur - Clicking (or Enter) on config name navigates to that view - Can navigate with arrow keys to select option TODO: - [x] If approach makes sense, replicate in the missions view <img width="690" height="280" alt="image" src="https://github.com/user-attachments/assets/1778b3d3-6d5e-4745-864b-93a7f4623d1f" /> <img width="744" height="448" alt="image" src="https://github.com/user-attachments/assets/ece710c2-9cdc-400d-9f3c-7420aede3767" /> <img width="534" height="331" alt="image" src="https://github.com/user-attachments/assets/51c474cd-6abe-4738-83d5-7fef4b4bae05" /> [Asana Task](https://app.asana.com/1/1209016784099267/project/1210348820405981/task/1212304005487515) [Asana Task](https://app.asana.com/1/1209016784099267/project/1210006613945419/task/1212210337332221)
Adds ability to edit the config title, which opens a list of all other configs, when typing starts configs are filtered based on whether it contains the typed text.
TODO:
Asana Task
Asana Task