hide-text mixin using legacy technique #2586

Closed
paulirish opened this Issue Mar 14, 2012 · 6 comments

5 participants

@paulirish

The .hide-text mixin that was added from #2362 is using a technique the originator of it has already abandoned.

h5bp/html5-boilerplate@aa0396e discusses the why, but using em's on padding/margin is the primary reason this original Kellum technique can be trouble. That said there are plenty of other reasons why this technique is strong along with a full swath of screenreader testing.

It'd be nice to standardize our approaches here leading to much lower maintainability cost.

cc @nimbupani @scottkellum @zslabs

@nimbupani

Agreed, I would love to propagate the use of well-tested defaults and this kind of standardization would help immensely.

@scottkellum

Hiding text is nothing more than a glorified hack. A useful one but still a hack. display: none is the obvious choice but screen readers and search engines skip it. Driving us to more obscure and inelegant solutions to trick screen readers and search engines from thinking the text is still there. Lots of ways to do this and they all have drawbacks but that is the nature of hacks.

In short, I don't think we are going to find an elegant solution to image replacement any time soon and maybe we need to have a new CSS declaration to actually do it right text-decoration: hide; or something. There was a long debate last decade and I assume this will be no different. The debate is so heated because all the methods suck.

@fat fat added a commit that referenced this issue Mar 14, 2012
@fat fat update hide text to h5bp technique #2586 8e0afbe
@mdo
Bootstrap member

This looks to be resolved as of 2.0.2.

@mdo mdo closed this Mar 23, 2012
@scottkellum

Is it resolved? Still settling on H5BP method?

@stempler stempler pushed a commit to stempler/bootstrap that referenced this issue Apr 11, 2014
@fat fat (less) update hide text to h5bp technique #2586 618def1
@stempler stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
@fat fat (less) update hide text to h5bp technique #2586 faae47d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment