Skip to content
This repository has been archived by the owner on Sep 20, 2018. It is now read-only.

Icon clarity suggestion #2

Open
gilgongo opened this issue Jul 10, 2016 · 5 comments
Open

Icon clarity suggestion #2

gilgongo opened this issue Jul 10, 2016 · 5 comments

Comments

@gilgongo
Copy link

gilgongo commented Jul 10, 2016

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.

@awalgarg
Copy link
Owner

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

@blueelvis
Copy link

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

@SlickHackz
Copy link

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
Copy link

Added PR : #5 (changing the color of background from green to red if the count is lesser than 20 for 'ON' status)

@SamHasler
Copy link

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants