-
-
Notifications
You must be signed in to change notification settings - Fork 64
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: ImageHelper - new methods with ImageSize parameters + minor fix #572
fix: ImageHelper - new methods with ImageSize parameters + minor fix #572
Conversation
Impacted file: * `ImageHelper.dart`: added flexibility and `ImageSize` parametered methods; fixed a minor bug (trailing slash)
Would it be possible to shorten function naming a bit. Some are getting overly verbose imho. I suggest making |
Impacted file: * `ImageHelper.dart`: added optional parameters instead of methods.
Sounds reasonable. |
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.
LGTM, thanks @monsieurtanuki
@@ -27,23 +27,31 @@ class ImageHelper { | |||
} | |||
} | |||
|
|||
/// Returns the product image full url, or null if [barcode] is null | |||
/// Returns the [image] full url - for a specific [imageSize] if needed. |
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.
/// Returns the [image] full url - for a specific [imageSize] if needed. | |
/// Returns the image full url as [String] - for a specific [imageSize] if needed. |
|
||
/// Returns the product image filename | ||
/// Returns the [image] filename - for a specific [imageSize] if needed. |
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.
Same here
Impacted file:
ImageHelper.dart
: added flexibility andImageSize
parametered methods; fixed a minor bug (trailing slash)What
ImageHelper
Part of