-
-
Notifications
You must be signed in to change notification settings - Fork 381
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
Replacement widget icons are outdated #1857
Comments
Also, at least FacebookLike, FacebookShare, Pinterest and Twitter buttons seem also be outdated. |
I created a fix for the google icon only: #1926 I could not find SVGs of the other icons |
We could use PNGs too, SVGs just might be nicer (crisp at all resolutions). |
@ghostwords in that case, I might be able to fix the other icons too |
Hi, where can the "disabled by PB" icon be found? I'd be happy to help out if someone could point me in the direction of the PB icon asset. |
The icons are in |
Here is a higher resolution copy: https://github.com/EFForg/privacybadger/blob/9d8e273e35631d97d5ab6df5187559b4c44f843f/src/skin/badger-logo-revise-wide.png |
@ghostwords just looking into this. Before I start trying to help can I check a couple of things. Where are these used? Do they need to be exactly the same as others on the web or is this simply a case of making an updated version that looks more refined etc? Thanks for the help |
The icons are used by the placeholders that Privacy Badger uses to replace certain potentially-useful third-party page elements. We call these placeholders "replacement widgets". The ones in this issue are specifically used for "social" buttons. You can see a bunch of them together on the ShareMeNot test page: This issue is about updating the icons to the latest versions used by each corresponding service, with the Badger logo in the corner the same way it is now. The icon used for Google+ has already been updated: #1926 Let me know if this makes sense or not. |
@ghostwords that makes sense, thanks! I'll give it a go now and we can work out the details with the first pull request. |
@ghostwords How would you feel about doing the share buttons like ShareMeNot do without the company logo. Just the privacy badger logo and then the word next to it? See the link you sent me I feel like this may be more timeless and does not make the buttons look old as the companies contantly change their branding. |
@ghostwords I have given it a go. I really struggled at making it SVG. For some reason even with the same size as the current the code makes my SVG images huge so I have moved to png. This shouldnt cause any problems unless the user zooms all the way in. I have changed a few things. Please see the pull request for a full list #2287 |
I've also had a look at this and I have a few ideas. It looks like there is potential for adding quite a few items to the When it comes to using 3rd party logos, there are various rules that each third party has about how they want their logo used. We'd have to try and adhere to these standards so that we don't run into legal issues. A big no-no with all companies that care about their branding is allowing overlays on top of their logos. I don't feel that SVG is the way to go here, since creation and maintenance is more challenging and the bandwidth saving is negligible in this case. I feel that the way to go is to create a set of standardized icons that can easily be generated from a master Photoshop file. From my quick testing, it seems like the icons are mostly just injected into the page and are not scaled, so I don't think that the trade-off between ease of use and scalability is warranted. We'd do fine with ~72px x 24px icons like the ones in the example below. I've added some ideas here In my mind, these icons exist simply to make the user aware that something has changed and that Privacy Badger has sanitized the buttons. So we don't need anything too obtrusive. I've been tinkering with a few ideas, here are a couple of 'in the wild' screenshots of the 3 icons I've created so far to test with. I've took the brand guidelines for the major brands into account while designing the base icon. If you folks think that this is useful at all and want me to send up a pull request with a full set of icons that replace the current set, just let me know. Some considerations I had in mind while building these... Buttons must:
I think that absolute consistency in how the buttons look would help create a feeling of trust with users. |
Google+ is discontinued, so imo they should just be hidden. |
As per the following comments: - #1734 (comment) - #1857 (comment)
I mean this icon: https://github.com/EFForg/privacybadger/blob/master/src/skin/socialwidgets/GooglePlus.svg
What Privacy Badger is using:
The new one:
The text was updated successfully, but these errors were encountered: