As a general usability rule, checking a box should always have a positive meaning.
Instead of "Do not display birthday publicly", the checking should mean "Display birthday publicly" maybe describing the formfield "Uncheck to hide this field on your profile."

Another approach would be the radio solution I proposed here: http://drupal.org/node/156082
Keep the positive case the first option.

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed

Thanks, I agree with this approach. I've committed a fix that will be in the 1.2 version that reverses the meaning of the checkboxes with the text "Display [field] publicly".

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.