Linked images, eg the Drupal footer block, the admin menu icon, are getting big fat oldskool borders on them in Firefox. This is due to the resource stylesheet.

In Garland, this happens:

img, a img {
border:medium none;
}
style.css?N (line 189)
img {
border:0 none;
}
defaults.css?N (line 18)
*|*:-moz-any-link img, img[usemap], object[usemap] {
border:2px solid;
}
html.css (line 400)

Does Clean do something to defaults.css that removes that img rule?

Comments

psynaptic’s picture

psynaptic commented

Yep, Clean kills most core stylesheets.

I can't seem to reproduce this though.

I was going to suggest that we are missing this:

a img {
  border: none;
}

But I can't reproduce. Are you saying that by default any images within anchor tags are given a border? I don't see this in Firefox.

Could you try the dev version and see if it's still there?

joachim’s picture

joachim commented

> Are you saying that by default any images within anchor tags are given a border?

Yes. On FF 3.5.6.

psynaptic’s picture

psynaptic commented

I'm also in 3.5.6 and I can't see what you're talking about. Could you post a screenshot and confirm you have updated to dev and still see the same issue?

psynaptic’s picture

psynaptic commented
Status: Active » Needs review

I think I know what you have done. Did you comment out the following in reset.css?

* {
  margin: 0;
  padding: 0;
  border: 0;
  outline: 0;
  background: transparent;
}

I have changed the structure of the reset now as using the asterisk wasn't the most efficient thing. I have changed it to:

html, body, div, span, applet, object, iframe,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, dfn, em, font, img, ins, kbd, q, s, samp,
small, strike, strong, sub, sup, tt, var,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
table, caption, tbody, tfoot, thead, tr, th, td {
  margin: 0;
  padding: 0;
  border: 0;
  outline: 0;
  font-weight: inherit;
  font-style: inherit;
  font-size: 100%;
  font-family: inherit;
  vertical-align: baseline;
}

which is taken from Eric Meyer's Reset Reloaded. Maybe there is more we can use from this reset?

psynaptic’s picture

psynaptic commented
Status: Needs review » Fixed

This is now fixed.

Status: Fixed » Closed (fixed)

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