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

Prefer Wikidata SVG logos over social media icons for brand presets #6361

Closed
1ec5 opened this issue May 15, 2019 · 11 comments
Closed

Prefer Wikidata SVG logos over social media icons for brand presets #6361

1ec5 opened this issue May 15, 2019 · 11 comments
Labels
considering Not Actionable - still considering if this is something we want enhancement An enhancement to make an existing feature even better preset An issue with an OpenStreetMap preset or tag waitfor-upstream Waiting for something in an upstream project

Comments

@1ec5
Copy link
Collaborator

1ec5 commented May 15, 2019

When a brand entry in the name-suggestion-index has an SVG image as its Wikidata logo, prefer that image over any Facebook or Twitter logo.

Official Facebook and Twitter accounts are not always as reliable as Wikidata items for brand logos. A brand may adopt a non-logo as its social media photo as part of a marketing campaign. However effective the marketing message may be in the context of a social media profile, it’s simply jarring when it appears in iD preset search results, where the user isn’t expecting an advertising message.

KFC search

KFC name-suggestion-index

Wikidata on the left.

Here, the SVG logo from Wikidata is much more recognizable than the profile picture KFC is currently displaying on Facebook and Twitter, which is meant to be viewed in the context of a larger profile cover photo. As I understand it, the point of putting these icons in the search results is to increase the presets’ recognizability.

KFC’s marketing campaign may be ephemeral, but some brands choose social media photos that are still less recognizable than a logo would be:

In-N-Out name-suggestion-index

This isn’t to say that logos in Wikidata are always the most suitable for iD. But most of the time, an image from Wikidata would be unsuitable because it’s a photo of a store location or a very wide logo. To mitigate these issues, we could keep the current behavior whenever the Wikidata logo has an illegible aspect ratio like 1:6 or wider, or when the Wikidata logo is a JPEG or PNG.

iD/build_data.js

Lines 253 to 260 in ddc9d16

let logoURLs = nameSuggestionsWikidata[qid] && nameSuggestionsWikidata[qid].logos;
if (logoURLs) {
if (logoURLs.facebook) {
logoURL = logoURLs.facebook.replace('?type=square', '?type=large');
} else {
logoURL = logoURLs.twitter || logoURLs.wikidata;
}
}

@1ec5 1ec5 added preset An issue with an OpenStreetMap preset or tag enhancement An enhancement to make an existing feature even better labels May 15, 2019
@bhousel
Copy link
Member

bhousel commented May 15, 2019

For the record, I see absolutely nothing wrong with this 😆
Screenshot 2019-05-15 09 53 49

@bhousel
Copy link
Member

bhousel commented May 17, 2019

ok.. So I did this, and it didn't work as well as I had hoped..
The main problem is that these wiki commons logos are often really not square. I left the code in build_data.js commented out for now, because it was producing noticeably worse results.

commons:
Screenshot 2019-05-17 00 53 30

facebook:
Screenshot 2019-05-17 00 59 07

@bhousel bhousel added the considering Not Actionable - still considering if this is something we want label May 17, 2019
@matkoniecz
Copy link
Contributor

Is non squareness sole problem or is it worse?

@bhousel
Copy link
Member

bhousel commented May 17, 2019

Is non squareness sole problem or is it worse?

That's the sole problem. They just don't look good in the space where we are using them, in the way that a profile picture would..

@matkoniecz
Copy link
Contributor

Is allowing only relatively square ones too complicated or is it just case "someone needs to code it"?

@BjornRasmussen
Copy link
Contributor

Is allowing only relatively square ones too complicated or is it just case "someone needs to code it"?

Currently, iD only flags buildings if they are very close to being square (some corners are almost 90°), don't have too many nodes, and aren't connected to other buildings. A triangular building, for example, would not be flagged

@bhousel
Copy link
Member

bhousel commented May 17, 2019

Is allowing only relatively square ones too complicated or is it just case "someone needs to code it"?

We'd need to fetch the images to determine their dimensions.
Also they can change at any time, and I wouldn't expect the Wikipedia folks to change all their logos to accommodate iD. I understand the appeal of using Wikipedia commons logos, but it really feels like a dead-end to me.

@bhousel
Copy link
Member

bhousel commented May 17, 2019

Currently, iD only flags buildings if they are very close to being square (some corners are almost 90°), don't have too many nodes, and aren't connected to other buildings. A triangular building, for example, would not be flagged

This issue isn't about buildings - it's about logos. 😆

But for the record.. iD can detect and square triangular buildings just fine 👍
triangular building

@BjornRasmussen
Copy link
Contributor

Yikes! I confused two issues I've been following - plus I'm used to people not understanding the square validation.
I meant above that a triangle with only 60° angles won't get turned into a square.

@1ec5
Copy link
Collaborator Author

1ec5 commented May 20, 2019

We do have some triangular logos in the index, but thankfully they’re encased in rectangular images. 😉

We'd need to fetch the images to determine their dimensions.

The Wikimedia Commons API can fetch file metadata in bulk without fetching individual images: osmlab/name-suggestion-index#2641.

@1ec5
Copy link
Collaborator Author

1ec5 commented Jun 16, 2019

KFC is back to being… less buff… but now Burger King has joined in on the fun: osmlab/name-suggestion-index#2798.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
considering Not Actionable - still considering if this is something we want enhancement An enhancement to make an existing feature even better preset An issue with an OpenStreetMap preset or tag waitfor-upstream Waiting for something in an upstream project
Projects
None yet
Development

No branches or pull requests

4 participants