Skip to content
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

feat: added dialog if users click on unselect image button #2427 #3707

Merged
merged 5 commits into from
Feb 14, 2023

Conversation

BhuvanAde
Copy link
Contributor

Signed-off-by: BhuvanAde bhuvanadey@gmail.com

What

  • Added an "are you sure?" dialog when the user clicks on the "unselect image" button.
  • Added a new dialog and added one entry in app_en.ar

Screenshot

image

Fixes bug(s)

#2427

Part of

@BhuvanAde
Copy link
Contributor Author

@monsieurtanuki Please review this PR. I have fixed the suggested changes. Please let me know if there is anything to change

@Sudhanva-Nadiger
Copy link
Contributor

@BhuvanAde remove the unused imports from the code ! That's why pre submit tests are failing

Copy link
Contributor

@monsieurtanuki monsieurtanuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @BhuvanAde!
Please have a look at my comments.

packages/smooth_app/lib/l10n/app_en.arb Outdated Show resolved Hide resolved
packages/smooth_app/lib/l10n/app_en.arb Outdated Show resolved Hide resolved
@BhuvanAde
Copy link
Contributor Author

@monsieurtanuki. Sorry for the inconvenience. Please take a look at these.

Copy link
Contributor

@monsieurtanuki monsieurtanuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @BhuvanAde, that looks OK!

packages/smooth_app/lib/l10n/app_en.arb Outdated Show resolved Hide resolved
@@ -164,7 +166,7 @@ class _ProductImageViewerState extends State<ProductImageViewer> {
EditImageButton(
iconData: Icons.do_disturb_on,
label: appLocalizations.edit_photo_unselect_button_label,
onPressed: _actionUnselect,
onPressed: () => _actionUnselect(appLocalizations),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure of the added value to make this call different from the other.

@monsieurtanuki monsieurtanuki merged commit 0dd30c4 into openfoodfacts:develop Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add image unselect in the expanded view accessible from the carousel
3 participants