Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Add support Private Tab extension (#69) #311

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ba3r
Copy link
Contributor

@ba3r ba3r commented Jan 19, 2015

No description provided.

@seleznev
Copy link
Contributor

@0rAX0, can you review this icon please (the second from the right)?

screenshot from 2015-01-20 23 01 52

@0rAX0
Copy link
Member

0rAX0 commented Jan 20, 2015

Hmm, I didn't know what it was without looking at the title. :\ Maybe a stripped tab like we use in the theme?

@ba3r
Copy link
Contributor Author

ba3r commented Jan 20, 2015

Thanks for the feedback. I was considering many options, the stripes as
well. I went for this one because I wanted to use the known mask metaphor
for private mode, and the original extension also uses it. I'll have to
experiment with the stripes again and will update this with a few icon
concepts. Just leave the pull open, thanks.

On Tue, Jan 20, 2015 at 8:04 PM, Reda Lazri [email protected]
wrote:

Hmm, I didn't know what it was without looking at the title. :\ Maybe a
stripped tab like we use in the theme?


Reply to this email directly or view it on GitHub
#311 (comment)
.

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

Successfully merging this pull request may close these issues.

3 participants