Opened 7 years ago

Closed 6 years ago

#1584 closed improve feature (fixed)

gender options

Reported by: bikepunk Owned by: matt
Priority: major Milestone: 1.6
Component: BW Profile Keywords:
Cc:

Description

from : http://www.bewelcome.org/forums/s1959-What_about_gender_/

Since "gender" is a box that allows users to search for hosts/guests based on their gender, I feel that we should offer the following :

  1. have three options (one - and only one - should be compulsory) : Male - Female - Other

AND

  1. provide a free text field with the "other" so that members can write about their particular situation. Since this is a free text field, you can write "i don't tell" or "i'm a transsexual" or "I don't believe in gender-discrimination" or whatever you want to say about your gender views.

This way members can still use the search function and those members who don't want to put their gender have a way of expressing that.


At the moment, there is no way to display gender informations on profiles. Of course, we have to make it possible to display. I guess next to the "age" info would be fine.

Change History (40)

comment:1 Changed 7 years ago by crumbking

We should add this to the signup process, too!

Last edited 7 years ago by crumbking (previous) (diff)

comment:2 Changed 6 years ago by TimLoal

  • Component changed from unknown to BW Profile

comment:3 Changed 6 years ago by jsfan

  • Milestone Future deleted

Milestone Future deleted

comment:4 Changed 6 years ago by guaka

  • Milestone set to unassigned

Suggestion:

Rox: Male Female Other Welen: custom field for other

comment:5 Changed 6 years ago by jsfan

This is actually a rather controversial topic and should be discussed further by the community before anything is implemented.

Last edited 6 years ago by jsfan (previous) (diff)

comment:6 Changed 6 years ago by pablobd

I think bikepunk proposal adreses all the points of view and gives more freedom for members without losing any current functionality of the site.

comment:7 Changed 6 years ago by matt

  • Owner set to matt
  • Status changed from new to assigned

comment:8 Changed 6 years ago by James_Oder_Dave

I think I agree with Jsfan. Not only is it a controversial topic but I would expect that the way that most users will use the third option may actually limit or exclude a sizable chunk of members from hosting.

I don't want to copy and paste here, but see my post on the practical implications:

James_Oder_Dave - posted 2012/08/19 03:13

on this thread.

http://www.bewelcome.org/forums/s1959-What_about_gender_

comment:9 Changed 6 years ago by shevek

  • Milestone changed from unassigned to 1.6

comment:10 Changed 6 years ago by crumbking

Why do we assign a controversial topic to a milestone? Better: discuss->than implement. Could we learn from the profile visitor feature? ;-)

comment:11 Changed 6 years ago by shevek

@crumbking: The options are already there for editmyprofile. To add these to signup is just common sense as the reason this ticket exists is that this is missing.

We obviously have different streams where things are discussed but this popped up in Hong Kong and on the forum so I guess we don't do the same as with the profile visitor feature.

comment:12 Changed 6 years ago by dima42

matt points out that his commit https://gitorious.org/~mattoz/bewelcome/mattozs-rox/commit/fc1cde14ff5287bc0ad9e123bd3b2428cd05a4f3 is actually meant for this ticket, not 1959. my bad.

it looks as though there might be some technical problems with it; i'm in correspondence with matt by email.

that said, I am +1'ing postponing solely based on the fact that three people posted on this ticket asking to not take action, including two within the last two weeks. i rather disagree with their stance - i personally think that the proposed changes would be better than status quo - but i think that going forward right now would be quite undemocratic.

comment:13 Changed 6 years ago by crumbking

Just wanna mention than I'm +1 for the integration of this gender tickets. I checked James arguments in the forum thread and I can't see an issue of loosing quality (site search) of information by enabling this third option. All the search based arguments are a matter of implementation and UI design (even with free text fields)

What I wanna say is as much as I would like to see this happen I see more critical issues which should be addressed.

comment:14 Changed 6 years ago by shevek

@crumbking: I agree that there are more pressing issues to be addressed but this is an esay ticket for a newbie (sorry matt :)) and if we don't include some of these we will never get people to learn what they need to do to fix the more important issues.

comment:15 Changed 6 years ago by dima42

I am withdrawing my postpone +1 per last two comments. If someone wants to work on a ticket based on their internal motivation, and the change is a favorable one, we should encourage this. Arguing over priorities only makes sense if other people have to spend time on the ticket. Given that Matt will try to code this and I'll be happy to look it over, there won't be unwilling resources consumed.

That said, I'm guessing we all agree it would be a waste of time to hold up a release over this, so if the other 1.6 tickets get done much sooner, we might postpone for that reason.

comment:17 Changed 6 years ago by matt

  • Status changed from assigned to local_testing

comment:18 Changed 6 years ago by midsch

So on my profile I've got male/female/other plus "hidden" option; I'm able to choose and save it. But besides my profile settings t's not shown anywhere. (Much discussion for an invisible setting.)

On search I can only search for male/female/all. It works, but you can't search for "other" only.

comment:19 Changed 6 years ago by shevek

