-
Notifications
You must be signed in to change notification settings - Fork 71
Finalize tooltips style + functionality #139
Comments
@caheberer @nataliafitzgerald @stephanieosan @mebates Regarding tooltips, OaH is currently using black with white text on our loan comparison tool, which is on build here: http://build.consumerfinance.gov/owning-a-home/loan-comparison/ We took this pattern from HMDA: We explored using a light grey background but felt it didn't stand against the grey of our tool well enough. And a different color, such as green, is too similar to our green education notes, which we want to be visually differentiated. Thoughts re: whether our use of this is further solidifying black with white text as a design standard? Or should the design standard be further explored and our use of this style be considered a special case situation due to the visual needs of the tool? |
Also--- perhaps with the black background we should take a look again at whether or not it should be transparent. |
I've always been against the black background, because I think a big block of reversed-out type is hard to read. There is lengthy discussion of this on our internal github, search "modals" and its the first to pop up (specifically, issue 78 in the digital-product-guide repo). I also think we need to standardize the size and shape of the arrrow that comes off of the modal. I've always used our caret minicon as a starting point. |
I agree that the reversed-out type seems harder to read but it does help create clearer contrast between the tooltip and the rest of the page. Yet the heavy weight of the dark background doesn't seem to match with the rest of our styling. Some questions in addition to style:
Adding the 508 tag for @JenniferHoran as we'll need to make sure that these are accessible for keyboard controls and screen readers. |
Okay, I'm splitting out tooltips and modals, because they're two separate elements and it seems like modals may have previously been finalized (at least in terms of styling). But as far as I can tell from the previous issue in the digital product guide, styling (and behavior) of tooltips requires a little more consideration. Modals now live here: #325 |
Recommendation: move to popovers instead of tooltips(and add a close button for mobile users) Currently, on desktop, the tooltip disappears when the user's mouse moves away from the tooltip icon. See www.consumerfinance.gov/hmda for an example. This is a persnickety interaction—the hit area for that tooltip icon is pretty small. I think we want to make this less frustrating for users, by transitioning to popovers. With popovers, the information stays visible as long as the user's mouse is hovering over the popover. This makes it possible to include links. See below examples. Also, Bootstrap has some specifications for popovers here. User can click links in the popover: Recommendation: Add a close button to the upper right hand corner of the popoverI'd recommend adding a close button in the upper right-hand corner of the tooltip for mobile users. This seems the lowest lift approach to translating popovers to mobile. So, if we are implementing the Bootstrap popover component, it should be modified with a close button. |
I showed these during Graceland's session yesterday. Here are a handful of thoughts:
|
Can we add a solid triangle to the icon font? That will make implementing this much easier. cc: @Dnpizarro Thanks! |
I could make an argument for implementing it other ways, but we could use solid triangles for other reasons, too, so might as well. |
I'd be curious how else you'd implement that triangle (other than an image?). Maybe an inline SVG in the CSS? |
Yeah, I was thinking maybe inline SVG with PNG fallback. The main question in my head was, "Considering this as a component standing alone, do we really want to add a cf-icons dependency just to get that shape in there?" |
I think that's a good point. I'd argue for SVG data uri with no PNG fallback (do IE8 users really need that caret?). |
Probably not :) |
For the inline tooltips, would it make more sense to use something that we could base off of the |
This is great. On the UI front y’alls are totally approved! Some granular notes as you finalize all the details:
|
Style
States
Any example of a tooltip next to a form field label will violate this rule. Maybe change the wording to say that this is preferred but not always practical?
Should be something like "Dark Pacific", as it's darker than regular Pacific.
Should be full Navy? Might depend on how much both colors contrast with Pacific and Dark Pacific.
Probably clearer to call this "solid drop shadow" rather than border. The whole box already has a 1px border.
This sounds lovely, but difficult. I look forward to seeing how your Cat and Virginia handle it :) Inline
Other than those items, all looks good! |
Transferring @nataliafitzgerald 's comments to this issue: DM Styling edits:
|
Thanks for the feedback, I’ve compiled a revision list that I’m working on to complete approval.
Other notes: Max character count of 225 was chosen based on a sampling of explanations and definitions that currently exist and to not exceed 5 lines on large screens. |
@Scotchester @benguhin @nataliafitzgerald Following the new palette, I'm proposing the attached for the tooltip icon states, which I also think we should use as our button standards. This change will make our buttons 508 compliant, because as @Scotchester mentioned, our current white on Pacific 80 isn't accessible. I've attached an image with the new hex/named values. |
This is great, thank you all! |
Pinging @benguhin @nataliafitzgerald @Scotchester - Can each of you give a 👍 or otherwise comment about changing the tooltip icon hover/focus/active state colors and matching our new color palette? It's our only thing left to figure out for this! |
👍 for UI |
👍 |
👍 for graphic design. |
Let's finish this! |
@caheberer will look for the design of this page. |
Backlog item, for a future date post-release.
#137
#128
The text was updated successfully, but these errors were encountered: