-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
bounty : minor changes #768
Conversation
@mbeacom Still a lil uneasy on showing the change in actions before we fire the API. (but if we do it post the API fire -> everything seems super slow ) Checkout the network tabs in the gif |
Codecov Report
@@ Coverage Diff @@
## master #768 +/- ##
==========================================
- Coverage 33.99% 33.95% -0.04%
==========================================
Files 101 101
Lines 5775 5695 -80
Branches 672 666 -6
==========================================
- Hits 1963 1934 -29
+ Misses 3733 3685 -48
+ Partials 79 76 -3
Continue to review full report at Codecov.
|
@PixelantDesign Are you good with these changes? |
@mbeacom @thelostone-mc Yes! Thank you!!!!!!!!!!! |
mind resolving the conflicts here and we can merge? |
@owocki Done !! |
- changed the order of action buttons - start work turns white on starting work - added svg background to comment - fixed margin between action buttons
The change in order of buttons and color was implemented as discussed offline with @PixelantDesign