Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change Attack mode icon #221

Closed
psiinon opened this issue Oct 24, 2018 · 13 comments
Closed

Change Attack mode icon #221

psiinon opened this issue Oct 24, 2018 · 13 comments
Assignees
Milestone

Comments

@psiinon
Copy link
Member

psiinon commented Oct 24, 2018

At the moment its the same as the Active scan one, which is confusing.

@psiinon psiinon self-assigned this Oct 24, 2018
@dscrobonia
Copy link
Contributor

agreed. Is there an icon for attack already? Or do we need a new one?

@psiinon
Copy link
Member Author

psiinon commented Oct 24, 2018

No :/
Unfortunately the fugue icons we mostly use arent great on 'attacking' icons.
@thc202 suggested https://github.com/yusukekamiyamane/fugue-icons/blob/master/icons/fire-big.png as a possibility, which I'm fine with.
Or I could try merging the std fire icon with another one...

@dscrobonia
Copy link
Contributor

hmmm... I was thinking a cross hair overlaid over the flame might be good. Because attack mode is like active scan (the fire) but its more concentrated on the page/features your visiting (the crosshair).

I tried to make the icon but it is still a little too busy. so if y'all like that idea you might want to take a crack at it. or if youhave suggestions on how I can improve these lmk and I'll try it out

attack

attack2

@psiinon
Copy link
Member Author

psiinon commented Oct 29, 2018

How about this?
attack-mode2

@dscrobonia
Copy link
Contributor

dscrobonia commented Oct 29, 2018

I think all of ours icons still aren't clear enough... its hard to see the fire through our crosshairs.. maybe it was a little too ambitious..

Maybe a flamethrower?? hahah. might be too violent.

@psiinon
Copy link
Member Author

psiinon commented Nov 2, 2018

OK, back to the std fugue icons, how about:
burn

@kingthorin
Copy link
Member

I’m good with that. Or a falling bomb, or a sword. There are lots of things that seem “attack”ish to me.

@dscrobonia
Copy link
Contributor

I was a fan of @thc202s original idea of having it be fire based, but more so, so that it was kind of linked to the active scan. But the radioactive symbol is also pretty nice. We should put that in there for now and try it out. :)

@psiinon
Copy link
Member Author

psiinon commented Nov 4, 2018

How about these?
crosshairs crosshairs-grey

@kingthorin
Copy link
Member

My vote would be the red one

@psiinon
Copy link
Member Author

psiinon commented Nov 4, 2018

I was thinking red for enabled, grey for disabled :) Similar to the way we grey the other disabled icons.

@dscrobonia
Copy link
Contributor

I like it! I was hoping to use the crosshair icon to be injected by a form field to click on and pick a fuzzing payload. but could probably still be the same icon?

@psiinon psiinon added this to the 0.2.0 milestone Dec 28, 2018
@psiinon psiinon modified the milestones: 0.3.0, 0.2.0 Dec 31, 2018
@dscrobonia
Copy link
Contributor

@psiinon very nice!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants