-
Notifications
You must be signed in to change notification settings - Fork 643
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
Mount redesign #3698
base: default-mount-target-dart-ffi
Are you sure you want to change the base?
Mount redesign #3698
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## default-mount-target-dart-ffi #3698 +/- ##
================================================================
Coverage ? 88.93%
================================================================
Files ? 254
Lines ? 14302
Branches ? 0
================================================================
Hits ? 12720
Misses ? 1582
Partials ? 0 ☔ View full report in Codecov by Sentry. |
ad871c5
to
a6d6dac
Compare
0b14c4b
to
b4b4bed
Compare
a6d6dac
to
2bcf6d0
Compare
b4b4bed
to
4f2908f
Compare
2bcf6d0
to
7529af8
Compare
4f2908f
to
9648897
Compare
7529af8
to
82eec21
Compare
9648897
to
f9d9341
Compare
82eec21
to
bd96a13
Compare
864ba49
to
f6fa82e
Compare
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.
Well done!! The mount section looks good and is intuitive to use. I also liked how you turned ConfirmationDialog into a reusable widget 😃
bd96a13
to
4a72529
Compare
932bcc8
to
b1ce55f
Compare
b1ce55f
to
2b46fba
Compare
4a72529
to
d9c986f
Compare
This PR implements the redesign of the mounting interface, as well as some cosmetic changes to the pages that use the mounting interface, namely the launch panel and the individual instance page.
See https://warthogs.atlassian.net/browse/MULTI-894 and https://warthogs.atlassian.net/browse/MULTI-895 for the details of the changes.
There was also some refactoring of adjacent code that is not necessarily mount-related (e.g. splitting some files into multiple files), so that the modification and addition of the mount code is easier