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

What is the purpose of the input text box on the main page of the replay interface? #406

Closed
machawk1 opened this issue Jun 29, 2018 · 8 comments

Comments

@machawk1
Copy link
Member

machawk1 commented Jun 29, 2018

I know this but it is not immediately apparent in the interface of how the users should use this text box. Entering a value makes it more clear but it ought to be self-evident in the main interface.

@machawk1
Copy link
Member Author

screen shot 2018-06-29 at 5 44 38 pm

@ibnesayeed
Copy link
Member

The placeholder text http:// can be changed to something like Search a URL in the archive.

@machawk1 machawk1 changed the title What is the purpose of the input text box on the main page of the replay interface. What is the purpose of the input text box on the main page of the replay interface? Jun 29, 2018
@machawk1
Copy link
Member Author

@ibnesayeed That is what I was thinking as well. I recall you recently having a similar conversation with @shawnmjones on this placeholder text. Would having a submission button help with the usability/accessibility/clarity of the text box's function in addition to modifying the placeholder text?

@ibnesayeed
Copy link
Member

ibnesayeed commented Jun 29, 2018

Having a call for action button is not a bad idea. In that case the placeholder text can be a sample URL (e.g., http://example.com/), but not just http:// and the text describing the purpose of the form (e.g., Search URL in the archive) can go to the button.

@machawk1
Copy link
Member Author

machawk1 commented Jul 1, 2018

I believe this suggestion is more accessible, as it is providing an example of what should be entered as the input instead of instruction as the placeholder.

Based on the above I am proposing:

  • Adding a label (again, for accessibility)
  • Replacing the "http://" with "http://example.com"
  • Adding a submit button (potentially with the value text provided)

Do we anticipate any other UI elements on this page beyond those associated with this text box?

The bottom portion of the page is admittedly a mess and won't scale, but is useful for users trying out the app. Adjusting the availability (or lack thereof) of the admin interface from this page is addressed in another ticket, so some refactoring is in order there, too.

@ibnesayeed
Copy link
Member

If the text field is semantically named and the button text is descriptive enough then an additional label will be a noise without mush accessibility benefits.

@machawk1
Copy link
Member Author

machawk1 commented Jul 1, 2018

@ibnesayeed Are you proposing we not have a label for the text input? Is using the name attribute for the text input sufficient for it be considered accessible a la Section 508?

@ibnesayeed
Copy link
Member

I was suggesting we not have a visible label text, but we can always include attributes like title and aria-label for better accessibility on non-visual media.

machawk1 added a commit that referenced this issue Jul 3, 2018
Add semantics/accessibility to main replay UI. Closes #406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants