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

Nutrition Facts Page: view jumps to empty Serving size field when clicking unit buttons #3436

Closed
zanovis opened this issue Dec 12, 2022 · 4 comments · Fixed by #3921
Closed
Assignees
Labels

Comments

@zanovis
Copy link

zanovis commented Dec 12, 2022

What

On the Nutrition Facts edit page, the view jumps to the empty Serving size field when clicking unit buttons on the right side of page.

Steps to reproduce the behavior

  1. Go to Nutrition Facts page for a product
  2. If the Serving size field is empty, proceed to step 3. If not, clear the Serving size field
  3. Choose a nutrient and change its value
  4. Click the unit button for that nutrient
  5. See view jump to Serving size field

Expected behavior

Expect view to remain unchanged when clicking button

Smartphone model

  • Device: Pixel 6a
  • OS: Android 13
  • App Version: 4.1.0+848
@zanovis zanovis added the 🐛 bug Something isn't working label Dec 12, 2022
@monsieurtanuki
Copy link
Contributor

@zanovis Today I was not able to reproduce this issue, on an iOS emulator.
Do you still have this issue, e.g. with 4.3.0? Could be android-related.

@zanovis
Copy link
Author

zanovis commented Jan 22, 2023

Yes, I observed this on 4.3.0 yesterday on Android 13

@monsieurtanuki
Copy link
Contributor

@zanovis I've just tried again and still could not reproduce the issue on my ios emulator.
That said, I've fixed slightly related lines in #3716, which may fix that issue too.

@monsieurtanuki
Copy link
Contributor

@zanovis I've managed to reproduce it on my android smartphone.
About to PR a fix...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

3 participants