-
-
Notifications
You must be signed in to change notification settings - Fork 259
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
fix: 4176 - border for some edit image buttons #4218
fix: 4176 - border for some edit image buttons #4218
Conversation
Impacted files: * `edit_image_button.dart`: added an optional border * `edit_ocr_page.dart`: unrelated add of "logged in?" parameter * `product_field_editor.dart`: unrelated add of a "logged in?" parameter * `product_image_local_button.dart`: added an optional border and a "logged in?" parameter * `product_image_server_button.dart`: added an optional border and a "logged in?" parameter; added a test if display is relevant * `product_image_viewer.dart`: added a "logged in?" parameter
Codecov Report
@@ Coverage Diff @@
## develop #4218 +/- ##
===========================================
+ Coverage 10.88% 11.21% +0.32%
===========================================
Files 277 277
Lines 13600 13763 +163
===========================================
+ Hits 1481 1543 +62
- Misses 12119 12220 +101
... and 31 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. 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.
- Can we have a little space between text and border ?
- Also, can the border line be brown in light mode ?
Impacted file: * `edit_image_button.dart`: colors and padding slight change
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.
Visually good 👌
What
Now in the OCR pages
Screenshot
Fixes bug(s)
Impacted files
edit_image_button.dart
: added an optional borderedit_ocr_page.dart
: unrelated add of "logged in?" parameterproduct_field_editor.dart
: unrelated add of a "logged in?" parameterproduct_image_local_button.dart
: added an optional border and a "logged in?" parameterproduct_image_server_button.dart
: added an optional border and a "logged in?" parameter; added a test if display is relevantproduct_image_viewer.dart
: added a "logged in?" parameter