Modify

Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#16301 closed enhancement (fixed)

Add filter and icon for map types

Reported by: stoecker Owned by: Don-vip
Priority: normal Milestone: 19.05
Component: Core imagery Version:
Keywords: Cc:

Description (last modified by Klumbumbus)

See #16103

The list of imagery in prefs should allow to filter for the types.

If possible the different types should be displayed with a small icon.

Category is already added to all entries in the wiki.

Attachments (0)

Change History (81)

comment:1 by Klumbumbus, 6 years ago

And a filter by text please too, (similar to the filterbox for projections), which allows to e.g. filter entries by country (filtertext "de") or city (filtertext "berlin").

comment:2 by Klumbumbus, 6 years ago

Ticket #16136 has been marked as a duplicate of this ticket.

comment:3 by Don-vip, 6 years ago

Why 18.08?

comment:4 by stoecker, 6 years ago

June is too early (first data needs to be added to Maps) and in July I will have no time...

comment:5 by Klumbumbus, 6 years ago

The wiki doesn't yet allow the category attribute.
Warnung: Ungültige Wiki-Seite: XML validation for map failed: Element '{http://josm.openstreetmap.de/maps-1.0}entry', attribute 'category': The attribute 'category' is not allowed.

in reply to:  5 comment:6 by stoecker, 6 years ago

Replying to Klumbumbus:

The wiki doesn't yet allow the category attribute.
Warnung: Ungültige Wiki-Seite: XML validation for map failed: Element '{http://josm.openstreetmap.de/maps-1.0}entry', attribute 'category': The attribute 'category' is not allowed.

Tried
<category>xxx</category>
and it worked fine. Where exactly did you try?

comment:7 by Klumbumbus, 6 years ago

I tried <entry category="photo"> cause it is listed under attributes at wiki:Maps

comment:9 by Klumbumbus, 6 years ago

Description: modified (diff)

comment:10 by Don-vip, 6 years ago

Description: modified (diff)

comment:11 by Don-vip, 6 years ago

Description: modified (diff)

comment:12 by Klumbumbus, 6 years ago

Description: modified (diff)

comment:13 by Klumbumbus, 6 years ago

Description: modified (diff)

comment:14 by Klumbumbus, 6 years ago

Description: modified (diff)

comment:15 by Don-vip, 6 years ago

Milestone: 18.0818.09

comment:16 by Don-vip, 5 years ago

Milestone: 18.0918.10

comment:17 by Don-vip, 5 years ago

Milestone: 18.1018.11

comment:18 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:19 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:20 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:21 by Don-vip, 5 years ago

Milestone: 18.1118.12

comment:22 by stoecker, 5 years ago

We should icons now (also to honor Klumbumbus work). Any proposals for icons?

comment:23 by Don-vip, 5 years ago

Ideas. Licenses to check:

  • https://cdn0.iconfinder.com/data/icons/internet-line/512/Internet_Line-16-512.png photo aerial or satellite photo
  • https://cdn2.iconfinder.com/data/icons/essential-web-5/50/map-direction-diagram-path-route-512.png map a map
  • https://cdn1.iconfinder.com/data/icons/airline-flight-booking-mobile-app/15/recent-time-search-reload-time-512.png historicmap historic or otherwise outdated map
  • https://josm.openstreetmap.de/export/14475/josm/trunk/images/layer/osmdata_small.png osmbasedmap map based on OSM data
  • https://cdn1.iconfinder.com/data/icons/circus-element/64/Balloon-hot-transportation-transport-circus-512.png historicphoto historic or otherwise outdated aerial or satellite photo
  • https://cdn0.iconfinder.com/data/icons/glyphpack/34/question-circle-512.png other any other type
Last edited 5 years ago by Don-vip (previous) (diff)

in reply to:  23 ; comment:24 by Klumbumbus, 5 years ago

Replying to Don-vip:

Ideas.

Nice.
That one is not recognizable at 16px though. https://cdn1.iconfinder.com/data/icons/circus-element/64/Balloon-hot-transportation-transport-circus-512.png At least I didn't understand what that is until I opened the full size.

What about using the same icons for photo and historicphoto and for map and historicmap while the normal variant is coloured and the historic variant greyscale?

in reply to:  24 comment:25 by stoecker, 5 years ago

Replying to Klumbumbus:

Replying to Don-vip:

Ideas.

Nice.
That one is not recognizable at 16px though. https://cdn1.iconfinder.com/data/icons/circus-element/64/Balloon-hot-transportation-transport-circus-512.png At least I didn't understand what that is until I opened the full size.

Same for me. Also the historicmap is not really obvious.

What about using the same icons for photo and historicphoto and for map and historicmap while the normal variant is coloured and the historic variant greyscale?

I like that.

comment:26 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:27 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:28 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:29 by Klumbumbus, 5 years ago

All categories are added. We have:

303x photo
70x historicphoto
63x map
93x historicmap
42x osmbasedmap
154x other

In total 725 active (i.e. not disabled) entries.

comment:30 by Klumbumbus, 5 years ago

Can we please have a check either in SyncEditorLayerIndex.groovy or make it mandatory in the xml checks in the wiki that every entry has an category property?

comment:31 by Don-vip, 5 years ago

154 other? That's a huge number, do you think we could add a new category? What are all these sources about?

in reply to:  31 comment:32 by Klumbumbus, 5 years ago

Replying to Don-vip:

154 other? That's a huge number, do you think we could add a new category? What are all these sources about?

Mostly overlays of one feature (streets, buildings, administrative boundaries,...), but also elevation modells (hill shading), contour lines, OSM validation overlays, WMTS/wms_endpoint sources containing layers of different categories.

comment:33 by Don-vip, 5 years ago

In 14541/josm:

see #16301 - make category mandatory

comment:34 by Klumbumbus, 5 years ago

In 14552/josm:

  • see #16301 - make id mandatory
  • see #17058 - add asserts to territories.mapcss

comment:35 by stoecker, 5 years ago

I'm not so happy to make these attributes mandatory. That may be true for our wiki, but not necessary elsewhere. Making them mandatory prevents loading when the attributes aren't there.

I'd rather test them with the Sync script.

comment:36 by stoecker, 5 years ago

Hmm, seems minOccurs does not do anything at all. You can remove all these arguments without an error.

comment:37 by stoecker, 5 years ago

In 14554/josm:

see #16301 - make sync script stable against missing data

in reply to:  35 ; comment:38 by Klumbumbus, 5 years ago

Replying to stoecker:

I'm not so happy to make these attributes mandatory.

OK, I thought maps.xsd is only for the wiki. Please change it to a better solution.

in reply to:  38 comment:39 by stoecker, 5 years ago

Replying to Klumbumbus:

Replying to stoecker:

I'm not so happy to make these attributes mandatory.

OK, I thought maps.xsd is only for the wiki. Please change it to a better solution.

No. These XSD are the files JOSM uses for validation.

But as said it seems minOccurs doesn't have an effect. The Sync script now screams if these fields are missing :-)

comment:40 by Don-vip, 5 years ago

Milestone: 18.1219.01

comment:41 by Don-vip, 5 years ago

Milestone: 19.0119.02

comment:42 by Don-vip, 5 years ago

Milestone: 19.0219.03

comment:43 by Klumbumbus, 5 years ago

Milestone: 19.0319.04

comment:44 by Don-vip, 5 years ago

Milestone: 19.0419.05

comment:45 by Don-vip, 5 years ago

@Dirk: do you plan to work on this? I didn't touch it because I thought when you created the ticket that it was your intention.

comment:46 by stoecker, 5 years ago

Please go on, when you have time for this.

comment:47 by Don-vip, 5 years ago

@Stefan: I'd like to use https://www.iconfinder.com/icons/1587506/internet_line_satellite_icon for the first icon but the SVG file is 2Mb. I can't reduce it using svg-cleaner, do you know how we can improve it? It shouldn't be more than a few kilobytes.

comment:48 by Don-vip, 5 years ago

Ah, the SVG includes a font. WTF. without it it's only 4Kb :)

comment:49 by Don-vip, 5 years ago

Owner: changed from team to Don-vip
Status: newassigned

comment:50 by Don-vip, 5 years ago

In 15049/josm:

see #16301 - add category icons in imagery preferences

comment:51 by Don-vip, 5 years ago

TODO:

  • add tooltip above icon, displaying name and description of the imagery category. Probably requires custom renderers
  • add tooltip above country, using Java Locale class
  • add filter text field searching on all columns (category, country, title, url)

comment:52 by Don-vip, 5 years ago

@Dirk the icon script is not happy with my dynamic loading:
https://josm.openstreetmap.de/jenkins/job/JOSM-Integration/3078/warnings4Result/

How can we ignore these warnings?

comment:53 by stoecker, 5 years ago

Simply add /* ICON(data/imagery/) */ in front of the names in ImageryInfo.java ;-)

comment:54 by Don-vip, 5 years ago

OK thanks :)

comment:55 by Don-vip, 5 years ago

In 15057/josm:

see #16301 - suppress icon warning

comment:57 by stoecker, 5 years ago

You took ImageryType enum. You should have used ImageryCategory enum. :-)

I saw your commit first and though: Ah, he implemented type image display as well. Fine. But found no related change :-)

Last edited 5 years ago by stoecker (previous) (diff)

comment:58 by Don-vip, 5 years ago

Appfff I was so tired I shouldn't have committed this xD My eyes were probably half-closed already :)

comment:59 by Don-vip, 5 years ago

In 15059/josm:

see #16301 - suppress icon warning (on the correct enum this time)

comment:60 by stoecker, 5 years ago

You should go to your boss and say: "I make silly mistakes in my open source free time coding - I need a vacation". He sure will grant it :-)

in reply to:  60 comment:61 by stoecker, 5 years ago

Replying to stoecker:

You should go to your boss and say: "I make silly mistakes in my open source free time coding - I need a vacation". He sure will grant it :-)

Or maybe not when you use your free time coding for JOSM...

comment:62 by Don-vip, 5 years ago

:D I was on vacation and exhausted because of it, just got back to work today so it does not work ;)

comment:63 by Don-vip, 5 years ago

In 15114/josm:

see #16301 - display tooltips for all columns

comment:64 by Don-vip, 5 years ago

In 15115/josm:

see #16301 - extract ImageryProvidersPanel

comment:65 by Don-vip, 5 years ago

Resolution: fixed
Status: assignedclosed

In 15116/josm:

fix #16301 - add filter for default imagery entries

comment:66 by Klumbumbus, 5 years ago

Thats great, thanks!
One thing: the map on the right side in the preferences displays only error tiles for me. It seems that got broken.

comment:67 by Don-vip, 5 years ago

I didn't touch this part. Probably a network error or something.

comment:68 by Klumbumbus, 5 years ago

the console says Failed loading https://c.tile.openstreetmap.org/3/5/2.png: class java.io.IOException: Server returned HTTP response code: 429 for URL: https://a.tile.openstreetmap.org/3/5/2.png however in browser i can load https://a.tile.openstreetmap.org/3/5/2.png without problems.

comment:69 by stoecker, 5 years ago

429 is "rate limit".

in reply to:  69 comment:70 by Klumbumbus, 5 years ago

Replying to stoecker:

429 is "rate limit".

Seems justified. JOSM is doing strange things. See attached gifs.

comment:71 by stoecker, 5 years ago

Seems justified. JOSM is doing strange things. See attached gifs.

Please open a ticket for that for the TMS image loader (assigned to wiktorn?).

comment:72 by Klumbumbus, 5 years ago

Ahh, I think we need a hotfix:

What steps will reproduce the problem?

  1. type "cy" in the filter box above the list in the imagery preferences
  2. select the cyprus entry on position 5
  3. click activate

What is the expected result?

the cyprus entry is added to the list below

What happens instead?

the entry which is on position 5 in the unfiltered list is added (Digital Globe Standard Imagery)

comment:73 by Don-vip, 5 years ago

Problem of HTTP 429 fixed in #17772.
Also deleting the big gif files, I did some cleanup of giant attachments with #17757 because it complexifies #16857.

I'm fixing the "Cyprus" selection problem plus also the problem reported in #17779.

comment:74 by Don-vip, 5 years ago

In 15153/josm:

see #16301 - fix activation problem when filtering imagery sources

comment:75 by Klumbumbus, 5 years ago

BTW, not only the "Activate" button but also the "Edit" button from the xml bounds plugin is affected. Not sure if the plugin needs an own fix.

in reply to:  75 comment:76 by Don-vip, 5 years ago

Replying to Klumbumbus:

also the "Edit" button from the xml bounds plugin is affected.

Thanks, plugin fixed in [o35023:35024].

comment:77 by Don-vip, 5 years ago

Hotfix released: r15155

comment:78 by Klumbumbus, 5 years ago

Thanks. Works fine for me.

in reply to:  77 comment:79 by Klumbumbus, 5 years ago

Replying to Don-vip:

Hotfix released: r15155

Also nice that you managed to use a fancy number for the hotfix too, similar to the stable release. I'm very pleased. :)

comment:80 by Don-vip, 5 years ago

I wanted 15151 initially. It's a palindrome.

comment:81 by Klumbumbus, 4 years ago

--> follow up in #18172

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.