-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add list of keyboard shortcuts to document settings #447
Conversation
Space: "Start/stop path playback", | ||
Escape: "Clear sidebar and navbar selection" | ||
}, | ||
"Field View": { |
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.
why are some keys with quotes and some without? Are we trying to keep single words without quotes? Is there a purpose between differentiating?
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.
The Prettier formatter unquoted the ones with single words. IDK if it's worth figuring out how to ignore it here
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.
then thats fine.
E: "Rotates 1/32 turn clockwise", | ||
Q: "Rotates 1/32 turn counterclockwise", | ||
"Shift + Rotation Key": "Rotate by 1/8 turn instead of 1/32 turn", | ||
W: "Translate 0.1 m up (+Y)", |
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.
Doesn't this decrease y value?
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.
(Line 57)
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.
No
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 don't know what you're seeing but upwards on the screen is increasing Y
@@ -68,17 +69,28 @@ class SettingsModal extends Component<Props, State> { | |||
<Tab label="Robot Config" /> | |||
<Tab label="Export Config" /> | |||
<Tab label="Betas" /> | |||
<Tab label="Controls" /> |
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.
should controls go before betas?
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.
Sure
isMac = (type == "Darwin"); | ||
}) | ||
// These default to the Mac ⌘ key because it's easier to replace with Ctrl than vice versa | ||
let shortcuts = { |
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.
all of these should be flipped. I see people looking for what is the generate key to find the shortcut. not, oh i forgot what command G does?
tldr should be descriptions on left and the shortcut on right (flipped).
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.
Done
> | ||
{Object.entries(shortcuts).map((entry) => ( | ||
<> | ||
<Accordion> |
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.
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.
Fixed
Co-authored-by: Jacob Trentini <[email protected]>
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.
lgtm
idk why but my ide is acting up so there are probably errors that ill fix later. Anyway, I added a static field in Choreo.java which stores the current trajectory that the robot is on. Calling choreoSwerveCommand() will update this to the trajectory it is given, and then, when finished, put it back to an empty trajectory. There is also now a new static event() method in Choreo.java which returns a trigger for if the robot is currently on a trajectory given as an argument. holy guacamole i forgot a semicolon imported trigger i mightve jumped the gun but i was bored so heres another event function. is this good more methodsgit add .! more le fixes Add list of keyboard shortcuts to document settings (SleipnirGroup#447) Co-authored-by: Jacob Trentini <[email protected]> Feature: Context Menu (SleipnirGroup#448) * add context menu with popover * format * silence error with conflicting mouseevent * format * works but only if you hot reload (fixing with next commit) * step 1 out of predicted 2: migrate local state to uiState * step 2/2: move d3 callback to overlaywaypoint * format * format and refactorings * address @calcmogul's concerns Upgrade TrajoptLib to use larger solver tolerance (SleipnirGroup#451) organization and comments renaming cleanup java impl fix docs format
idk why but my ide is acting up so there are probably errors that ill fix later. Anyway, I added a static field in Choreo.java which stores the current trajectory that the robot is on. Calling choreoSwerveCommand() will update this to the trajectory it is given, and then, when finished, put it back to an empty trajectory. There is also now a new static event() method in Choreo.java which returns a trigger for if the robot is currently on a trajectory given as an argument. holy guacamole i forgot a semicolon imported trigger i mightve jumped the gun but i was bored so heres another event function. is this good more methodsgit add .! more le fixes Add list of keyboard shortcuts to document settings (SleipnirGroup#447) Co-authored-by: Jacob Trentini <[email protected]> Feature: Context Menu (SleipnirGroup#448) * add context menu with popover * format * silence error with conflicting mouseevent * format * works but only if you hot reload (fixing with next commit) * step 1 out of predicted 2: migrate local state to uiState * step 2/2: move d3 callback to overlaywaypoint * format * format and refactorings * address @calcmogul's concerns Upgrade TrajoptLib to use larger solver tolerance (SleipnirGroup#451) organization and comments renaming cleanup java impl fix docs format
Closes #373
Ctrl/Command key is shown depending on the user's detected OS