-
Notifications
You must be signed in to change notification settings - Fork 17
Icon clarity suggestion #2
Comments
Agreed :) Will try to fix when I get some time. PRs most welcome! |
It would be really good if some documentation could also be added apart from the icon change. That way, it would be clear to users ;) |
Awal, I liked the effectiveness of the idea (use simple js listeners and pull out something valuable). I also partly agree with gilgongo's but I feel that classifying a website a good/bad with no. of requests made is not logical. Anyways this is subjective. I also appreciate his efforts and yours... |
Added PR : #5 (changing the color of background from green to red if the count is lesser than 20 for 'ON' status) |
Wouldn't it be better to implement it as a page action instead of a browser action, so it appears inside the address bar (and is thus associated with the page), and only on pages that are misbehaving. |
This is a very nice plugin - excellent in fact!
One small thing: the visual difference between the icon states seem reversed to me. That is, the state for a "good" site is more noticeable (and appears more urgent) than the "bad" site one.
This seems to be due to the use of more red in the good state, and a slightly confusing icon which shows a network diagram (indicating something to do with a network of course). So I have to think twice each time I see the icon - and the use of the strong red colour draws my eye to it unnecessarily.
Perhaps the "good" state should not have a graphical icon at all and just a request count, while the bad one should have the red network diagram?
Other than that tiny niggle - it's perfect.
The text was updated successfully, but these errors were encountered: