даже не знаю как назвать топик, но во внутрь загляните

[devote]

Это можно было бы сократить нмного

Для этого есть минификаторы. Делать код в одну строчку, это не код а говнокод

sel=document.querySelector.bind(document)

Метода .bind () нету в ИЕ8

[krutoy]

Tecvid,
Но вообще, да, ронни прав, если надо выставить атрибут в значение пустой строки, мой вариант не сработает, я правда не знаю, зачем это нужно

[Tecvid]

если надо выставить атрибут в значение пустой строки, мой вариант не сработает

а вообще-то можно и это тоже проверить в условии, тем самым усложнив функцию, в теории звучит хорошо но на практике мне пока не нужна)) но возьму на заметку

короче, всем большое спасибо очень признателен)

[рони]

я правда не знаю, зачем это нужно

сбросить атрибут в значение по умолчанию

[krutoy]

сбросить атрибут в значение по умолчанию

Можно сбросить во что угодно, он все равно не сработает. Зачем все усложнять.

[devote]

function attr(selector, attrName, attrValue) {
  if (arguments.length > 2) {
    if (attrValue === null) {
      // удаляет атрибут
      return sel(selector).removeAttribute(attrName);
    }
    // задает значение атрибуту
    return sel(selector).setAttribute(attrName, attrValue);
  }
  // возвращает значение атрибута
  return sel(selector).getAttrubute(attrName);
}

[krutoy]

рони,
В любом случае, проблема не велика, это пустое все, как выше заметил Tecvid

if(value||value==="") reurn ...
...
, делов то.

[Tecvid]

function attr(selector, attrName, attrValue) {
  if (arguments.length > 2) {
    if (attrValue === null) {
      // удаляет атрибут
      return sel(selector).removeAttribute(attrName);
    }
    // задает значение атрибуту
    return sel(selector).setAttribute(attrName, attrValue);
  }
  // возвращает значение атрибута
  return sel(selector).getAttrubute(attrName);
}

спасибо огромное, вот это уже функция)
хотя странно что в первоначальной функции setAttribute работал и без return, а тут нет)

[devote]

в первоначальной функции setAttribute работал и без return, а тут нет)

ну можно убрать ретурны:

function attr(selector, attrName, attrValue) {
  if (arguments.length > 2) {
    if (attrValue === null) {
      // удаляет атрибут
      sel(selector).removeAttribute(attrName);
    } else {
      // задает значение атрибуту
      sel(selector).setAttribute(attrName, attrValue);
    }
  } else {
    // возвращает значение атрибута
    return sel(selector).getAttrubute(attrName);
  }
}

[Tecvid]

ну можно убрать ретурны:

когда их убираю не срабатывает) поэтому сказал что странно