Skip to content

Commit

Permalink
Remove image replacement helper class .ir
Browse files Browse the repository at this point in the history
This class was removed because it doesn't provide enough use
cases within modern web development. Cases are mostly handled
indiviually by developers.

Reference #1472.
Closes #1475.
  • Loading branch information
drublic committed Nov 18, 2013
1 parent 94c3540 commit f1a8e91
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 25 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
### HEAD

* Remove image replacement helper class `.ir`
* Update to Modernizr 2.7.0.
* Update to Apache Server Configs 2.0.0.
* Add vertical centering for `svg` ([#1453](https://github.com/h5bp/html5-boilerplate/issues/1453)).
Expand Down
18 changes: 0 additions & 18 deletions css/main.css
Original file line number Diff line number Diff line change
Expand Up @@ -118,23 +118,6 @@ textarea {
Helper classes
========================================================================== */

/*
* Image replacement
*/

.ir {
background-color: transparent;
border: 0;
overflow: hidden;
}

.ir:before {
content: "";
display: block;
width: 0;
height: 150%;
}

/*
* Hide from both screenreaders and browsers: h5bp.com/u
*/
Expand Down Expand Up @@ -251,7 +234,6 @@ textarea {
* Don't show links for images, or javascript/internal links
*/

.ir a:after,
a[href^="javascript:"]:after,
a[href^="#"]:after {
content: "";
Expand Down
7 changes: 0 additions & 7 deletions doc/css.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,13 +38,6 @@ You are free to modify or add to these base styles as your project requires.

## Common helpers

#### `.ir`

Add the `.ir` class to any element you are applying image-replacement to. When
replacing an element's content with an image, make sure to also set a specific
`background-image: url(pathtoimage.png);`, `width`, and `height` so that your
replacement image appears.

#### `.hidden`

Add the `.hidden` class to any elements that you want to hide from all
Expand Down

4 comments on commit f1a8e91

@frankstallone
Copy link

Choose a reason for hiding this comment

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

Very interesting we live in a world sans-ir. It was nice to have this in my back pocket even though I used it less and less over the last few years. I do understand the browser issues though stated in the #1472 and would want there to be a universal .ir class that worked cross browser instead of one that worked for half the browsers out there (just as a for instance).

@gmeben
Copy link

@gmeben gmeben commented on f1a8e91 Dec 12, 2013

Choose a reason for hiding this comment

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

Wow.

@andydownunder
Copy link

Choose a reason for hiding this comment

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

Dude.

@jlbruno
Copy link

Choose a reason for hiding this comment

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

interesting

Please sign in to comment.