smooth-app icon indicating copy to clipboard operation
smooth-app copied to clipboard

feat: Edit product page UI improvements

Open g123k opened this issue 1 year ago • 1 comments

  • On scroll, the barcode will be visible in the AppBar
  • The copy barcode is now in the AppBar
  • The photo icon is filled, like the other icons
  • The title on each item is in bold
  • Ripples on each card is now OK
  • Scrollbar for the list

https://user-images.githubusercontent.com/246838/183444328-c5bd611b-713b-470c-bdbd-d171caae0f11.mp4

g123k avatar Aug 08 '22 14:08 g123k

Codecov Report

Merging #2754 (46a03d3) into develop (2ea0da3) will decrease coverage by 1.72%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2754      +/-   ##
==========================================
- Coverage     8.86%   7.13%   -1.73%     
==========================================
  Files          161     218      +57     
  Lines         6623   10604    +3981     
==========================================
+ Hits           587     757     +170     
- Misses        6036    9847    +3811     
Impacted Files Coverage Δ
...kages/smooth_app/lib/widgets/attribute_button.dart 0.00% <0.00%> (-92.00%) :arrow_down:
...s/smooth_app/lib/data_models/user_preferences.dart 8.73% <0.00%> (-23.57%) :arrow_down:
packages/smooth_app/lib/themes/smooth_theme.dart 62.26% <0.00%> (-20.72%) :arrow_down:
...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart 15.11% <0.00%> (-19.10%) :arrow_down:
...mooth_app/lib/data_models/product_preferences.dart 21.68% <0.00%> (-9.75%) :arrow_down:
packages/smooth_app/lib/main.dart 14.65% <0.00%> (-3.24%) :arrow_down:
.../smooth_app/lib/pages/onboarding/welcome_page.dart 0.00% <0.00%> (-3.13%) :arrow_down:
.../smooth_app/lib/pages/onboarding/scan_example.dart 0.00% <0.00%> (-2.28%) :arrow_down:
...ackages/smooth_app/lib/pages/scan/scan_header.dart 2.50% <0.00%> (-2.27%) :arrow_down:
...p/lib/pages/onboarding/consent_analytics_page.dart 0.00% <0.00%> (-1.57%) :arrow_down:
... and 237 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

codecov-commenter avatar Aug 08 '22 14:08 codecov-commenter

@g123k needs rebase before merging

teolemon avatar Aug 16 '22 11:08 teolemon

Should be OK now 👌

g123k avatar Aug 16 '22 11:08 g123k