iD icon indicating copy to clipboard operation
iD copied to clipboard

Don't suggest values from Taginfo for `name:xx` tags

Open Dimitar5555 opened this issue 1 year ago • 5 comments

URL

No response

How to reproduce the issue?

  1. Create an element
  2. Go to it's tags
  3. Add name:xx
  4. Switch to the value box
  5. Start typing
  6. (notice the suggestions below)

Screenshot(s) or anything else?

image

Which deployed environments do you see the issue in?

Released version at openstreetmap.org/edit

What version numbers does this issue effect?

2.29.0

Which browsers are you seeing this problem on?

Firefox

Dimitar5555 avatar Jun 21 '24 13:06 Dimitar5555

Related discussion: https://github.com/openstreetmap/iD/issues/7485.

There’s a hard-coded list of keys that don’t get taginfo suggestions. It could be supplemented by a regular expression for keys starting with name:.

https://github.com/openstreetmap/iD/blob/49492a0e06aef71f90e96ab3ed50098df0f4ce93/modules/services/taginfo.js#L196-L209

1ec5 avatar Jun 22 '24 15:06 1ec5

@1ec5 @Dimitar5555 can I start working on it if no one is doing it??

draunger avatar Oct 17 '24 09:10 draunger

I don’t think anyone has submitted a pull request about this issue so far. (You’d see a link to it somewhere on this page.) Feel free to submit a PR once you’ve got something that you’d like feedback about, and we’ll try to respond when we’re able to.

1ec5 avatar Oct 17 '24 09:10 1ec5

By the way, I noted above that we’re hard-coding a list of “popular” keys to ignore, but there’s a large, technically unbounded number of possible name:* keys. So rather than trying to enumerate all of them, consider looking at where this _popularKeys variable is used and adjusting the logic there.

1ec5 avatar Oct 17 '24 10:10 1ec5

@1ec5 ok sir

draunger avatar Oct 17 '24 10:10 draunger

hey @1ec5 i have done this issue and here the PR of this issue https://github.com/openstreetmap/iD/pull/10521 you can review that PR as soon as possible.

draunger avatar Nov 01 '24 12:11 draunger