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

Add a duplicate/copy button for each action in the autos on the PathPlanner client #653

Merged
merged 10 commits into from
Mar 28, 2024

Conversation

jasonli0616
Copy link
Contributor

@jasonli0616 jasonli0616 commented Mar 18, 2024

Create a button on the PathPlanner GUI to duplicate an action (a path, command, command group).

Implementation of #652

@github-actions github-actions bot added the GUI Changes to the PathPlanner GUI label Mar 18, 2024
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

Attention: Patch coverage is 53.33333% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 85.41%. Comparing base (5aa0063) to head (fd84a0c).

Files Patch % Lines
...or/tree_widgets/commands/command_group_widget.dart 22.22% 14 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #653      +/-   ##
==========================================
- Coverage   85.55%   85.41%   -0.15%     
==========================================
  Files          77       78       +1     
  Lines        6737     6767      +30     
==========================================
+ Hits         5764     5780      +16     
- Misses        973      987      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jasonli0616 jasonli0616 marked this pull request as ready for review March 19, 2024 00:44
@jasonli0616
Copy link
Contributor Author

Example of usage (duplicating paths, commands, groups) in video below:

Screen.Recording.2024-03-18.at.9.32.14.PM.mov

@mjansen4857
Copy link
Owner

Bit of a nitpick but can you move the duplicate button for command groups to be next to the delete button so its the same order as other commands?

@jasonli0616
Copy link
Contributor Author

Yep! Change has been made
Screenshot 2024-03-26 at 10 44 26 PM

@mjansen4857 mjansen4857 merged commit d0e12f5 into mjansen4857:main Mar 28, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Changes to the PathPlanner GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants