The border bottom goes under the help icon, and can appear to touch it also, I think this looks a bit shabby and can be done better, i.e. either not underline the icon or create some space below the icon so it does not touch the border bottom.

At first this seems like an innocuous "move the icon to .filter-help.form-wrapper {}", however that moves the icon out of the anchor.

What I'd propose here perhaps that UX either veto this move or approve, and if veto'd then we just add a bit of neg margin to move the icon up 1 or 2 pixels and I think that will work OK.

Potentially pile other small cleanups for border bottom issues here also if they're quite minor like this one.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

Status: Active » Needs review
FileSize
348 bytes

So is it worth just removing it from this element?

Jeff Burnz’s picture

Probably can work removing the border, I suspect the icon indicates a link/focusable element?

mgifford’s picture

I don't know of any best practices about this in general, however it stands alone at the moment and isn't crowded by other content.

It would be useful to have a consistent pattern though which we can leverage in Bartik...

At the moment links are underlined other than those in the header.

LewisNyman’s picture

Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +frontend, +CSS

Yeah, the icon does look weird like this. I double checked and the help icon *is* clickable, because it's inside of the link tag, so I've modified the summary to reflect this.

I think we have two solutions:

1. Move the icon up a few pixels
2. Move the icon outside of link tag

I think I prefer 1, should we try that first?

emma.maria’s picture

Assigned: Unassigned » emma.maria

I will create a patch for idea 1.

emma.maria’s picture

Assigned: emma.maria » Unassigned
FileSize
14.48 KB

This patch adds more spacing between icon and underline.

emma.maria’s picture

Status: Needs work » Needs review
FileSize
353 bytes

Oops here is the patch ;)

Bojhan’s picture

Great, am I correct that here is some useless height?

LewisNyman’s picture

Status: Needs review » Needs work
mgifford’s picture

Seven doesn't have the underline under the ? icon. I think that's a better pattern that we should be repeating.

LewisNyman’s picture

@mgifford We don't have a choice because Bartik is using border-bottom for links instead of text underline.

Jeff Burnz’s picture

Lewis, I think you misread the OP - what it meant was IF we moved the icon out of the anchor that might be a UX issue. I'm not sure why you changed it, sorry but I think you misread it and removed an important sentence alluding to the UX issue of moving the icon.

#10, you can do that sort of thing, even with border bottom, its a tad on the tricky-dicky side of things, I did not cross browser test this, but it should work pretty good even with minor adjustment. I have to admit I'm not entirely down on :after content and what gets announced etc (by screenreader software).

.filter-help a:after {
  background: url("../../../misc/help.png") no-repeat scroll right center / 100% auto transparent;  /* LTR */
  content: " ";
  height: 1rem;
  width: 1rem;
  position: absolute;
  top: 0.2em;
}
.filter-help {
  position: relative;
  left: -1rem; /* LTR */
}

EDIT: removed inline-block, sorry, wrong copy/paste.

Also note: what this does (in supporting browsers) is stretch the icon as the font-size is increased, so it gets bigger, i.e. covers the 1rem square, which is how it maintains its vertical alignment even though its absolutely positioned. I kind of dreamed this up while watching telly, so take it as read I did not test this a whole lot, but I think its pretty solid. The top and or right positioning could be adjusted per theme rather than in filter.admin.css

Jeff Burnz’s picture

Screenshot of the code in #12, forgot to attach...

LewisNyman’s picture

Issue tags: +Usability

@Jeff Oops sorry. I discussed this with Bojhan in Austin and we both agreed it's better to keep the icon within the clickable area. That's why my preferred option was to try and shift the icon position

Jeff Burnz’s picture

@Lewis, oh good you got to discuss with Bojhan, yes indeed I agree its better to keep it clickable. Good stuff.

mgifford’s picture

Issue tags: +dcamsa11y
BarisW’s picture

Status: Needs work » Needs review
FileSize
897 bytes
10.71 KB

How about this? The icon still works on hover and the underline is gone.

BarisW’s picture

Issue tags: +Amsterdam2014
mgifford’s picture

That looks better. The ? is click-able. Here's a view in RTL. Wish we had the string itself translated, but the icon is in the right location:

LewisNyman’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
325.27 KB
356.06 KB

This looks really good now, great work! I've attached screenshots from Bartik and Stark

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 1f59908 and pushed to 8.0.x. Thanks!

  • alexpott committed 1f59908 on 8.0.x
    Issue #2271327 by emma.maria, mgifford, BarisW | Jeff Burnz: Fixed Tips...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.