Modify

Opened 6 years ago

Last modified 3 years ago

#17324 new defect

Adding bridge preset to highways not working

Reported by: anonymous Owned by: team
Priority: normal Milestone:
Component: Core Version: latest
Keywords: Cc:

Description

Steps to reproduce:

  1. Select a highway (eg. residential)
  2. Search for bridge in presets
  3. Apply bridge preset to the selected way
  4. Nothing happens

Attachments (0)

Change History (9)

comment:1 by anonymous, 6 years ago

Although it's working if you add the bridge preset to an untagged way.

comment:2 by GerdP, 6 years ago

With an untagged way the presets defaults to bridge=yes while with a tagged way you have to select what kind of bridge it is.
No idea if that is intended.

in reply to:  2 comment:3 by anonymous, 6 years ago

Replying to GerdP:

No idea if that is intended.

Yeah, maybe intended but it's confusing.

comment:4 by anonymous, 6 years ago

Replying to GerdP:

[...] with a tagged way you have to select what kind of bridge it is.

I see two possible solutions:

  1. solution: Gray out the apply button until something is changed in the preset editor (or whatever it called)
  2. solution: Use the same defaults (bridge=yes and layer=1) as with untagged ways or at least bridge=yes.

comment:5 by GerdP, 6 years ago

I've now learned that the behaviour depends on a preference setting:
taggingpreset.fill-default-for-tagged-primitives
This is false by default. In Expertmode you can change it to true to get what you want.

comment:6 by Klumbumbus, 6 years ago

The current behavior (ignoring defaults) is intended as users maybe do not expect that opening a preset from the Tags/Membership panel, changing nothing themself and closing with the apply preset button changes the object by adding tags. I can't find the ticket where Dirk explained that to me when I was confused about this ;)

in reply to:  6 comment:7 by anonymous, 6 years ago

Replying to GerdP:

I've now learned that the behaviour depends on a preference setting:
taggingpreset.fill-default-for-tagged-primitives
This is false by default. In Expertmode you can change it to true to get what you want.

Thanks, it's working. But I don't want to be selfish. It would be great if all JOSM users would know about this flag (or fixed the issue :)

Replying to Klumbumbus:

The current behavior (ignoring defaults) is intended as users maybe do not expect that opening a preset from the Tags/Membership panel, changing nothing themself and closing with the apply preset button changes the object by adding tags. I can't find the ticket where Dirk explained that to me when I was confused about this ;)

Oh okay, this way it makes sense. Please, not forget that the presets can bring forth by searching (default shortcut is F3). This way it's not acceptable, that apply button does nothing. Also, I think by searching most people would like to add new tags, not modifying them.

The scenario you described above can be easily solved by:

  1. Greying out the apply button until something is changed in the preset editor AND/OR
  2. Checking where that preset editor activity called from. If from Tags/Membership panel -> disable defaults, else no change (enable).

in reply to:  6 ; comment:8 by skyper, 3 years ago

Replying to Klumbumbus:

The current behavior (ignoring defaults) is intended as users maybe do not expect that opening a preset from the Tags/Membership panel, changing nothing themself and closing with the apply preset button changes the object by adding tags. I can't find the ticket where Dirk explained that to me when I was confused about this ;)

#12733

in reply to:  8 comment:9 by gaben, 3 years ago

Replying to skyper:

#12733

Thanks! The anonymous reporter was me :) At the time I haven't registered yet.

This option should have a checkbox somewhere in the settings.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to anonymous.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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