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

QuickEditor: Update the Avatar's menu UI #561

Merged
merged 5 commits into from
Jan 28, 2025

Conversation

AdamGrzybkowski
Copy link
Contributor

Closes #556

Description

This PR modifies the Avatar's menu by:

  • Changing the Rating item label from Change rating to Rating: {current}
  • Adding the Arrow icon to indicate the submenu
  • Moving the Download item to the top

Testing Steps

  1. Launch the QE
  2. Tap on the ... button to open the Avatar's menu
  3. Confirm you can see the changes above

@AdamGrzybkowski AdamGrzybkowski added Enhance Enhancement [Feature] Gravatar-Quickeditor Gravatar Quick Editor module labels Jan 27, 2025
@AdamGrzybkowski AdamGrzybkowski added this to the 2.3.0 milestone Jan 27, 2025
@AdamGrzybkowski AdamGrzybkowski force-pushed the adam/556_rating_item_label branch from 4791664 to f6528b6 Compare January 27, 2025 11:19
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jan 27, 2025

📲 You can test the changes from this Pull Request in Gravatar Demo by scanning the QR code below to install the corresponding build.
App Name Gravatar Demo
Commita3ca960
Direct Downloadgravatar-demo-prototype-build-pr561-a3ca960.apk

@@ -25,50 +25,61 @@ internal fun AvatarMoreOptionsPickerPopup(
onDismissRequest: () -> Unit,
onAvatarOptionClicked: (AvatarOption) -> Unit,
) {
val altTextItemLabel = stringResource(
Copy link
Member

Choose a reason for hiding this comment

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

Why altTextItemLabel? Shouldn't it be named ratingItemLabel?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh my... yes... Thanks!

Copy link
Member

@mlumeau mlumeau left a comment

Choose a reason for hiding this comment

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

LGTM!

@AdamGrzybkowski AdamGrzybkowski force-pushed the adam/556_rating_item_label branch from 9e08e3b to a3ca960 Compare January 28, 2025 11:01
@AdamGrzybkowski AdamGrzybkowski merged commit f263834 into trunk Jan 28, 2025
17 checks passed
@AdamGrzybkowski AdamGrzybkowski deleted the adam/556_rating_item_label branch January 28, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhance Enhancement [Feature] Gravatar-Quickeditor Gravatar Quick Editor module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the Avatar's menu UI
3 participants