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

Clarify language and menu interface for saved items. #3489

Merged
merged 3 commits into from Mar 8, 2024

Conversation

demiankatz
Copy link
Member

@demiankatz demiankatz commented Mar 7, 2024

This PR is intended to clarify the labeling of interface elements related to saved items.

It removes three confusing language strings (Favorites, My Favorites, Your Favorites) and replaces them with two more semantically meaningful strings (default_list_title and saved_items).

It also removes the duplicate link to "Saved Items" at the top of the "Your Lists" section of the account menu. This was redundant with the identical link under "Your Account" and was more confusing than helpful.

TODO

  • Run full test suite.
  • Update changelog when merging (note changed language strings).
  • Resolve VUFIND-1592 when merging.

@demiankatz demiankatz added this to the 10.0 milestone Mar 7, 2024
Copy link
Contributor

@sturkel89 sturkel89 left a comment

Choose a reason for hiding this comment

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

The new arrangement in the test branch is cleaner and makes more logical sense to me than what we have on dev. It’s cleaner because of the removal of the redundant Your Favorites in the Your Lists section; in dev, that link gives identical results as Favorites at the top under Your Account.

I like that in test, only ONE item is highlighted at a time – either the Saved Items link at the top of the Your Account section, or a specific list in the Your Lists section.

My only suggestion is that it wouldn’t hurt/might help if there were a circled number next to the superset Saved Items link in the Your Account section giving an overall total number of items saved. That would go above and beyond to differentiate what you get on that list from what’s in an un-renamed Saved Items list under Your Lists, if the user has one generic list and other named lists.

Screenshots from test:

image

image

image

Screenshots from dev:

image

image

image

@demiankatz
Copy link
Member Author

Thanks, @sturkel89. I agree that showing a total of saved items in the menu is a good idea, but it will take a bit of effort to implement. Thus, I'll merge this now for the sake of progress, and will work on that additional improvement as a separate PR when time permits.

@demiankatz demiankatz merged commit fed0f7a into vufind-org:dev Mar 8, 2024
7 checks passed
@demiankatz demiankatz deleted the vufind-1592 branch March 8, 2024 20:08
@demiankatz
Copy link
Member Author

See VUFIND-1674 for the suggested improvement. I've pinned that ticket to 10.1, since I'm not sure if I'll have time to do it before then. This will ensure it doesn't get forgotten!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants