search

sul-dlss-deprecated / universalviewer

The Universal Viewer is a community-developed open source project on a mission to help you share your content with the world
http://universalviewer.io
Other
0 stars 1 forks source link

Update wording and case of Settings options #1

Open ggeisler opened 6 years ago

ggeisler commented 6 years ago

The Settings modal currently looks like this:

strad___and_other_stories___stanford_digital_repository

The settings might be more readable and understandable if we change all options to use sentence-case rather than title-case and update the wording of the options. So the options would read like this:

I'm not 100% sure about the two-page view option because it seems like it might not be working correctly. Is it supposed to disable two-page view if you uncheck it, or just change the default? Because currently if I uncheck it, the two-page view of the object switches to a single-page, but I can still get back to it by clicking the two-page icon. And then when I return to Settings the option is selected again. So it seems like the setting just controls what is the default. (But if so, I'm not sure this justifies even being a setting? I must be missing something.)

@jvine Feel free to update the wording of the options if you have better ideas.

edsilv commented 6 years ago

@ggeisler I agree that Two Page View could be removed from settings as it's now just duplicating the function of the icons next to the cog. We may need to run this past the community however as I might be forgetting something about why it's important.

I also prefer sentence case instead of title case. This is a config option and anyone can change this text to whatever they like, but I'd suggest that we make this the default.

Let's post this thread to the UX channel on the UV Slack to gather some opinion.

mialondon commented 6 years ago

@ggeisler my old usability notes suggest changing the 'navigator' line to 'Turn on overview box' or something similar, as no-one we talked to knew what a navigator was

mialondon commented 6 years ago

And the BL's version also says 'Keep zoom level the same between pages' - wordy but hopefully clear

ggeisler commented 6 years ago

@edsilv Sounds good.

@mialondon Sure, I'm not surprised that "navigator" wasn't always understood. Maybe something like "Enable overview box" or "Display overview box" would work ("enable" and "display" seem slightly better than "turn on" to me)?

And 'Keep zoom level the same between pages' is a definite improvement. Something like "'Maintain zoom level between pages' is a bit shorter and might also work.