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

refactor: reorder menuitems on history page #4181

Merged

Conversation

khanjasir90
Copy link
Contributor

What

Reorder menu items of History Page from Clear, View on the web, Share to Share, View on Web, Clear you history

Screenshot

WhatsApp Image 2023-06-15 at 10 18 17 PM

Signed-off-by: Mohd Jasir Noor Khan <khanmohdjasir@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #4181 (4c0f9ba) into develop (79ec954) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #4181   +/-   ##
========================================
  Coverage    10.87%   10.87%           
========================================
  Files          276      276           
  Lines        13591    13591           
========================================
  Hits          1478     1478           
  Misses       12113    12113           
Impacted Files Coverage Δ
...pp/lib/pages/product/common/product_list_page.dart 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@g123k
Copy link
Collaborator

g123k commented Jun 16, 2023

Hi @khanjasir90,

Thanks for your PR.
But there's also another part in the issue, where we want to be able to delete selected items.

@khanjasir90
Copy link
Contributor Author

@g123k how about doing it in another PR. So that this PR context is more relevant to the mentioned issue?

@khanjasir90
Copy link
Contributor Author

@g123k Will do it in another PR pointing to #4153

@@ -1306,7 +1306,7 @@
"@added_to_list_msg": {
"description": "Message when products have been successfully added to a list"
},
"user_list_popup_clear": "Clear",
"user_list_popup_clear": "Clear your history",
"@user_list_popup_clear": {
"description": "Short label of a 'clear list' popup"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you change the description of this translation? As Clear list is incorrect.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

Signed-off-by: Mohd Jasir Noor Khan <khanmohdjasir@gmail.com>
Copy link
Collaborator

@g123k g123k left a comment

Choose a reason for hiding this comment

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

Thanks for your first PR 🏆

@g123k g123k merged commit 42b8788 into openfoodfacts:develop Jun 16, 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.

History menu: shouldn't we reorder the items?
3 participants