SearchBox enhancements

[zbycz]

Few ideas - each one should be one PR

Fixes:

• fix matching presets by name – searching for building p should offer Building Part on the first place

Explorations

• find what is practical limit for overpass query size, eg if user asks for op:nwr, it can freeze their browser.

Features

• add search history -> if I searched for San Francisco, it should be offered to me
• when coordinates are out of BBox, show an arrow like this
• allow users to share search -> add search term to URL after 2 seconds eg /?q=op:node["shop"] #119
• add type coordinates – parse all coordinate types
• add type osmId – parse shortId, apiId or URL with some of these ids.

Completed

• fix broken flyTo on mobile – search for anything with bbox, eg Bexhill-on-Sea - it can't be clicked (SearchBox: Fix broken flyTo on mobile #574)
• filter through favourites points while typing (SearchBox: Refactoring and searchable stars #552)

[Dlurak]

fix broken flyTo on mobile – search for anything with bbox, eg Bexhill-on-Sea - it can't be clicked

On my iOS device it works, but only with a double click.

[amenk]

allow users to share search -> add search term to URL after 2 seconds eg /?q=op:node["shop"]

depends on #119

add type coordinates – parse all coordinate types

see also #510

[Dlurak]

How about also adding types to the options?
Even though it's internal, I believe this would greatly simplify future enhancements to the search box and make it easier to maintain!

[zbycz]

 How about also adding types to the options?

@Dlurak – yes, please! There is a TODO in code already, though we would need to completely refactor the types. Typescript can't discriminate among the different options based on our current shape. It need explicit type field with enum

// from
{
  star: { ... }
}
// to
{
  type: "star",    // enum "star" | "geocoder" | "preset" | "overpass" | "coordinates" | "osmId" | "loader"
  ...
}

I will review your #552 today, and then you can add the types. If you decided to do that, please, create one PR only for the refactoring, and second PR (based on first) with the additional types with adding new functionality. Otherwise the code review is a headache 😃

//edit @Dlurak - wow, I am reviewing the #552 and I am very pleasantly surprised! You already added the type: "x" all by yourself! Good job!!