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

report: add help link icon #3844

Closed
wants to merge 2 commits into from
Closed

report: add help link icon #3844

wants to merge 2 commits into from

Conversation

patrickhulce
Copy link
Collaborator

@patrickhulce patrickhulce commented Nov 17, 2017

closes #3696

first pass at adding some help icons, definitely open to other ideas

screenshot below
image

@paulirish
Copy link
Member

i missed the original discussion, and have some concerns about this UI. (IMO adds too much clutter to the UI) But i understand the core problem we're trying to solve (i think)

two alternatives to this i see.

  1. try to make this UI work. lower contrast with these icons.. make them gray. slide them next to the title text.
  2. don't add new UI. instead follow what PSI does in making their "learn more" links more obvious by incorporating as part of the details rather than at the end of the nobody-reads description:
    image

i slightly favor 2, but i have no idea which will work better for users.

@vinamratasingal-zz
Copy link

Patrick- I agree with Paul that the UI looks a lot more cluttered with the question marks, and I don't think will help solve our discoverability issues given that they're on the right side (where the eye will probably not naturally follow, so users will still be confused)

Paul- I like the second suggestion, and I think we should do it, but I don't think it's going to solve to discoverability issue to begin with (because it will still require users to discover the drop down, which is not happening atm). Is there anyway we can setup an A/B experiment within the UI to test multiple options and see whether our discoverability improves? Additionally, can we try just making the arrows bigger, or maybe move their placement after the word (so like "Does not register a service worker [insert arrow]")

@patrickhulce
Copy link
Collaborator Author

@vinamratasingal what do you think of the updated screenshot?

@paulirish
Copy link
Member

I can live with the updated UI. Still skeptical, but not enough to block it. :)

@vinamratasingal-zz
Copy link

Errr I think you misunderstood what I said Patrick. I actually think we should get rid of the question marks completely :) I decided to make a screenshot to explain what I am envisioning:

screen shot 2017-11-18 at 1 34 41 pm

Does this make sense? Can we make the dropdown a little bit more gray/darker, so there's more emphasis to it?

@patrickhulce
Copy link
Collaborator Author

@vinamratasingal I didn't misunderstand. The current screenshot is the 2nd take on the UI after iterating with Paul (it was already done before you had commented just didn't update the screenshot). I have no dog in this fight, just was doing what was assigned that we decided in a meeting that seemed reasonable :) that being said...

Stuff like moving the arrows after the sentence and tweaking shades of gray feels like a futile exercise. If we're not taking any substantial steps like moving the link outside the dropdown, I'd vote to abandon until a designer or research is involved here.

@paulirish
Copy link
Member

We can't move those arrows anytime soon.

+1 to deferring this in general.

Copy link
Member

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

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

yeah, I also vote defer and add it as a top level priority for the next report UX reworking

@patrickhulce
Copy link
Collaborator Author

cool cool then

@paulirish paulirish deleted the add_docs_links branch March 2, 2018 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix discoverability of drop downs and learn more
4 participants