Closed (fixed)
Project:
Webform
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Dec 2010 at 20:58 UTC
Updated:
19 Jan 2011 at 04:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
james.elliott commentedComment #2
quicksketchHmm, unbelievably Drupal 7 seems to handle an opposite situation, where it only adds "- Select -" if the element is NOT required and default NOT is provided. Webform currently adds "select..." if the element IS required and a default is NOT provided. Strangely Drupal 7 does not add "- Select -" in the same scenario that Webform handles. Honestly I'm not really sure which approach makes more sense.
Comment #3
quicksketchI reviewed this one again and I see what's going on here. Core handles both required and not-required fields, it just gives different labels depending on whether a field is required or not. Since Webform already contains the IS required condition, it seemed like core only auto-provided the NOT required condition (since it didn't double-up the required option if not needed). I'm giving this another review.
Also of interest to this issue is #799670: Translatability issue with "select...", which argues that "- Please select -" should be used over "Select..." (noting that Drupal core prefers just "- Select -" with no "Please ").
Comment #4
quicksketchI've committed these patches that make the suggested change to D7 plus removal of unnecessary translation strings. In D6, I've backported the D7 behaviors and appearance, adding the "- None -" option for non-required fields and changed "select..." to "- Select -" for required fields.
Comment #5
quicksketch