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

Issues with OSM Wikibase links #5753

Closed
my-na-me opened this issue Jan 24, 2019 · 14 comments
Closed

Issues with OSM Wikibase links #5753

my-na-me opened this issue Jan 24, 2019 · 14 comments
Milestone

Comments

@my-na-me
Copy link

my-na-me commented Jan 24, 2019

Starting from this:

Localization >> Fetch multilingual tag descriptions and images from the OSM Wikibase (#5647, thanks @nyurik)

I don't like the current way to offering/showing this to the iD-Editor end-user:
When you click on the "i" (i for information?) then I expect to see an instant view to any help* or a wiki* documentation or a direct link to any help* or a wiki* documentation (*human readable) instead of a link to an option or link to edit any documention.

Current view:
2-osm-wiki-link

So the option to edit the documentation should not shown in the "i-information" chapter

Expected view:
4-osm-wiki-link

Or at least below the osm-wiki link:
3-osm-wiki-link

By the way, why it is telling "There is no documentation available for this key" for highway=service ?
5-osm-wiki-link

@bhousel
Copy link
Member

bhousel commented Jan 24, 2019

@my-na-me I'm ok with the links being how they are, but I am concerned that the documentation isn't filled in and links to wrong pages.

@nyurik any ideas what is going on?
I don't see any description for the highway=service tag.
Also the "View documentation" link goes to a Greek page.. the sitelink language looks wrong.

@bhousel bhousel changed the title potential change of documentation-display (osm-wiki) link order Issues with OSM Wikibase links Jan 24, 2019
@bhousel bhousel added the waitfor-upstream Waiting for something in an upstream project label Jan 24, 2019
@nyurik
Copy link
Contributor

nyurik commented Jan 24, 2019

@my-na-me thx for the feedback! I think we should iterate on the design of the edit link, e.g. we could do this (i'm not a designer, suggestions are welcome):
image

I do not want to visually separate description from the link to edit that description, as that may cause confusion.

Possible link text:

  • [edit]
  • [edit or translate]
  • [edit description]
  • ..?

@bhousel The issue with highway=service was due to the incorrect lang parameter on the wiki page, I fixed it there, and it will be fixed in the data item the next time I run my bot. The wiki docs property (P31) is very new, so some bugs are still lurking. I will also update the bot to ignore the lang param, and instead use the language of the page title. Thx for reporting!

@my-na-me I suspect you didn't see the docs link for highway=service because my bot only filled in that data yesterday, and it might have been after the new version of iD has rolled out.

@nyurik
Copy link
Contributor

nyurik commented Jan 24, 2019

P.S. @my-na-me you can also talk to us directly on OSM Slack - get an invite.

@nyurik
Copy link
Contributor

nyurik commented Jan 25, 2019

While on the subject of design, should we:

  • if both key & tag data items exist, show description for the key AND description for the tag if both are available?
    • what if tag is described in English, but key has the description in the local language?
  • offer two [edit] links - for both the tag and for the key data items?

@my-na-me
Copy link
Author

@bhousel
Thanks for updating my long title version :-)

I'm not so OK with the link order, I would like to see "View documentation" (=> with link to the osm wiki key=value) as top line.

@nyurik
view-docu big
This looks much better, but as mentioned above, I would like to see "View documentation" (=> with link to the osm wiki key=value) as top line, (as it was - for a long time - before ;-) ).

PS: Thank you for the invitation link, actually I stopped instant-messaging long time ago, after a pause-period I switched to this system. Nowadays this is the one I use (also only on low frequency). If you want, we can meet there.

PSS: I opened a thread in the forum, let’s see if somebody is interested in this topic and we could get some more input.

@nyurik
Copy link
Contributor

nyurik commented Jan 25, 2019

@my-na-me I do not think we should show a link to documentation before the short description text. The [edit] should be right next to the description, so that users see how they can change that text, hence the proposed design. Could you sketch what you have in mind? Also, see my other question about possibly showing key + tag descriptions / edit links. Thx!

@nyurik
Copy link
Contributor

nyurik commented Jan 25, 2019

P.S. Maybe we should give a visual indication that the description we are showing is about a key (when there is no tag-specific description is available)... Not sure about the best design for that either.

@bhousel
Copy link
Member

bhousel commented Jan 29, 2019

@nyurik any ideas what is going on?
I don't see any description for the highway=service tag.
Also the "View documentation" link goes to a Greek page.. the sitelink language looks wrong.

Still seeing this today.. @nyurik how often do the cleanup scripts run?

@nyurik
Copy link
Contributor

nyurik commented Jan 31, 2019

@bhousel sorry, was a bit swamped with another project. Just ran the bot, and got everything updated. I will work on automating it very soon - so that the moment wiki page is updated, data items reflect that.

@bhousel bhousel removed the waitfor-upstream Waiting for something in an upstream project label Feb 1, 2019
@bhousel bhousel closed this as completed in d1fca87 Feb 1, 2019
@bhousel
Copy link
Member

bhousel commented Feb 1, 2019

@bhousel sorry, was a bit swamped with another project. Just ran the bot, and got everything updated. I will work on automating it very soon - so that the moment wiki page is updated, data items reflect that.

Thanks @nyurik looks good now 👍

I ended up replacing the link with a simple pencil icon.. I like this!

screenshot 2019-02-01 11 47 16

@nyurik
Copy link
Contributor

nyurik commented Feb 1, 2019

@bhousel thanks, looks good! What's the tooltip text?

@bhousel
Copy link
Member

bhousel commented Feb 1, 2019

@bhousel thanks, looks good! What's the tooltip text?

Oh yeah, it says "edit/translate" on the tooltip.. That just didn't get captured in the screenshot.

@nyurik
Copy link
Contributor

nyurik commented Feb 1, 2019

Thanks! Also, what happens when the data item exists, but it does not have description in either user's language nor English? I think we should show a big link "add description" in place of the description text? Note that most likely there will be no "view documentation" link, so it will be the only link shown. Perhaps we should show some big question mark instead of an image too?

@bhousel bhousel added this to the 2.14.0 milestone Feb 1, 2019
@bhousel
Copy link
Member

bhousel commented Feb 2, 2019

Thanks! Also, what happens when the data item exists, but it does not have description in either user's language nor English?

screenshot 2019-02-01 16 51 10

this seems fine..

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

No branches or pull requests

3 participants