search

denverfoundation / storybase

The code behind Floodlight
http://floodlightproject.org
MIT License
11 stars 7 forks source link

Help drawer in Story Builder needs "View the tour again" button #508

Closed jwirfs-brock closed 11 years ago

jwirfs-brock commented 11 years ago

This came out of our recent discussion on user feedback (https://docs.google.com/document/d/1eBYGQJJ9G0sWBQS2QNPYD1bP6fC3LxTQGIODfSYT-qw/edit). It's also User Story 297.

ghing commented 11 years ago

Assigning to @jwirfs-brock for feedback on button placement and styling. Would also love to hear from @bpawlak on this.

ghing commented 11 years ago

Did some basic wiring up of this feature in ef2363672d49235ee91fb9f46a298e6e8b47b66a, deployed on dev

ghing commented 11 years ago

@jwirfs-brock Should this be visible during all workflow steps, or only during the build step?

bpawlak commented 11 years ago

If we show it during all steps, we'd need different content as the "tour" of the tagging page is different enough (no section browser, for example) from the rest if the builder.

ghing commented 11 years ago

@bpawlak, I was thinking more that if it was on a different workflow step, it would take the user back to the build step before showing the tour, but that might be a confusing side-effect and probably not desirable. The more I think about this, I'm leaning toward only showing it in the builder.

bpawlak commented 11 years ago

No, I don't think it should take them back to the builder, especially if they're done building... So it should just be on the builder section....

jwirfs-brock commented 11 years ago

Agree that it only makes sense during the build step. Does that add coding complexity? On Dec 18, 2012 3:01 PM, "bpawlak" notifications@github.com wrote:

No, I don't think it should take them back to the builder, especially if they're done building... So it should just be on the builder section....

— Reply to this email directly or view it on GitHubhttps://github.com/PitonFoundation/atlas/issues/508#issuecomment-11507357.

bpawlak commented 11 years ago

I think the button styling is fine (it's consistent with the rest of the app), but I wonder if it calls too much attention to itself at the top like that? It kind of takes precedence over the rest of the help text which, arguably, is more important, especially if someone has viewed the tour before...

ghing commented 11 years ago

@bpawlak, I'll try making the button the gray style and making it a little smaller

ghing commented 11 years ago

@bpawlak, I made the button gray, smaller and right-aligned it. Check it out in the attached screenshot or on http://dev.atlas.terrorware.com/ Does this address your concerns?

less_prominent_tour_button

jwirfs-brock commented 11 years ago

One suggestion I had was that the tour is blue (very bright blue!), so if the button were similarly styled it might be a visual cue reminding the user that it's about the tour. @bpawlak and @ghing if you think that would look too loud, we can keep it grey.

By the way, I also showed this to Brigid to get some quick feedback. Her general sense was that she doesn't really notice the Help or Asset drawers, so she'd like to see the tour button somewhere else. We don't really have a lot of room elsewhere, so it's OK in the drawer for now. When we change the workflow steps, we may want to reconsider this.

ghing commented 11 years ago

@bpawlak and @jwirfs-brock,

Check out the latest pass, now up on dev.

As Bill suggested, I changed the button to an icon, but kept the blue as a reminder. It's noticeable, but less competitive with the help text. In order to make this more noticeable, I added a step to the tour that pointed out that the tour could be shown again using the icon (how meta).

show_tour_again_icon

bpawlak commented 11 years ago

I suggested to Geoff that perhaps a small, light blue (same color as tour) icon (maybe that looks like a speech bubble since that's what we're using for the tour) either as part of (or instead of) the button, with a tooltip?

jwirfs-brock commented 11 years ago

I like the icon! Looks good.

Without the tool tip (which I'm not seeing now on dev), it's too subtle. I like @bpawlak's suggestion of a tool tip, and maybe also small text underneath the chat bubble that says "Tour."

jwirfs-brock commented 11 years ago

Oops, I am seeing the tool tip. It's just not styled the way the others are, so I missed it at first.

ghing commented 11 years ago

@jwirfs-brock, I did another pass, visible on dev, where I added the text "Tour" and the fancy tooltips.

jwirfs-brock commented 11 years ago

@ghing I think this looks great. It's good on my end.

ghing commented 11 years ago

@jwirfs-brock This is done and staged. The only changes I made since yesterday are back-end stuff to ensure it doesn't show up in other workflow steps. Feel free to take another pass and reopen if you find anything weird.