Skip to content
This repository has been archived by the owner on Oct 12, 2021. It is now read-only.

Change ambiguous "Go" label to a more descriptive verb-based label for the submit button in the Add RSS & Add Podcast popups. #275

Merged
merged 4 commits into from May 22, 2018

Conversation

KZeni
Copy link
Contributor

@KZeni KZeni commented May 22, 2018

Description of the Change

A minor UI/UX improvement to the submit button on the Add RSS & Add Podcast popups. "Go" is generally a fallback option for a button as it's nondescript & requires understanding of the context. It's usually better to simply state the action that's going to be performed as the button's label to make it as clear as possible for the user. As such, the podcast popup has "Add Podcast" as the submit button's label, and the RSS popup switches between "Add RSS" and "Import OPML" depending if the "Import OPML" section has been expanded or not.

Alternate Designs

The simple "Add" option was initially chosen for simplicity while still being an improvement. "Add / Import" was also considered since the user might be adding an RSS feed or Importing OPML using this one button. I then opted to have the button change to "Add RSS", "Import OMPL", and "Add Podcast" accordingly.

Why Should This Be in Core?

This changes a UI element within the core app.

Benefits

The user will better understand the action that will be performed when clicking the button that was updated.

Possible Drawbacks

None that I can think of.

Applicable Issues

Just a general UI/UX improvement here. No open issues to reference.

KZeni added 4 commits May 22, 2018 11:55
…up's submit button.

Very minor UI/UX improvement.
… popup's submit button.

Very minor UI/UX improvement.
…up (differentiating between the Import OPML and Add RSS feed actions)
@astrotars astrotars requested a review from kenhoff May 22, 2018 20:40
@tschellenbach
Copy link
Member

Nice, great change

@tschellenbach tschellenbach merged commit bb64b3c into GetStream:master May 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants