Modify

Opened 8 years ago

Closed 8 years ago

Last modified 6 years ago

#13144 closed enhancement (fixed)

Include sex/gender in hairdresser/barber preset

Reported by: naoliv Owned by: team
Priority: normal Milestone: 16.07
Component: Internal preset Version:
Keywords: Cc:

Description

While creating a hairdresser/barber using the preset we only have Name, Opening Hours and Wheelchairs

Is it possible to also include the sex/gender like https://wiki.openstreetmap.org/wiki/Tag:shop%3Dhairdresser#Tags_to_use_in_combination says, please?

Options should be "empty", "female", "male", "unisex", in a drop-down box (ie, the user will be able to select only one option and, except for the "empty" one, the selected value will be yes)

Attachments (0)

Change History (6)

comment:1 by Don-vip, 8 years ago

Milestone: 16.07

in reply to:  description comment:2 by Klumbumbus, 8 years ago

Replying to naoliv:

Options should be "empty", "female", "male", "unisex", in a drop-down box (ie, the user will be able to select only one option and, except for the "empty" one, the selected value will be yes)

This is confusing for the user. In a combobox you expect only one key with different values and not different keys. What should the combobox display if both male=yes and female=yes are already tagged and you open the preset?

I think the better solution would be check boxes and validation rules.

comment:3 by Klumbumbus, 8 years ago

patch for the validator rules:

/* #13144 */
*[unisex=yes][female=yes][male!=yes],
*[unisex=yes][male=yes][female!=yes] {
  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
}
*[unisex=yes][female=yes][male=yes] {
  throwWarning: tr("{0} together with {1} and {2}. Remove {1} and {2}", "{0.tag}", "{1.tag}", "{2.tag}");
  fixRemove: "female";
  fixRemove: "male";
}
*[female=yes][male=yes][!unisex][shop=hairdresser] { /* for toilets male=yes female=yes unisex=no combination is valid */
  throwWarning: tr("{0} together with {1}", "{0.tag}", "{1.tag}");
  suggestAlternative: "unisex=yes";
  fixRemove: "female";
  fixRemove: "male";
  fixAdd: "unisex=yes";
}

Has i18n impact. Waiting a bit in case a hotfix for milestone 16.06 is required.

comment:4 by naoliv, 8 years ago

I only thought about the case where the user creates a hairdresser/barber using the preset (and I forgot that he can modify it using the preset too).
So indeed, a combobox won't properly fit this.

comment:5 by Klumbumbus, 8 years ago

Resolution: fixed
Status: newclosed

In 10537/josm:

fix #13144 - Include female, male and unisex in hairdresser/barber preset and add related validation

comment:6 by Klumbumbus, 6 years ago

In 13102/josm:

see #13144, fix #15536 - limit male/female/unisex test to hairdressers

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
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.