Olaf Bosch schrieb:
> Hi all, here is FaviconLinkMarker Beta3:
> 
> http://olaf-bosch.de/bugs/jquery/links3.html
> 
> - No IMG-tag, all CSS
> - config. Favicon left or right (before/after)
> - editable TITLE-tag
> - No CSS edit for using
> - No CSS overwritten, all what you in CSS given is after JS given (i 
> hope ;) )
> 
> *attantion!!!* IE have a bug, the image are not consistent for linebreaks

The only fix I know for that bug is to give the link white-space: nowrap.

I would condense your code a little bit, the only difference in here:

  if (place != "after") {
    jQuery(this).attr({ title: ""+titletxt+" "+hrefvalue+"" }).css({
    background: "transparent url("+cuesrc+") no-repeat left center",
    "padding-left": "19px",
    "white-space": "nowrap"});
  } else {
    jQuery(this).attr({ title: ""+titletxt+" "+hrefvalue+"" }).css({
    background: "transparent url("+cuesrc+") no-repeat right center",
    "padding-right": "19px",
    "white-space": "nowrap"});
  }

is the background-positioning, thus to avoid redundant code:

jQuery(this).attr({ title: ""+titletxt+" "+hrefvalue+"" }).css({
    background: "transparent url("+cuesrc+") no-repeat " + (place == 
"after" ? "right" : "left") + " center",
    "padding-left": "19px",
    "white-space": "nowrap"});

Downside with inline styles is that you can't override them in IE for print.


-- Klaus


_______________________________________________
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/

Reply via email to