Ładnie obcinaj łańcuch, aby pasował do danej szerokości piksela

Jest kilka problemów z Twoim kodem.

• dlaczego / 5? Szerokość znaków zależy od font-family i font-size.
• musisz uciec str w tytule abbr (w przeciwnym razie " spowoduje nieważność kodu).
• diff nie jest zadeklarowana i kończy się w zasięgu globalnym
• substring nie powinno tak działać. Jakiej przeglądarki używasz?
• {[8] } nie jest poprawną wartością style.display. Aby go ukryć należy użyć wartości none, ale wtedy przeglądarka nie oblicza offsetWidth. Zamiast tego użyj style.visibility="hidden".
• poszukiwanie odpowiedniej długości jest bardzo nieefektywne.
• musi uciec </abbr> "

Przepisałem go dla Ciebie i dodałem className, abyś mógł użyć stylu, aby ustawić font-family i font-size. Pan Fooz zasugerował, aby użyć mouseover, aby pokazać cały ciąg. Nie jest to konieczne, ponieważ nowoczesne przeglądarki robią to za Ciebie (testowane z FF, IE, Opera i Chrome) {]}

    function fitStringToSize(str,len,className) {
    var result = str; // set the result to the whole string as default
    var span = document.createElement("span");
    span.className=className; //Allow a classname to be set to get the right font-size.
    span.style.visibility = 'hidden';
    span.style.padding = '0px';
    document.body.appendChild(span);


    // check if the string don't fit 
    span.innerHTML = result;
    if (span.offsetWidth > len) {
        var posStart = 0, posMid, posEnd = str.length;
        while (true) {
            // Calculate the middle position
            posMid = posStart + Math.ceil((posEnd - posStart) / 2);
            // Break the loop if this is the last round
            if (posMid==posEnd || posMid==posStart) break;

            span.innerHTML = str.substring(0,posMid) + '…';

            // Test if the width at the middle position is
            // too wide (set new end) or too narrow (set new start).
            if ( span.offsetWidth > len ) posEnd = posMid; else posStart=posMid;
        }
        //Escape
        var title = str.replace("\"",""");
        //Escape < and >
        var body = str.substring(0,posStart).replace("<","&lt;").replace(">","&gt;");
        result = '<abbr title="' + title + '">' + body + '&hellip;<\/abbr>';
    }
    document.body.removeChild(span);
    return result;
    }

Edytuj: Podczas testowania trochę więcej znalazłem kilka robaków.

• Użyłem Math.ceil zamiast intended Math.floor (I blame this on ten angielski nie jest moim ojczystym język)

• Jeśli łańcuch wejściowy ma znaczniki html wtedy wynik byłby nieokreślony (nie jest dobrze obcinać znacznik w środek lub zostawić otwarte znaczniki)

Ulepszenia:

• Ucieczka ciągu, który jest kopiowany do zakresu we wszystkich miejscach. Możesz nadal używać encji html, ale nie wolno używać znaczników (< i > zostanie wyświetlony)
• przepisać while-oświadczenie (jest to trochę szybciej, ale głównym powodem było pozbyć się robaka, który spowodowało dodatkowe rundy i pozbyć się Oświadczenie o zerwaniu)
• zmieniono nazwę funkcji na fitStringToWidth

Wersja 2:

function fitStringToWidth(str,width,className) {
  // str    A string where html-entities are allowed but no tags.
  // width  The maximum allowed width in pixels
  // className  A CSS class name with the desired font-name and font-size. (optional)
  // ----
  // _escTag is a helper to escape 'less than' and 'greater than'
  function _escTag(s){ return s.replace("<","&lt;").replace(">","&gt;");}

  //Create a span element that will be used to get the width
  var span = document.createElement("span");
  //Allow a classname to be set to get the right font-size.
  if (className) span.className=className;
  span.style.display='inline';
  span.style.visibility = 'hidden';
  span.style.padding = '0px';
  document.body.appendChild(span);

  var result = _escTag(str); // default to the whole string
  span.innerHTML = result;
  // Check if the string will fit in the allowed width. NOTE: if the width
  // can't be determined (offsetWidth==0) the whole string will be returned.
  if (span.offsetWidth > width) {
    var posStart = 0, posMid, posEnd = str.length, posLength;
    // Calculate (posEnd - posStart) integer division by 2 and
    // assign it to posLength. Repeat until posLength is zero.
    while (posLength = (posEnd - posStart) >> 1) {
      posMid = posStart + posLength;
      //Get the string from the beginning up to posMid;
      span.innerHTML = _escTag(str.substring(0,posMid)) + '&hellip;';

      // Check if the current width is too wide (set new end)
      // or too narrow (set new start)
      if ( span.offsetWidth > width ) posEnd = posMid; else posStart=posMid;
    }

    result = '<abbr title="' +
      str.replace("\"","&quot;") + '">' +
      _escTag(str.substring(0,posStart)) +
      '&hellip;<\/abbr>';
  }
  document.body.removeChild(span);
  return result;
}