OpenSeaMap / josm

Presets, styles & icons for our various JOSM tools.
11 stars 6 forks source link

Update INT-1-preset.xml #6

Closed XPcRUSHER closed 10 years ago

XPcRUSHER commented 10 years ago

Regulation Notices, Restriction Notices German translated. 3 CHUNKS created. Please check and release.

Thanks More then coming soon.

PS: Edit #5 don't realase, it have errors

malcolmh commented 10 years ago

Stefan,

I have merged your changes and also made some of my own:

  1. I removed all english “short_description” parameters. I deprecate the usage of these as they double the length of menus, but still do allow enough words to be useful. Trying to embed documentation into menus is very inefficient. My preferred method is to always have a link the the appropriate Wiki page on every preset panel and to use “display_value” to de-obfuscate tag values where appropriate.
  2. Your usage of Notice Orientation was incorrect. The orientation of the notice is the compass orientation of the entire notice assembly - the bearing ‘to' the notice from the observer: e.g. orientation = 0 degrees is a notice that can be read when looking north.

Please remember to pull my update before making further changes.

Regards, Malcolm=

XPcRUSHER commented 10 years ago

OK Malcolm, you have a error in line 224

XPcRUSHER commented 10 years ago

I don't find the error on line 224 >_< JOSM said "error on list_entry, it must end with > or />"

Skippern commented 10 years ago

Look at this:

<list_entry value="upstream" display_value="Upstream"de.display_value="Strom aufwärts" de.short_description="”/>

I think the error is due to the lack of space between display_value and de.display_value.

Also, all empty de.short_description=“” can be removed.

malcolmh commented 10 years ago

On 9 Oct 2014, at 17:18, Aun Johnsen notifications@github.com wrote:

Look at this:

<list_entry value="upstream" display_value="Upstream"de.display_value="Strom aufwärts" de.short_description="”/>

I think the error is due to the lack of space between display_value and de.display_value.

Also, all empty de.short_description=“” can be removed. — Reply to this email directly or view it on GitHub.

Fixed - sorry about that!=

XPcRUSHER commented 10 years ago

I have see it, i can don't edit this. I faild to edit the file on time. Malcolm have fixed the error. Thanks.

I will remember your instruction.

Regards, Stefan