netmonitor icon indicating copy to clipboard operation
netmonitor copied to clipboard

Icon clarity suggestion

Open gilgongo opened this issue 9 years ago • 5 comments

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.

gilgongo avatar Jul 10 '16 07:07 gilgongo

Agreed :) Will try to fix when I get some time. PRs most welcome!

awalgarg avatar Jul 10 '16 10:07 awalgarg

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 ;)

blueelvis avatar Jul 10 '16 19:07 blueelvis

Awal, I liked the effectiveness of the idea (use simple js listeners and pull out something valuable). A few suggestion : For sites like FB, we see that the hex-icon shoots to 700 after logging in. From Home Page, just do a scroll and the count shoots to 1500+ and furtherdown it is 2500+... Like others (my bro and my techie frnd), I would be really happy if we get to know more about these requests after clicking the icon and we see a link/button 'Requests Info' and on clicking that, we provide extensive information about the requests. Beneath that, we could have the 'Icons made by etc. etc.' stuffs

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...

SlickHackz avatar Jul 11 '16 07:07 SlickHackz

Added PR : https://github.com/awalGarg/netmonitor/pull/5 (changing the color of background from green to red if the count is lesser than 20 for 'ON' status)

SlickHackz avatar Jul 11 '16 10:07 SlickHackz

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.

SamHasler avatar Jul 11 '16 14:07 SamHasler