search

angular / batarang

AngularJS WebInspector Extension for Chrome
MIT License
2.43k stars 338 forks source link

Performance watchers right #274

Closed drpicox closed 8 years ago

drpicox commented 8 years ago

It is difficult to read the watcher-text with watchers stats (total time...) when there is very long watcher texts (for example, a function body in a single line).

Here the proposal is to align right the watcher text to make easy to read all toghether.

An alternative, not implemented here, is to change order of columns and reorder stats to be on the left, and align watcher text to left.

SomeKittens commented 8 years ago

Thanks! This has annoyed me and has been on my todo list for a while.

Screenshot of this in action:

screen shot 2015-10-28 at 11 12 15 am

I think this would be better served by limiting the total number of characters of a watcher shown.

drpicox commented 8 years ago

I do not like the idea of limiting the total number of characters, better to set maximum width and wrap characters (for example: word-wrap: break-word;)

SomeKittens commented 8 years ago

Yeah, I think I like setting max width and just wrapping best, now that you mention it.

SomeKittens commented 8 years ago

Fixed in 884b3ba0b436ce4840f880c00ede1cecf730cbb5