diff --git a/assets/layers/scouting_group/scouting_group.json b/assets/layers/scouting_group/scouting_group.json index 31a4becb1a..26c15965e2 100644 --- a/assets/layers/scouting_group/scouting_group.json +++ b/assets/layers/scouting_group/scouting_group.json @@ -115,7 +115,23 @@ "mastodon" ], "filter": [ - "nsi_brand.brand" + "nsi_brand.brand", + { + "id": "brand_search", + "options": [ + { + "osmTags": "brand~i~.*{search}.*", + "fields": [ + { + "name": "search" + } + ], + "question": { + "en": "Search for brand: {search}" + } + } + ] + } ], "allowMove": true, "credits": "Osmwithspace", diff --git a/assets/layers/wayside_shrines/shrine.svg b/assets/layers/wayside_shrines/shrine.svg new file mode 100644 index 0000000000..1d13b2ef28 --- /dev/null +++ b/assets/layers/wayside_shrines/shrine.svg @@ -0,0 +1,31 @@ + + + + + + image/svg+xml + + + + + + + + diff --git a/assets/layers/wayside_shrines/shrine.svg.license b/assets/layers/wayside_shrines/shrine.svg.license new file mode 100644 index 0000000000..547f873e6c --- /dev/null +++ b/assets/layers/wayside_shrines/shrine.svg.license @@ -0,0 +1,2 @@ +SPDX-FileCopyrightText: Wanderwütiger +SPDX-License-Identifier: CC0-1.0 diff --git a/assets/layers/wayside_shrines/shrine_example1.jpg b/assets/layers/wayside_shrines/shrine_example1.jpg new file mode 100644 index 0000000000..6093b3f590 Binary files /dev/null and b/assets/layers/wayside_shrines/shrine_example1.jpg differ diff --git a/assets/layers/wayside_shrines/shrine_example1.jpg.license b/assets/layers/wayside_shrines/shrine_example1.jpg.license new file mode 100644 index 0000000000..0ed57fe73b --- /dev/null +++ b/assets/layers/wayside_shrines/shrine_example1.jpg.license @@ -0,0 +1,2 @@ +SPDX-FileCopyrightText: Bastian Greshake Tzovaras +SPDX-License-Identifier: CC-BY-SA-4.0 diff --git a/assets/layers/wayside_shrines/shrine_example2.jpg b/assets/layers/wayside_shrines/shrine_example2.jpg new file mode 100644 index 0000000000..a76edb087c Binary files /dev/null and b/assets/layers/wayside_shrines/shrine_example2.jpg differ diff --git a/assets/layers/wayside_shrines/shrine_example2.jpg.license b/assets/layers/wayside_shrines/shrine_example2.jpg.license new file mode 100644 index 0000000000..0ed57fe73b --- /dev/null +++ b/assets/layers/wayside_shrines/shrine_example2.jpg.license @@ -0,0 +1,2 @@ +SPDX-FileCopyrightText: Bastian Greshake Tzovaras +SPDX-License-Identifier: CC-BY-SA-4.0 diff --git a/assets/layers/wayside_shrines/shrine_example3.jpg b/assets/layers/wayside_shrines/shrine_example3.jpg new file mode 100644 index 0000000000..4d909a06d3 Binary files /dev/null and b/assets/layers/wayside_shrines/shrine_example3.jpg differ diff --git a/assets/layers/wayside_shrines/shrine_example3.jpg.license b/assets/layers/wayside_shrines/shrine_example3.jpg.license new file mode 100644 index 0000000000..0ed57fe73b --- /dev/null +++ b/assets/layers/wayside_shrines/shrine_example3.jpg.license @@ -0,0 +1,2 @@ +SPDX-FileCopyrightText: Bastian Greshake Tzovaras +SPDX-License-Identifier: CC-BY-SA-4.0 diff --git a/assets/layers/wayside_shrines/wayside_shrines.json b/assets/layers/wayside_shrines/wayside_shrines.json new file mode 100644 index 0000000000..521bc9ba7c --- /dev/null +++ b/assets/layers/wayside_shrines/wayside_shrines.json @@ -0,0 +1,335 @@ +{ + "id": "wayside_shrine", + "name": { + "en": "Wayside Shrines" + }, + "description": { + "en": "Shrines are religious places that are dedicated to specific deities, saints and other figures of religious importance. Typically, the contain religious depictions and people frequently leave offerings at those places. Wayside shrines are small small shrines that can be found next to a road or pathway and are frequented by travellers passing by." + }, + "source": { + "osmTags": "historic=wayside_shrine" + }, + "minzoom": 12, + "title": { + "render": { + "en": "Wayside Shrine {name}" + } + }, + "pointRendering": [ + { + "iconSize": "40,40", + "location": [ + "point" + ], + "anchor": "center", + "marker": [ + { + "icon": "circle", + "color": "white" + }, + { + "icon": "./assets/layers/wayside_shrines/shrine.svg" + } + ] + } + ], + "lineRendering": [ + { + "color": "#00f", + "width": "8" + } + ], + "presets": [ + { + "tags": [ + "historic=wayside_shrine" + ], + "title": { + "en": "a wayside shrine" + }, + "description": { + "en": "A wayside shrine typically shows a religious depiction, usually placed by a road or pathway. " + }, + "exampleImages": [ + "./assets/layers/wayside_shrines/shrine_example1.jpg", + "./assets/layers/wayside_shrines/shrine_example2.jpg", + "./assets/layers/wayside_shrines/shrine_example3.jpg" + ] + } + ], + "tagRenderings": [ + "images", + { + "question": { + "en": "What's the name of this shrine?" + }, + "id": "shrine_name", + "freeform": { + "key": "name", + "type": "string" + }, + "render": { + "en": "The name of this shrine is {name}" + } + }, + { + "question": { + "en": "To which religion is this shrine dedicated?" + }, + "id": "religion", + "mappings": [ + { + "if": "religion=christian", + "then": { + "en": "This is a Christian shrine" + } + }, + { + "if": "religion=buddhist", + "then": { + "en": "This is a Buddhist shrine" + } + }, + { + "if": "religion=hindu", + "then": { + "en": "This is a Hindu shrine" + } + }, + { + "if": "religion=jain", + "then": { + "en": "This is a Jain shrine" + } + }, + { + "if": "religion=jewish", + "then": { + "en": "This is a Jewish shrine" + } + }, + { + "if": "religion=muslim", + "then": { + "en": "This is an Islamic shrine" + } + }, + { + "if": "religion=pagan", + "then": { + "en": "This is a Pagan shrine" + } + }, + { + "if": "religion=shinto", + "then": { + "en": "This is a Shinto shrine" + } + }, + { + "if": "religion=sikh", + "then": { + "en": "This is a Sikh shrine" + } + }, + { + "if": "religion=taoist", + "then": { + "en": "This is a Taoist shrine" + } + }, + { + "if": "religion=zoroastrian", + "then": { + "en": "This is a Zoroastrian shrine" + } + } + ], + "multiAnswer": false, + "freeform": { + "key": "religion", + "type": "string" + }, + "render": { + "en": "This shrine is {religion}" + } + }, + { + "question": { + "en": "What's the Christian denomination of the shrine?" + }, + "id": "denomination_christian", + "condition": "religion=christian", + "mappings": [ + { + "if": "denomination=catholic", + "then": { + "en": "It's denomination is: Catholic" + } + }, + { + "if": "denomination=roman_catholic", + "then": { + "en": "It's denomination is: Roman Catholic" + } + }, + { + "if": "denomination=orthodox", + "then": { + "en": "It's denomination is Orthodox" + } + }, + { + "if": "denomination=greek_orthodox", + "then": { + "en": "It's denomination is Greek-Orthodox" + } + }, + { + "if": "denomination=russian_orthodox", + "then": { + "en": "It's denomination is Russian-Orthodox" + } + }, + { + "if": "denomination=serbian_orthodox", + "then": { + "en": "It's denomination is Serbian Orthodox" + } + }, + { + "if": "denomination=protestant", + "then": { + "en": "It's denomination is Protestant" + } + }, + { + "if": "denomination=anglican", + "then": { + "en": "It's denomination is Anglican" + } + }, + { + "if": "denomination=adventist", + "then": { + "en": "It's denomination is Adventist" + } + }, + { + "if": "denomination=evangelical", + "then": { + "en": "It's denomination is evangelical" + } + } + ], + "render": { + "en": "Other denomination: {denomination}" + }, + "freeform": { + "key": "denomination" + } + }, + { + "mappings": [ + { + "if": "denomination=shia", + "then": { + "en": "It's denomination is Shia" + } + }, + { + "if": "denomination=sunni", + "then": { + "en": "It's denomination is Sunni" + } + }, + { + "if": "denomination=sufi", + "then": { + "en": "It's denomination is Sufi" + } + } + ], + "id": "denomination_muslim", + "freeform": { + "key": "denomination" + }, + "render": { + "en": "It's denomination is {denomination}" + }, + "question": { + "en": "What's the Muslim denomination of this shrine?" + }, + "condition": "religion=muslim" + }, + { + "mappings": [ + { + "if": "denomination=conservative", + "then": { + "en": "It's denomination is Conservative" + } + }, + { + "if": "denomination=orthodox", + "then": { + "en": "It's denomination is Orthodox" + } + }, + { + "if": "denomination=hasidic", + "then": { + "en": "It's denomination is Hasidic" + } + }, + { + "if": "denomination=reform", + "then": { + "en": "It's denomination is Reform" + } + } + ], + "id": "denomination_jewish", + "freeform": { + "key": "denomination" + }, + "render": { + "en": "It's denomination is {denomination}" + }, + "question": { + "en": "What's the Jewish denomination of this shrine?" + }, + "condition": "religion=jewish" + }, + { + "question": { + "en": "What's the denomination of this shrine?" + }, + "id": "denomination_other", + "render": { + "en": "The denomination of this shrine is: {denomination}" + }, + "freeform": { + "key": "denomination" + }, + "condition": { + "and": [ + "religion!=christian", + "religion!=muslim", + "religion!=jewish" + ] + } + } + ], + "deletion": { + "softDeletionTags": { + "and": [ + "disused:historic:={historic}", + "historic=" + ] + }, + "neededChangesets": 1 + }, + "credits": "Bastian Greshake Tzovaras", + "credits:uid": 20617622 +} diff --git a/assets/themes/scouting/scouting.json b/assets/themes/scouting/scouting.json index 7323ac68e2..65b48679c3 100644 --- a/assets/themes/scouting/scouting.json +++ b/assets/themes/scouting/scouting.json @@ -26,9 +26,13 @@ "=osmTags": { "and": [ "tourism=camp_site", + "scout!=no", + "group!=no", { "or": [ "scout=yes", + "scout=only", + "group=yes", "group_only=yes" ] } @@ -62,9 +66,13 @@ "=osmTags": { "and": [ "tourism=hostel", + "scout!=no", + "group!=no", { "or": [ "scout=yes", + "scout=only", + "group=yes", "group_only=yes" ] } diff --git a/assets/themes/wayside_shrines/shrine.svg b/assets/themes/wayside_shrines/shrine.svg new file mode 100644 index 0000000000..1d13b2ef28 --- /dev/null +++ b/assets/themes/wayside_shrines/shrine.svg @@ -0,0 +1,31 @@ + + + + + + image/svg+xml + + + + + + + + diff --git a/assets/themes/wayside_shrines/shrine.svg.license b/assets/themes/wayside_shrines/shrine.svg.license new file mode 100644 index 0000000000..547f873e6c --- /dev/null +++ b/assets/themes/wayside_shrines/shrine.svg.license @@ -0,0 +1,2 @@ +SPDX-FileCopyrightText: Wanderwütiger +SPDX-License-Identifier: CC0-1.0 diff --git a/assets/themes/wayside_shrines/wayside_shrines.json b/assets/themes/wayside_shrines/wayside_shrines.json new file mode 100644 index 0000000000..8c1a249a3f --- /dev/null +++ b/assets/themes/wayside_shrines/wayside_shrines.json @@ -0,0 +1,13 @@ +{ + "id": "wayside_shrines", + "title": { + "en": "Wayside shrines" + }, + "description": { + "en": "This map shows shrines found on the side of roads and paths, and allows adding new ones" + }, + "icon": "./assets/themes/wayside_shrines/shrine.svg", + "layers": [ + "wayside_shrines" + ] +} diff --git a/src/Logic/FeatureSource/Actors/SaveFeatureSourceToLocalStorage.ts b/src/Logic/FeatureSource/Actors/SaveFeatureSourceToLocalStorage.ts index d9505822b6..c9cb4b04c0 100644 --- a/src/Logic/FeatureSource/Actors/SaveFeatureSourceToLocalStorage.ts +++ b/src/Logic/FeatureSource/Actors/SaveFeatureSourceToLocalStorage.ts @@ -76,7 +76,7 @@ export default class SaveFeatureSourceToLocalStorage { const storage = TileLocalStorage.construct(backend, layername, maxCacheAge) this.storage = storage const singleTileSavers: Map = new Map() - features.features.addCallbackAndRunD((features) => { + features.features.stabilized(5000).addCallbackAndRunD((features) => { if ( features.some((f) => { let totalPoints = 0 @@ -116,7 +116,7 @@ export default class SaveFeatureSourceToLocalStorage { tileSaver = new SingleTileSaver(src, featureProperties) singleTileSavers.set(tileIndex, tileSaver) } - // Don't cache not-uploaded features yet - they'll be cached when the receive their id + // Don't cache not-uploaded features yet - they'll be cached when they receive their id features = features.filter((f) => !f.properties.id.match(/(node|way)\/-[0-9]+/)) tileSaver.saveFeatures(features) }) diff --git a/src/Logic/Search/CombinedSearcher.ts b/src/Logic/Search/CombinedSearcher.ts index 07affc4c12..10bf7c2f9b 100644 --- a/src/Logic/Search/CombinedSearcher.ts +++ b/src/Logic/Search/CombinedSearcher.ts @@ -1,8 +1,4 @@ -import GeocodingProvider, { - GeocodeResult, - GeocodingOptions, - SearchResult, -} from "./GeocodingProvider" +import GeocodingProvider, { GeocodeResult, GeocodingOptions } from "./GeocodingProvider" import { Utils } from "../../Utils" import { Store, Stores } from "../UIEventSource" @@ -44,12 +40,12 @@ export default class CombinedSearcher implements GeocodingProvider { return results } - async search(query: string, options?: GeocodingOptions): Promise { + async search(query: string, options?: GeocodingOptions): Promise { const results = await Promise.all(this._providers.map((pr) => pr.search(query, options))) return CombinedSearcher.merge(results) } - suggest(query: string, options?: GeocodingOptions): Store { + suggest(query: string, options?: GeocodingOptions): Store { return Stores.concat( this._providersWithSuggest.map((pr) => pr.suggest(query, options)) ).map((gcrss) => CombinedSearcher.merge(gcrss)) diff --git a/src/Logic/Search/GeocodingFeatureSource.ts b/src/Logic/Search/GeocodingFeatureSource.ts index f4f3960f61..a5c6723d08 100644 --- a/src/Logic/Search/GeocodingFeatureSource.ts +++ b/src/Logic/Search/GeocodingFeatureSource.ts @@ -1,4 +1,4 @@ -import { SearchResult } from "./GeocodingProvider" +import { GeocodeResult } from "./GeocodingProvider" import { Store } from "../UIEventSource" import { FeatureSource } from "../FeatureSource/FeatureSource" import { Feature, Geometry } from "geojson" @@ -6,7 +6,7 @@ import { Feature, Geometry } from "geojson" export default class GeocodingFeatureSource implements FeatureSource { public features: Store>[]> - constructor(provider: Store) { + constructor(provider: Store) { this.features = provider.map((geocoded) => { if (geocoded === undefined) { return [] diff --git a/src/Logic/Search/GeocodingProvider.ts b/src/Logic/Search/GeocodingProvider.ts index a698e3aa9d..ebd2b9d8a5 100644 --- a/src/Logic/Search/GeocodingProvider.ts +++ b/src/Logic/Search/GeocodingProvider.ts @@ -42,7 +42,6 @@ export type GeocodeResult = { payload?: object source?: string } -export type SearchResult = GeocodeResult export interface GeocodingOptions { bbox?: BBox diff --git a/src/Logic/Search/LocalElementSearch.ts b/src/Logic/Search/LocalElementSearch.ts index 4e3d180f53..9db1de6542 100644 --- a/src/Logic/Search/LocalElementSearch.ts +++ b/src/Logic/Search/LocalElementSearch.ts @@ -1,4 +1,4 @@ -import GeocodingProvider, { GeocodingOptions, SearchResult } from "./GeocodingProvider" +import GeocodingProvider, { GeocodeResult, GeocodingOptions } from "./GeocodingProvider" import ThemeViewState from "../../Models/ThemeViewState" import { Utils } from "../../Utils" import { Feature } from "geojson" @@ -26,7 +26,7 @@ export default class LocalElementSearch implements GeocodingProvider { this._limit = limit } - async search(query: string, options?: GeocodingOptions): Promise { + async search(query: string, options?: GeocodingOptions): Promise { return this.searchEntries(query, options, false).data } @@ -92,7 +92,7 @@ export default class LocalElementSearch implements GeocodingProvider { query: string, options?: GeocodingOptions, matchStart?: boolean - ): Store { + ): Store { if (query.length < 3) { return new ImmutableStore([]) } @@ -126,7 +126,7 @@ export default class LocalElementSearch implements GeocodingProvider { } return results.map((entry) => { const [osm_type, osm_id] = entry.feature.properties.id.split("/") - return { + return { lon: entry.center[0], lat: entry.center[1], osm_type, @@ -141,7 +141,7 @@ export default class LocalElementSearch implements GeocodingProvider { }) } - suggest(query: string, options?: GeocodingOptions): Store { + suggest(query: string, options?: GeocodingOptions): Store { return this.searchEntries(query, options, true) } } diff --git a/src/Logic/Search/NominatimGeocoding.ts b/src/Logic/Search/NominatimGeocoding.ts index f191306093..9371fab457 100644 --- a/src/Logic/Search/NominatimGeocoding.ts +++ b/src/Logic/Search/NominatimGeocoding.ts @@ -3,7 +3,7 @@ import { BBox } from "../BBox" import Constants from "../../Models/Constants" import { FeatureCollection } from "geojson" import Locale from "../../UI/i18n/Locale" -import GeocodingProvider, { GeocodingOptions, SearchResult } from "./GeocodingProvider" +import GeocodingProvider, { GeocodeResult, GeocodingOptions } from "./GeocodingProvider" export class NominatimGeocoding implements GeocodingProvider { private readonly _host @@ -15,7 +15,7 @@ export class NominatimGeocoding implements GeocodingProvider { this._host = host } - public search(query: string, options?: GeocodingOptions): Promise { + public search(query: string, options?: GeocodingOptions): Promise { const b = options?.bbox ?? BBox.global const url = `${this._host}search?format=json&limit=${ this.limit diff --git a/src/Logic/SimpleMetaTagger.ts b/src/Logic/SimpleMetaTagger.ts index 708b29bfaa..51c905cbf6 100644 --- a/src/Logic/SimpleMetaTagger.ts +++ b/src/Logic/SimpleMetaTagger.ts @@ -11,6 +11,8 @@ import { UIEventSource } from "./UIEventSource" import ThemeConfig from "../Models/ThemeConfig/ThemeConfig" import OsmObjectDownloader from "./Osm/OsmObjectDownloader" import countryToCurrency from "country-to-currency" +import { Unit } from "../Models/Unit" +import { Denomination } from "../Models/Denomination" /** * All elements that are needed to perform metatagging @@ -476,10 +478,8 @@ export default class SimpleMetaTaggers { doc: "If 'units' is defined in the layoutConfig, then this metatagger will rewrite the specified keys to have the canonical form (e.g. `1meter` will be rewritten to `1m`; `1` will be rewritten to `1m` as well)", keys: ["Theme-defined keys"], }, - (feature, _, __, state) => { - const units = Utils.NoNull( - [].concat(...(state?.theme?.layers?.map((layer) => layer.units) ?? [])) - ) + (feature, layer, __, state) => { + const units: Unit[] = layer.units if (units.length == 0) { return } @@ -497,7 +497,7 @@ export default class SimpleMetaTaggers { continue } const value = feature.properties[key] - const denom = unit.findDenomination(value, () => feature.properties["_country"]) + const denom: [string, Denomination] = unit.findDenomination(value, () => feature.properties["_country"]) if (denom === undefined) { // no valid value found break @@ -515,7 +515,7 @@ export default class SimpleMetaTaggers { if (canonical === value) { break } - console.log("Rewritten ", key, ` from '${value}' into '${canonical}'`) + console.log("Rewritten ", key, ` from '${value}' into '${canonical}' due to denomination`, denomination) if (canonical === undefined && !unit.eraseInvalid) { break } diff --git a/src/Logic/State/SearchState.ts b/src/Logic/State/SearchState.ts index 642a64fa36..01746cb6a7 100644 --- a/src/Logic/State/SearchState.ts +++ b/src/Logic/State/SearchState.ts @@ -1,4 +1,4 @@ -import GeocodingProvider, { type SearchResult } from "../Search/GeocodingProvider" +import GeocodingProvider, { GeocodeResult, GeocodingUtils } from "../Search/GeocodingProvider" import { ImmutableStore, Store, Stores, UIEventSource } from "../UIEventSource" import CombinedSearcher from "../Search/CombinedSearcher" import FilterSearch, { FilterSearchResult } from "../Search/FilterSearch" @@ -16,12 +16,13 @@ import LayerConfig from "../../Models/ThemeConfig/LayerConfig" import { FeatureSource } from "../FeatureSource/FeatureSource" import { Feature } from "geojson" import OpenLocationCodeSearch from "../Search/OpenLocationCodeSearch" +import { BBox } from "../BBox" export default class SearchState { public readonly feedback: UIEventSource = new UIEventSource(undefined) public readonly searchTerm: UIEventSource = new UIEventSource("") public readonly searchIsFocused = new UIEventSource(false) - public readonly suggestions: Store + public readonly suggestions: Store public readonly filterSuggestions: Store public readonly themeSuggestions: Store public readonly layerSuggestions: Store @@ -60,7 +61,7 @@ export default class SearchState { return new ImmutableStore(true) } return Stores.concat(suggestions).map((suggestions) => - suggestions.some((list, i) => list === undefined) + suggestions.some(list => list === undefined) ) }) this.suggestions = suggestionsList.bindD((suggestions) => @@ -100,7 +101,7 @@ export default class SearchState { this.showSearchDrawer = new UIEventSource(false) - this.searchIsFocused.addCallbackAndRunD((sugg) => { + this.searchIsFocused.addCallbackAndRunD(sugg => { if (sugg) { this.showSearchDrawer.set(true) } @@ -124,7 +125,6 @@ export default class SearchState { const state = this.state const layersToShow = payload.map((fsr) => fsr.layer.id) - console.log("Layers to show are", layersToShow) for (const otherLayer of state.layerState.filteredLayers.values()) { const layer = otherLayer.layerDef if (!layer.isNormal()) { @@ -167,4 +167,45 @@ export default class SearchState { this.state.featureProperties.trackFeature(f) this.state.selectedElement.set(f) } + + public moveToBestMatch() { + const suggestion = this.suggestions.data?.[0] + if (suggestion) { + this.applyGeocodeResult(suggestion) + } + if (this.suggestionsSearchRunning.data) { + this.suggestionsSearchRunning.addCallback(() => { + this.applyGeocodeResult(this.suggestions.data?.[0]) + return true // unregister + }) + } + } + + applyGeocodeResult(entry: GeocodeResult) { + if (!entry) { + console.error("ApplyGeocodeResult got undefined/null") + } + console.log("Moving to", entry.description) + const state = this.state + if (entry.boundingbox) { + const [lat0, lat1, lon0, lon1] = entry.boundingbox + state.mapProperties.bounds.set( + new BBox([ + [lon0, lat0], + [lon1, lat1] + ]).pad(0.01) + ) + } else { + state.mapProperties.flyTo( + entry.lon, + entry.lat, + GeocodingUtils.categoryToZoomLevel[entry.category] ?? 17 + ) + } + if (entry.feature?.properties?.id) { + state.selectedElement.set(entry.feature) + } + state.userRelatedState.recentlyVisitedSearch.add(entry) + this.closeIfFullscreen() + } } diff --git a/src/Logic/Tags/TagTypes.ts b/src/Logic/Tags/TagTypes.ts index c46d13126d..17e37d96e0 100644 --- a/src/Logic/Tags/TagTypes.ts +++ b/src/Logic/Tags/TagTypes.ts @@ -25,6 +25,11 @@ export class TagTypes { return and.and } + static uploadableAnd(and: And & UploadableTag): UploadableTag[] { + return and.and + } + + static safeOr(or: Or & OptimizedTag): ((FlatTag | (And & OptimizedTag)) & OptimizedTag)[] { return or.or } diff --git a/src/Logic/Tags/TagUtils.ts b/src/Logic/Tags/TagUtils.ts index c3933cdca8..aa92c24bd4 100644 --- a/src/Logic/Tags/TagUtils.ts +++ b/src/Logic/Tags/TagUtils.ts @@ -21,7 +21,7 @@ export class TagUtils { ["<=", (a, b) => a <= b], [">=", (a, b) => a >= b], ["<", (a, b) => a < b], - [">", (a, b) => a > b], + [">", (a, b) => a > b] ] public static modeDocumentation: Record< string, @@ -48,7 +48,7 @@ export class TagUtils { "### Removing a key\n" + "\n" + "If a key should be deleted in the OpenStreetMap-database, specify `key=` as well. This can be used e.g. to remove a\n" + - "fixme or value from another mapping if another field is filled out.", + "fixme or value from another mapping if another field is filled out." }, "!=": { name: "strict not equals", @@ -62,7 +62,7 @@ export class TagUtils { "### If key is present\n" + "\n" + "This implies that, to check if a key is present, `key!=` can be used. This will only match if the key is present and not\n" + - "empty.", + "empty." }, "~": { name: "Value matches regex", @@ -73,12 +73,12 @@ export class TagUtils { "The regex is put within braces as to prevent runaway values.\n" + "\nUse `key~*` to indicate that any value is allowed. This is effectively the check that the attribute is present (defined _and_ not empty)." + "\n" + - "Regexes will match the newline character with `.` too - the `s`-flag is enabled by default.", + "Regexes will match the newline character with `.` too - the `s`-flag is enabled by default." }, "~i~": { name: "Value matches case-invariant regex", overpassSupport: true, - docs: "A tag can also be tested against a regex with `key~i~regex`, where the case of the value will be ignored. The regex is still matched against the _entire_ value", + docs: "A tag can also be tested against a regex with `key~i~regex`, where the case of the value will be ignored. The regex is still matched against the _entire_ value" }, "!~": { name: "Value should _not_ match regex", @@ -87,27 +87,27 @@ export class TagUtils { "A tag can also be tested against a regex with `key!~regex`. This filter will match if the value does *not* match the regex. " + "\n If the\n" + "value is allowed to appear anywhere as substring, use `key~.*regex.*`.\n" + - "The regex is put within braces as to prevent runaway values.\n", + "The regex is put within braces as to prevent runaway values.\n" }, "!~i~": { name: "Value does *not* match case-invariant regex", overpassSupport: true, - docs: "A tag can also be tested against a regex with `key~i~regex`, where the case of the value will be ignored. The regex is still matched against the _entire_ value. This filter returns true if the value does *not* match", + docs: "A tag can also be tested against a regex with `key~i~regex`, where the case of the value will be ignored. The regex is still matched against the _entire_ value. This filter returns true if the value does *not* match" }, "~~": { name: "Key and value should match given regex", overpassSupport: true, - docs: "Both the `key` and `value` part of this specification are interpreted as regexes, both the key and value musth completely match their respective regexes", + docs: "Both the `key` and `value` part of this specification are interpreted as regexes, both the key and value musth completely match their respective regexes" }, "~i~~": { name: "Key and value should match a given regex; value is case-invariant", overpassSupport: true, - docs: "Similar to ~~, except that the value is case-invariant", + docs: "Similar to ~~, except that the value is case-invariant" }, "!~i~~": { name: "Key and value should match a given regex; value is case-invariant", overpassSupport: true, - docs: "Similar to !~~, except that the value is case-invariant", + docs: "Similar to !~~, except that the value is case-invariant" }, ":=": { name: "Substitute `... {some_key} ...` and match `key`", @@ -133,24 +133,24 @@ export class TagUtils { "\n" + "```json\n" + "{\n" + - ' "mappings": [\n' + + " \"mappings\": [\n" + " {\n" + - ' "if":"key:={some_other_key}",\n' + - ' "then": "...",\n' + - ' "hideInAnswer": "some_other_key="\n' + + " \"if\":\"key:={some_other_key}\",\n" + + " \"then\": \"...\",\n" + + " \"hideInAnswer\": \"some_other_key=\"\n" + " }\n" + " ]\n" + "}\n" + "```\n" + "\n" + "One can use `key!:=prefix-{other_key}-postfix` as well, to match if `key` is _not_ the same\n" + - "as `prefix-{other_key}-postfix` (with `other_key` substituted by the value)", + "as `prefix-{other_key}-postfix` (with `other_key` substituted by the value)" }, "!:=": { name: "Substitute `{some_key}` should not match `key`", overpassSupport: false, - docs: "See `:=`, except that this filter is inverted", - }, + docs: "See `:=`, except that this filter is inverted" + } } private static keyCounts: { keys: any; tags: any } = key_counts public static readonly numberAndDateComparisonDocs = @@ -175,10 +175,10 @@ export class TagUtils { "\n" + "```json\n" + "{\n" + - ' "osmTags": {\n' + - ' "or": [\n' + - ' "amenity=school",\n' + - ' "amenity=kindergarten"\n' + + " \"osmTags\": {\n" + + " \"or\": [\n" + + " \"amenity=school\",\n" + + " \"amenity=kindergarten\"\n" + " ]\n" + " }\n" + "}\n" + @@ -194,7 +194,7 @@ export class TagUtils { "If the schema-files note a type [`TagConfigJson`](https://github.com/pietervdvn/MapComplete/blob/develop/src/Models/ThemeConfig/Json/TagConfigJson.ts), you can use one of these values.\n" + "\n" + "In some cases, not every type of tags-filter can be used. For example, _rendering_ an option with a regex is\n" + - 'fine (`"if": "brand~[Bb]randname", "then":" The brand is Brandname"`); but this regex can not be used to write a value\n' + + "fine (`\"if\": \"brand~[Bb]randname\", \"then\":\" The brand is Brandname\"`); but this regex can not be used to write a value\n" + "into the database. The theme loader will however refuse to work with such inconsistencies and notify you of this while\n" + "you are building your theme.\n" + "\n" + @@ -205,18 +205,18 @@ export class TagUtils { "\n" + "```json\n" + "{\n" + - ' "and": [\n' + - ' "key=value",\n' + + " \"and\": [\n" + + " \"key=value\",\n" + " {\n" + - ' "or": [\n' + - ' "other_key=value",\n' + - ' "other_key=some_other_value"\n' + + " \"or\": [\n" + + " \"other_key=value\",\n" + + " \"other_key=some_other_value\"\n" + " ]\n" + " },\n" + - ' "key_which_should_be_missing=",\n' + - ' "key_which_should_have_a_value~*",\n' + - ' "key~.*some_regex_a*_b+_[a-z]?",\n' + - ' "height<1"\n' + + " \"key_which_should_be_missing=\",\n" + + " \"key_which_should_have_a_value~*\",\n" + + " \"key~.*some_regex_a*_b+_[a-z]?\",\n" + + " \"height<1\"\n" + " ]\n" + "}\n" + "```\n" + @@ -374,9 +374,9 @@ export class TagUtils { * TagUtils.FlattenMultiAnswer(([new Tag("x","y"), new Tag("a","b")])) // => new And([new Tag("x","y"), new Tag("a","b")]) * TagUtils.FlattenMultiAnswer(([new Tag("x","")])) // => new And([new Tag("x","")]) */ - static FlattenMultiAnswer(tagsFilters: UploadableTag[]): And { + static FlattenMultiAnswer(tagsFilters: UploadableTag[]): UploadableTag[] { if (tagsFilters === undefined) { - return new And([]) + return [] } const keyValues = TagUtils.SplitKeys(tagsFilters) @@ -386,7 +386,7 @@ export class TagUtils { values.sort() and.push(new Tag(key, values.join(";"))) } - return new And(and) + return and } /** @@ -985,10 +985,10 @@ export class TagUtils { return ["", "## `" + mode + "` " + doc.name, "", doc.docs, "", ""].join("\n") }), "## " + - TagUtils.comparators.map((comparator) => "`" + comparator[0] + "`").join(" ") + - " Logical comparators", + TagUtils.comparators.map((comparator) => "`" + comparator[0] + "`").join(" ") + + " Logical comparators", TagUtils.numberAndDateComparisonDocs, - TagUtils.logicalOperator, + TagUtils.logicalOperator ].join("\n") } diff --git a/src/Logic/Web/MangroveReviews.ts b/src/Logic/Web/MangroveReviews.ts index d94c300b91..77271ee339 100644 --- a/src/Logic/Web/MangroveReviews.ts +++ b/src/Logic/Web/MangroveReviews.ts @@ -177,7 +177,6 @@ export default class FeatureReviews { testmode?: Store, loadingAllowed?: UIEventSource ) { - console.trace(">>> Creating FeatureReviews", options) this.loadingAllowed = loadingAllowed const centerLonLat = GeoOperations.centerpointCoordinates(feature) ;[this._lon, this._lat] = centerLonLat @@ -217,12 +216,10 @@ export default class FeatureReviews { } this._name = tagsSource.map((tags) => { const defaultName = tags[nameKey] - console.trace(">>>", options, defaultName) if (defaultName && defaultName !== "") { console.log("Using default name:", defaultName, "fallback:", options.fallbackName) return defaultName } - console.trace("Using fallback name", options?.fallbackName, options) return options?.fallbackName }) diff --git a/src/Models/MenuState.ts b/src/Models/MenuState.ts index 322fd01214..f2ff936f86 100644 --- a/src/Models/MenuState.ts +++ b/src/Models/MenuState.ts @@ -4,6 +4,7 @@ import UserRelatedState from "../Logic/State/UserRelatedState" import { Utils } from "../Utils" import Zoomcontrol from "../UI/Zoomcontrol" import { LocalStorageSource } from "../Logic/Web/LocalStorageSource" +import { ProvidedImage } from "../Logic/ImageProviders/ImageProvider" export type PageType = (typeof MenuState.pageNames)[number] @@ -27,7 +28,7 @@ export class MenuState { "favourites", "usersettings", "share", - "menu", + "menu" ] as const /** @@ -38,6 +39,9 @@ export class MenuState { undefined ) + public static readonly nearbyImagesFeature: UIEventSource = new UIEventSource( + undefined + ) public readonly pageStates: Record> public readonly highlightedLayerInFilters: UIEventSource = new UIEventSource( @@ -45,6 +49,7 @@ export class MenuState { ) public highlightedUserSetting: UIEventSource = new UIEventSource(undefined) private readonly _selectedElement: UIEventSource | undefined + private isClosingAll = false constructor(selectedElement: UIEventSource | undefined) { this._selectedElement = selectedElement @@ -129,29 +134,49 @@ export class MenuState { * Returns 'true' if at least one menu was opened */ public closeAll(): boolean { - console.log("Closing all") + if (this.isClosingAll) { + return true + } + this.isClosingAll = true const ps = this.pageStates - if (ps.menu.data) { - ps.menu.set(false) - return true - } + try { - if (MenuState.previewedImage.data !== undefined) { - MenuState.previewedImage.setData(undefined) - return true - } - - for (const key in ps) { - const toggle = ps[key] - const wasOpen = toggle.data - toggle.setData(false) - if (wasOpen) { + if (ps.menu.data) { + ps.menu.set(false) return true } - } - if (this._selectedElement.data) { - this._selectedElement.setData(undefined) - return true + + if (MenuState.previewedImage.data !== undefined) { + MenuState.previewedImage.setData(undefined) + return true + } + + if (MenuState.nearbyImagesFeature.data !== undefined) { + MenuState.nearbyImagesFeature.setData(undefined) + return true + } + for (const key in ps) { + const toggle = ps[key] + const wasOpen = toggle.data + toggle.setData(false) + if (wasOpen) { + return true + } + } + if (this._selectedElement.data) { + this._selectedElement.setData(undefined) + return true + } + } finally { + this.isClosingAll = false } } + + public setPreviewedImage(img?: Partial) { + if (img === undefined && !this.isClosingAll) { + return + } + MenuState.previewedImage.setData(img) + + } } diff --git a/src/Models/RasterLayers.ts b/src/Models/RasterLayers.ts index 0e0cb0e79f..de192084d5 100644 --- a/src/Models/RasterLayers.ts +++ b/src/Models/RasterLayers.ts @@ -116,7 +116,7 @@ export class AvailableRasterLayers { availableLayersBboxes.map( (eliPolygons) => { const loc = location.data - const lonlat: [number, number] = [loc.lon, loc.lat] + const lonlat: [number, number] = [loc?.lon ?? 0, loc?.lat ?? 0] const matching: RasterLayerPolygon[] = eliPolygons.filter((eliPolygon) => { if (eliPolygon.geometry === null) { return true // global ELI-layer diff --git a/src/Models/ThemeConfig/TagRenderingConfig.ts b/src/Models/ThemeConfig/TagRenderingConfig.ts index 07e559401a..4db5a4f051 100644 --- a/src/Models/ThemeConfig/TagRenderingConfig.ts +++ b/src/Models/ThemeConfig/TagRenderingConfig.ts @@ -223,7 +223,7 @@ export default class TagRenderingConfig { inline: json.freeform.inline ?? false, default: json.freeform.default, postfixDistinguished: json.freeform.postfixDistinguished?.trim(), - args: json.freeform.helperArgs, + args: json.freeform.helperArgs } if (json.freeform["extraTags"] !== undefined) { throw `Freeform.extraTags is defined. This should probably be 'freeform.addExtraTag' (at ${context})` @@ -447,7 +447,7 @@ export default class TagRenderingConfig { iconClass, addExtraTags, searchTerms: mapping.searchTerms, - priorityIf: prioritySearch, + priorityIf: prioritySearch } if (isQuestionable) { if (hideInAnswer !== true && mp.if !== undefined && !mp.if.isUsableAsAnswer()) { @@ -554,7 +554,7 @@ export default class TagRenderingConfig { then: new TypedTranslation( this.render.replace("{" + this.freeform.key + "}", leftover).translations, this.render.context - ), + ) }) } } @@ -607,7 +607,7 @@ export default class TagRenderingConfig { return { then: this.render.PartialSubs({ [this.freeform.key]: v.trim() }), icon: this.renderIcon, - iconClass: this.renderIconClass, + iconClass: this.renderIconClass } } } @@ -662,7 +662,7 @@ export default class TagRenderingConfig { key: commonKey, values: Utils.NoNull( values.map((arr) => arr.filter((item) => item.k === commonKey)[0]?.v) - ), + ) } } @@ -677,7 +677,7 @@ export default class TagRenderingConfig { return { key, type: this.freeform.type, - values, + values } } catch (e) { console.error("Could not create FreeformValues for tagrendering", this.id) @@ -734,7 +734,7 @@ export default class TagRenderingConfig { singleSelectedMapping: number, multiSelectedMapping: boolean[] | undefined, currentProperties: Record - ): UploadableTag { + ): UploadableTag[] { if (typeof freeformValue === "string") { freeformValue = freeformValue?.trim() } @@ -746,14 +746,18 @@ export default class TagRenderingConfig { if (freeformValue === "") { freeformValue = undefined } - if (this.freeform?.postfixDistinguished && freeformValue !== undefined) { + if (this.freeform?.postfixDistinguished) { const allValues = currentProperties[this.freeform.key].split(";").map((s) => s.trim()) const perPostfix: Record = {} for (const value of allValues) { const [v, postfix] = value.split("/") perPostfix[postfix.trim()] = v.trim() } - perPostfix[this.freeform.postfixDistinguished] = freeformValue + if (freeformValue === "" || freeformValue === undefined) { + delete perPostfix[this.freeform.postfixDistinguished] + } else { + perPostfix[this.freeform.postfixDistinguished] = freeformValue + } const keys = Object.keys(perPostfix) keys.sort() freeformValue = keys.map((k) => perPostfix[k] + "/" + k).join("; ") @@ -778,14 +782,14 @@ export default class TagRenderingConfig { const freeformOnly = { [this.freeform.key]: freeformValue } const matchingMapping = this.mappings?.find((m) => m.if.matchesProperties(freeformOnly)) if (matchingMapping) { - return new And([matchingMapping.if, ...(matchingMapping.addExtraTags ?? [])]) + return [matchingMapping.if, ...(matchingMapping.addExtraTags ?? [])] } // Either no mappings, or this is a radio-button selected freeform value - const tag = new And([ + const tag = [ new Tag(this.freeform.key, freeformValue), - ...(this.freeform.addExtraTags ?? []), - ]) - const newProperties = tag.applyOn(currentProperties) + ...(this.freeform.addExtraTags ?? []) + ] + const newProperties = new And(tag).applyOn(currentProperties) if (this.invalidValues?.matchesProperties(newProperties)) { return undefined } @@ -807,19 +811,14 @@ export default class TagRenderingConfig { selectedMappings.push( new And([ new Tag(this.freeform.key, freeformValue), - ...(this.freeform.addExtraTags ?? []), + ...(this.freeform.addExtraTags ?? []) ]) ) } const and = TagUtils.FlattenMultiAnswer([...selectedMappings, ...unselectedMappings]) - if (and.and.length === 0) { + if (and.length === 0) { return undefined } - console.log( - ">>> New properties", - TagUtils.asProperties(and, currentProperties), - this.invalidValues - ) if ( this.invalidValues?.matchesProperties(TagUtils.asProperties(and, currentProperties)) ) { @@ -843,24 +842,23 @@ export default class TagRenderingConfig { !someMappingIsShown || singleSelectedMapping === undefined) if (useFreeform) { - return new And([ + return [ new Tag(this.freeform.key, freeformValue), - ...(this.freeform.addExtraTags ?? []), - ]) + ...(this.freeform.addExtraTags ?? []) + ] } else if (singleSelectedMapping !== undefined) { - return new And([ + return [ this.mappings[singleSelectedMapping].if, - ...(this.mappings[singleSelectedMapping].addExtraTags ?? []), - ]) + ...(this.mappings[singleSelectedMapping].addExtraTags ?? []) + ] } else { console.error("TagRenderingConfig.ConstructSpecification has a weird fallback for", { freeformValue, singleSelectedMapping, multiSelectedMapping, currentProperties, - useFreeform, + useFreeform }) - return undefined } } @@ -888,11 +886,11 @@ export default class TagRenderingConfig { } const msgs: string[] = [ icon + - " " + - "*" + - m.then.textFor(lang) + - "* is shown if with " + - m.if.asHumanString(true, false, {}), + " " + + "*" + + m.then.textFor(lang) + + "* is shown if with " + + m.if.asHumanString(true, false, {}) ] if (m.hideInAnswer === true) { @@ -901,7 +899,7 @@ export default class TagRenderingConfig { if (m.ifnot !== undefined) { msgs.push( "Unselecting this answer will add " + - m.ifnot.asHumanString(true, false, {}) + m.ifnot.asHumanString(true, false, {}) ) } return msgs.join(". ") @@ -925,7 +923,7 @@ export default class TagRenderingConfig { if (this.labels?.length > 0) { labels = [ "This tagrendering has labels ", - ...this.labels.map((label) => "`" + label + "`"), + ...this.labels.map((label) => "`" + label + "`") ].join("\n") } @@ -938,7 +936,7 @@ export default class TagRenderingConfig { freeform, mappings, condition, - labels, + labels ].join("\n") } @@ -964,11 +962,37 @@ export default class TagRenderingConfig { return Utils.NoNull(tags) } + /** + * Returns the freeform value that should be initially shown in the question + * @param properties + */ + public initialFreeformValue(properties: Record): string { + const value = properties[this.freeform.key] + if (!value) { + return "" + } + const distinguish = this.freeform.postfixDistinguished + if (!distinguish) { + return value + } + const parts = value.split(";") + for (const part of parts) { + if (part.indexOf("/") < 0) { + continue + } + const [v, denom] = part.split("/").map(s => s.trim()) + if (denom === distinguish) { + return v + } + } + return "" + } + /** * The keys that should be erased if one has to revert to 'unknown'. * Might give undefined if setting to unknown is not possible */ - public removeToSetUnknown( + private removeToSetUnknown( partOfLayer: LayerConfig, currentTags: Record ): string[] | undefined { @@ -1012,6 +1036,23 @@ export default class TagRenderingConfig { return Array.from(toDelete) } + + /** + * Gives all the tags that should be applied to "reset" the freeform key to an "unknown" state + */ + public markUnknown(layer: LayerConfig, currentProperties: Record): UploadableTag[] { + if (this.freeform?.postfixDistinguished) { + const allValues = currentProperties[this.freeform.key].split(";").filter( + part => part.split("/")[1]?.trim() !== this.freeform.postfixDistinguished + ) + return [new Tag(this.freeform.key, allValues.join(";"))] + } + + const keys = this.removeToSetUnknown(layer, currentProperties) + + + return keys?.map(k => new Tag(k, "")) + } } export class TagRenderingConfigUtils { @@ -1050,7 +1091,7 @@ export class TagRenderingConfigUtils { clone.mappings?.map((m) => { const mapping = { ...m, - priorityIf: m.priorityIf ?? TagUtils.Tag("id~*"), + priorityIf: m.priorityIf ?? TagUtils.Tag("id~*") } if (m.if.usedKeys().indexOf("nobrand") < 0) { // Erase 'nobrand=yes', unless this option explicitly sets it diff --git a/src/Models/Unit.ts b/src/Models/Unit.ts index 3d1435c3f0..4bdb7117be 100644 --- a/src/Models/Unit.ts +++ b/src/Models/Unit.ts @@ -2,7 +2,6 @@ import BaseUIElement from "../UI/BaseUIElement" import { Denomination } from "./Denomination" import UnitConfigJson from "./ThemeConfig/Json/UnitConfigJson" import unit from "../../assets/layers/unit/unit.json" -import { QuestionableTagRenderingConfigJson } from "./ThemeConfig/Json/QuestionableTagRenderingConfigJson" import TagRenderingConfig from "./ThemeConfig/TagRenderingConfig" import Validators, { ValidatorType } from "../UI/InputElement/Validators" import { Validator } from "../UI/InputElement/Validator" @@ -14,7 +13,6 @@ export class Unit { public readonly denominationsSorted: Denomination[] public readonly eraseInvalid: boolean public readonly quantity: string - private readonly _validator: Validator public readonly inverted: boolean constructor( @@ -26,7 +24,6 @@ export class Unit { inverted: boolean = false ) { this.quantity = quantity - this._validator = validator if ( !inverted && ["string", "text", "key", "icon", "translation", "fediverse", "id"].indexOf( @@ -97,7 +94,7 @@ export class Unit { tagRenderings: TagRenderingConfig[], ctx: string ): Unit[] { - let types: Record = {} + const types: Record = {} for (const tagRendering of tagRenderings) { if (tagRendering.freeform?.type) { types[tagRendering.freeform.key] = tagRendering.freeform.type @@ -185,7 +182,7 @@ export class Unit { ): Unit[] { const appliesTo = json.appliesToKey for (let i = 0; i < (appliesTo ?? []).length; i++) { - let key = appliesTo[i] + const key = appliesTo[i] if (key.trim() !== key) { throw `${ctx}.appliesToKey[${i}] is invalid: it starts or ends with whitespace` } @@ -265,7 +262,7 @@ export class Unit { const loaded = this.getFromLibrary(toLoad.quantity, ctx) const quantity = toLoad.quantity - function fetchDenom(d: string): Denomination { + const fetchDenom = (d: string): Denomination => { const found = loaded.denominations.find( (denom) => denom.canonical.toLowerCase() === d ) diff --git a/src/UI/Base/Hotkeys.ts b/src/UI/Base/Hotkeys.ts index a7fe3bf822..fb714230f9 100644 --- a/src/UI/Base/Hotkeys.ts +++ b/src/UI/Base/Hotkeys.ts @@ -14,6 +14,8 @@ export default class Hotkeys { }[] > = new UIEventSource([]) + private static readonly seenKeys: Set = new Set() + /** * Register a hotkey * @param key @@ -51,6 +53,9 @@ export default class Hotkeys { } } + const keyString = JSON.stringify(key) + this.seenKeys.add(keyString) + this._docs.data.push({ key, documentation, alsoTriggeredBy }) this._docs.ping() if (Utils.runningFromConsole) { diff --git a/src/UI/Base/Popup.svelte b/src/UI/Base/Popup.svelte index 49160d631b..0596746071 100644 --- a/src/UI/Base/Popup.svelte +++ b/src/UI/Base/Popup.svelte @@ -56,6 +56,7 @@ {/if} + {#if $$slots.footer} diff --git a/src/UI/Image/AttributedImage.svelte b/src/UI/Image/AttributedImage.svelte index a4b4d2d8c5..a60d9d64f0 100644 --- a/src/UI/Image/AttributedImage.svelte +++ b/src/UI/Image/AttributedImage.svelte @@ -11,7 +11,6 @@ import ImageOperations from "./ImageOperations.svelte" import Popup from "../Base/Popup.svelte" import { onDestroy } from "svelte" - import type { SpecialVisualizationState } from "../SpecialVisualization" import type { Feature, Point } from "geojson" import Loading from "../Base/Loading.svelte" import Translations from "../i18n/Translations" @@ -19,8 +18,9 @@ import DotMenu from "../Base/DotMenu.svelte" import LoadingPlaceholder from "../Base/LoadingPlaceholder.svelte" import { MenuState } from "../../Models/MenuState" + import ThemeViewState from "../../Models/ThemeViewState" - export let image: Partial + export let image: Partial & { id: string; url: string } let fallbackImage: string = undefined if (image.provider === Mapillary.singleton) { fallbackImage = "./assets/svg/blocked.svg" @@ -28,7 +28,7 @@ let imgEl: HTMLImageElement export let imgClass: string = undefined - export let state: SpecialVisualizationState = undefined + export let state: ThemeViewState = undefined export let attributionFormat: "minimal" | "medium" | "large" = "medium" let previewedImage: UIEventSource> = MenuState.previewedImage export let canZoom = previewedImage !== undefined @@ -36,9 +36,7 @@ let showBigPreview = new UIEventSource(false) onDestroy( showBigPreview.addCallbackAndRun((shown) => { - if (!shown) { - previewedImage?.set(undefined) - } + state.guistate.setPreviewedImage(shown ? image : undefined) }) ) if (previewedImage) { diff --git a/src/UI/Image/LinkableImage.svelte b/src/UI/Image/LinkableImage.svelte index e3a74ac3cf..c1527215f7 100644 --- a/src/UI/Image/LinkableImage.svelte +++ b/src/UI/Image/LinkableImage.svelte @@ -89,17 +89,6 @@ imgClass="max-h-64 w-auto sm:h-32 md:h-64" attributionFormat="minimal" > - {#if linkable} diff --git a/src/UI/Image/NearbyImagesCollapsed.svelte b/src/UI/Image/NearbyImagesCollapsed.svelte index a8e95b0bbc..0d5d1a6683 100644 --- a/src/UI/Image/NearbyImagesCollapsed.svelte +++ b/src/UI/Image/NearbyImagesCollapsed.svelte @@ -1,22 +1,19 @@ {#if enableLogin.data} @@ -37,10 +44,9 @@ > - - - - + + + shown?.set(false)} /> {/if} diff --git a/src/UI/Map/MapLibreAdaptor.ts b/src/UI/Map/MapLibreAdaptor.ts index da43a4b0dd..9973beaaf6 100644 --- a/src/UI/Map/MapLibreAdaptor.ts +++ b/src/UI/Map/MapLibreAdaptor.ts @@ -24,13 +24,13 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { "dragRotate", "dragPan", "keyboard", - "touchZoomRotate", + "touchZoomRotate" ] private static maplibre_zoom_handlers = [ "scrollZoom", "boxZoom", "doubleClickZoom", - "touchZoomRotate", + "touchZoomRotate" ] readonly location: UIEventSource<{ lon: number; lat: number }> private readonly isFlying = new UIEventSource(false) @@ -44,14 +44,14 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { readonly lastClickLocation: Store< | undefined | { - lon: number - lat: number - mode: "left" | "right" | "middle" - /** - * The nearest feature from a MapComplete layer - */ - nearestFeature?: Feature - } + lon: number + lat: number + mode: "left" | "right" | "middle" + /** + * The nearest feature from a MapComplete layer + */ + nearestFeature?: Feature + } > readonly minzoom: UIEventSource readonly maxzoom: UIEventSource @@ -141,7 +141,7 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { const features = map .queryRenderedFeatures([ [point.x - buffer, point.y - buffer], - [point.x + buffer, point.y + buffer], + [point.x + buffer, point.y + buffer] ]) .filter((f) => f.source.startsWith("mapcomplete_")) if (features.length === 1) { @@ -281,9 +281,9 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { return { map: mlmap, ui: new SvelteUIElement(MaplibreMap, { - map: mlmap, + map: mlmap }), - mapproperties: new MapLibreAdaptor(mlmap), + mapproperties: new MapLibreAdaptor(mlmap) } } @@ -347,7 +347,7 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { ) { const event = { date: new Date(), - key: key, + key: key } for (let i = 0; i < this._onKeyNavigation.length; i++) { @@ -536,7 +536,7 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { const bounds = map.getBounds() const bbox = new BBox([ [bounds.getEast(), bounds.getNorth()], - [bounds.getWest(), bounds.getSouth()], + [bounds.getWest(), bounds.getSouth()] ]) if (this.bounds.data === undefined || !isSetup) { this.bounds.setData(bbox) @@ -611,8 +611,11 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { if (!map) { return } + console.log("Bounds are", bbox?.asGeometry()) if (bbox) { - map?.setMaxBounds(bbox.toLngLat()) + if (GeoOperations.surfaceAreaInSqMeters(bbox.asGeojsonCached()) > 1) { + map?.setMaxBounds(bbox.toLngLat()) + } } else { map?.setMaxBounds(null) } @@ -730,14 +733,14 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { type: "raster-dem", url: "https://api.maptiler.com/tiles/terrain-rgb/tiles.json?key=" + - Constants.maptilerApiKey, + Constants.maptilerApiKey }) try { while (!map?.isStyleLoaded()) { await Utils.waitFor(250) } map.setTerrain({ - source: id, + source: id }) } catch (e) { console.error(e) @@ -762,7 +765,7 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { if (this.scaleControl === undefined) { this.scaleControl = new ScaleControl({ maxWidth: 100, - unit: "metric", + unit: "metric" }) } if (!map.hasControl(this.scaleControl)) { @@ -775,7 +778,7 @@ export class MapLibreAdaptor implements MapProperties, ExportableMap { window.requestAnimationFrame(() => { this._maplibreMap.data?.flyTo({ zoom, - center: [lon, lat], + center: [lon, lat] }) }) } diff --git a/src/UI/Map/ShowDataLayer.ts b/src/UI/Map/ShowDataLayer.ts index 710891da7a..588007c3ac 100644 --- a/src/UI/Map/ShowDataLayer.ts +++ b/src/UI/Map/ShowDataLayer.ts @@ -590,7 +590,11 @@ export default class ShowDataLayer { } const bbox = BBox.bboxAroundAll(features.map(BBox.get)) window.requestAnimationFrame(() => { - map.resize() + try { + map.resize() + } catch (e) { + console.error("Could not resize the map in preparation of zoomToCurrentFeatures; the error is:", e) + } map.fitBounds(bbox.toLngLat(), { padding: { top: 10, bottom: 10, left: 10, right: 10 }, animate: false, diff --git a/src/UI/Popup/TagHint.svelte b/src/UI/Popup/TagHint.svelte index 42e7ba7330..49549507c4 100644 --- a/src/UI/Popup/TagHint.svelte +++ b/src/UI/Popup/TagHint.svelte @@ -3,29 +3,30 @@ import FromHtml from "../Base/FromHtml.svelte" import { Translation } from "../i18n/Translation" import Tr from "../Base/Tr.svelte" - import type { SpecialVisualizationState } from "../SpecialVisualization" import Translations from "../i18n/Translations" /** * A 'TagHint' will show the given tags in a human readable form. * Depending on the options, it'll link through to the wiki or might be completely hidden */ - export let tags: TagsFilter - export let currentProperties: Record = {} - /** - * If given, this function will be called to embed the given tags hint into this translation - */ - export let embedIn: ((string: string) => Translation) | undefined = undefined - let tagsExplanation = "" - $: tagsExplanation = tags?.asHumanString(true, false, currentProperties) + export let tags: TagsFilter[] + export let currentProperties: Record = {} -
- {#if tags === undefined} - - {:else if embedIn === undefined} - - {:else} - - {/if} -
+{#if tags?.length > 0} + {#each tags as tag} +
+ {#if tag["value"] === ""} + + {tag["key"]} + + {:else} + + {/if} +
+ {/each} +{:else} + + + +{/if} diff --git a/src/UI/Popup/TagRendering/TagRenderingQuestion.svelte b/src/UI/Popup/TagRendering/TagRenderingQuestion.svelte index 4cebecb6f8..f78dc69162 100644 --- a/src/UI/Popup/TagRendering/TagRenderingQuestion.svelte +++ b/src/UI/Popup/TagRendering/TagRenderingQuestion.svelte @@ -1,5 +1,5 @@