This isn't yet on develop. To test this you would need to checkout matt's clone. I'll post again as soon as stuff is on develop.

comment:20 Changed 6 years ago by dima42

the search worked fine for me the first time (i searched without entering a location). however, i then refreshed the page and tried searching again. the result was "address not found"

at templates/filters.php:59 we should use GenderOther? instead of otherGender (it was mentioned on the mailing list and also it's used in shevek's commit (https://www.gitorious.org/bewelcome/rox/commit/bb7d247c46ba244804b0e019c0827d4b714275df)

comment:21 Changed 6 years ago by shevek

After a code review I'd say that's due the changes to the javascript in filter.php in line 25 in build/searchmembers/templates/teaser.php this needs to be reverted to the original as it is totally unrelated to the gender options.

Lines 168-176 in build/searchmembers/templates/filters.php should be removed as well as they are connected to the above change.

In build/members/templates/member.teasercontent.php line 46 needs to echo a translation for the gender as well.

Due to the inconsistency of gender types if gender is set to 'IDontTell' it needs to echo 'Gender: hidden' (of course translated).

Search currently reveals all members that have 'IDontTell' set if they didn't set gender to be hidden. In build/searchmembers/searchmembers.model.php line 784 should read

return "Gender = 'other' AND HideGender='No'";

(the type 'other' was only introduced with the commit for 1959 so couldn't be used here up to now).

Last edited 6 years ago by shevek (previous) (diff)

comment:22 Changed 6 years ago by matt

fixed the issues and merged into develop

comment:23 Changed 6 years ago by shevek

Looks better now. But the gender is still not shown translated and IDontTell isn't treated as hidden.

Last edited 6 years ago by shevek (previous) (diff)

comment:24 Changed 6 years ago by crumbking

Do I miss something or shouldn't be there a free text field while hitting other?

comment:25 Changed 6 years ago by shevek

@crumbking: The discussion in the forum turned to no free text field. if the member feels like sharing why other decided to choose 'other' we have the profile summary field for this.

comment:26 Changed 6 years ago by toub

Tested OK locally

comment:27 Changed 6 years ago by shevek

  • Status changed from local_testing to to_alpha

comment:28 Changed 6 years ago by shevek

  • Status changed from to_alpha to testing

comment:29 follow-up: Changed 6 years ago by crumbking

Should we really show gender: hidden? As we discussed this with the age field I would say this gives no value for a searching member or profile visitor. Except that you know that it's hidden. Other opinions?

Beside this small issue. Seems to work

comment:30 in reply to: ↑ 29 Changed 6 years ago by bikepunk

Replying to crumbking:

Should we really show gender: hidden? As we discussed this with the age field I would say this gives no value for a searching member or profile visitor. Except that you know that it's hidden. Other opinions?

Beside this small issue. Seems to work

I agree with crumbking, displaying "gender: hidden" makes no sense to me.

comment:31 Changed 6 years ago by midsch

Indeed, "gender=hidden" is no useful information. Maybe so it's easier for users to understand, why members don't show up in searches with a set gender. But I also think it should not be displayed.

-> if this can be fixed easily go for it

-> if not it still can make it into the next release to make #1959 more useful.

Last edited 6 years ago by midsch (previous) (diff)

comment:32 Changed 6 years ago by sitatara

Just created a test profile with gender = other. My testprofile now shows gender: male

comment:33 Changed 6 years ago by shevek

Gender will only be shown if it isn't hidden: https://www.gitorious.org/bewelcome/rox/commit/79a441c4946281e8539df00333122d39694e9d4c

Additionally added 'Gender' to the 'Age' column on '/searchmembers/mapoff'.

comment:34 Changed 6 years ago by shevek

Deployed on alpha. Please test.

comment:35 Changed 6 years ago by crumbking

There is one </br> to much on the map search while gender is hidden.

Could we add such a layoutbit for the age, too? (bit off topic I know)

comment:36 Changed 6 years ago by shevek

Removed the extra <br/> and the 'Gender: '.

Info now reads: '64 years old, male, Herat, Afghanistan'. In case gender is hidden: '42 years old, Paris, United States'.

Commit: https://www.gitorious.org/bewelcome/rox/commit/3538407b4bb0523cb27d7f696887d0615044faca

Last edited 6 years ago by shevek (previous) (diff)

comment:37 Changed 6 years ago by crumbking

  • Status changed from testing to needs_work

okay works locally. lets move this to alpha and lets do the age stuff in another ticket.

comment:38 Changed 6 years ago by crumbking

  • Status changed from needs_work to to_alpha

comment:39 Changed 6 years ago by shevek

  • Status changed from to_alpha to testing

Deployed to alpha.

comment:40 Changed 6 years ago by sitatara

  • Resolution set to fixed
  • Status changed from testing to closed

Hiding works and shows no extra lines on profile and in search results any more. (Agree that this would be good for hidden age too). I also tested the signup again and it worked for me now (but that's another ticket anyway.)

Note: See TracTickets for help on using tickets.