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

[UIX] Minor friction challenges #779

Closed
mghenciu opened this issue Jul 13, 2023 · 4 comments · Fixed by #781 or #784
Closed

[UIX] Minor friction challenges #779

mghenciu opened this issue Jul 13, 2023 · 4 comments · Fixed by #781 or #784
Labels
new feature Request for a new feature or functionality to be added to the project. released Indicate that an issue has been resolved and released in a particular version of the product.

Comments

@mghenciu
Copy link
Contributor

mghenciu commented Jul 13, 2023

What problem does this address?

While testing Feedzy, I think there are some small ares where we can do a better job on the details:

Improve how the Confirmation Toast looks

Currently we use the Default WP toast, and even if this is recommended - in our case it's a bit strange because it moves and in a way brakes the layout (attached video):

Screen.Recording.2023-07-13.at.18.17.40.mov

As a potential solution, I think we can use something similar to Spectra (also in the video above at the end) or how the toasts look in the editor:
Screenshot 2023-07-13 at 18 20 05

This way the toasts won't interfere with the Feedzy layout.


2 Primary buttons in the same area

On the Settings page there are 2 Primary buttons in the same area + the Remove image button is visible all the time, even when there are no image added. From an UIX pov, this increases the cognitive load since more buttons look important.
Screenshot 2023-07-13 at 18 27 14

This could be a potential solution for this:

Screen.Recording.2023-07-13.at.18.52.56.mov

Upgrade button opens in the current page

I think as a general rule, we open the upgrade links in a new tab. This doesn't happen in the current Feedzy build, which open the Admin Sidebar upgrade, in the same tab. This may be annoying for the users, since it takes them outside the wordpress context.
Screenshot 2023-07-13 at 19 40 08


What is your proposed solution?

Added above.

Will this feature require documentation? (Optional)

No.

@mghenciu mghenciu added the new feature Request for a new feature or functionality to be added to the project. label Jul 13, 2023
@mghenciu mghenciu changed the title [UiX] Minor friction challenges [UIX] Minor friction challenges Jul 13, 2023
girishpanchal30 added a commit that referenced this issue Jul 26, 2023
@girishpanchal30 girishpanchal30 linked a pull request Jul 26, 2023 that will close this issue
@girishpanchal30
Copy link
Contributor

Hey @mghenciu,

I've fixed the mentioned issue/improvement with this PR #781, please check with the attached PR zip, and do let me know if you have any issues.

Thanks

@irinelenache
Copy link

@girishpanchal30 Tested and everything's working fine here 🚀

I'll move this to Ready to merge after @mghenciu approves ✔️

@mghenciu
Copy link
Contributor Author

The changes look good 👏
Thank you Irinel for testing & @girishpanchal30 for the implementation.

@vytisbulkevicius vytisbulkevicius linked a pull request Aug 3, 2023 that will close this issue
@pirate-bot
Copy link
Contributor

🎉 This issue has been resolved in version 4.2.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Request for a new feature or functionality to be added to the project. released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants