console-cap icon indicating copy to clipboard operation
console-cap copied to clipboard

Added logLevel method

Open Anonym-tsk opened this issue 11 years ago • 2 comments

Usage: console.logLevel('debug'); Available levels: all - all messages will be shown debug - same as "all" excluding console.log and console.info log - same as "all" excluding console.debug and console.count info - same as "log" excluding console.log warn - same as "info" excluding console.info error - same as "warn" excluding console.warn

Anonym-tsk avatar Mar 17 '13 11:03 Anonym-tsk

Привет! Извини, что-то пропустил этот пул-реквест( Хотел бы обсудить его. Название метода мне кажется немного некорректным. Словно это настройка для метода console.log, а не для всего console в целом. Плюс передавать строковыми настройками - не очень корректно. Может, стоит сделать хеш методов? Как-то так:

console.level.warn()

theshock avatar Apr 15 '13 16:04 theshock

В принципе, разницы то нет. Просто logLevel - это, мне кажется, общепринятое обозначение, хотя я согласен, что строковый параметр это очень неудобно. В этом плане console.level.warn() будет удобнее, но тогда уж, если не logLevel, то, хотя бы setLevel, чтобы было понятно, что делает метод.

Anonym-tsk avatar Apr 16 '13 08:04 Anonym-tsk