Replace Tor image button with proper font icon #1470

jonah · 2019-11-06T08:26:30Z

dawidpotocki commented

2019-11-06 08:26:30 +00:00

(Migrated from github.com)

Who thought that it was a good idea?

netlify[bot] commented

2019-11-06 08:27:17 +00:00

(Migrated from github.com)

Deploy preview for privacytools-io ready!

Built with commit 1308ec0d3f

https://deploy-preview-1470--privacytools-io.netlify.com

Deploy preview for *privacytools-io* ready! Built with commit 1308ec0d3fcf07f9de9a07b7a6f921ffd7ed6519 https://deploy-preview-1470--privacytools-io.netlify.com

Mikaela (Migrated from github.com) approved these changes 2019-11-06 09:45:31 +00:00

Mikaela (Migrated from github.com) left a comment

LGTM

jonah approved these changes 2019-11-06 22:54:56 +00:00

jonah left a comment

kind of a weird icon, I'd probably prefer a more stylized one like in their browser since it'd look better as a font. but whatever, lgtm

kind of a weird icon, I'd probably prefer a more stylized one like in their browser since it'd look better as a font. but whatever, lgtm <img width="28" alt="image" src="https://user-images.githubusercontent.com/3637842/68345041-0c61b800-00b6-11ea-965a-54a4bfec2b55.png">

dawidpotocki commented

2019-11-06 22:58:47 +00:00

(Migrated from github.com)

kind of a weird icon, I'd probably prefer a more stylized one like in
their browser since it'd look better as a font. but whatever, lgtm

Well, I'm just using the official one :P.
https://styleguide.torproject.org/visuals/#assets

> kind of a weird icon, I'd probably prefer a more stylized one like in > their browser since it'd look better as a font. but whatever, lgtm Well, I'm just using the official one :P. https://styleguide.torproject.org/visuals/#assets

jonah commented

2019-11-06 23:16:01 +00:00

yeah, yeah, just doesn't look great as a button imo. seems like very thin lines.

djoate commented

2019-11-24 19:49:12 +00:00

(Migrated from github.com)

Missing Tor icon at https://www.privacytools.io/providers/email/