Closed (duplicate)
Project:
Location
Version:
6.x-3.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Jan 2007 at 20:26 UTC
Updated:
20 Jul 2007 at 20:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
geodaniel commentedI'd add a +1 for this too... it looks a little strange having the boxes nested like that when there's only one location
Comment #2
csc4 commentedIt would also be really nice if it only added the plural when needed -
ie Location
rather than Location(s)
Comment #3
geodaniel commentedThere's a patch here that changes Location(s) to Location or Location depending on the number of location forms available - it doesn't flatten the boxes for a single location though.
Comment #4
csc4 commentedYou're a marvel! Thank you so much!
Any idea if/when these will make it into core as they're really useful and it would be great if people could download the whole patched module.
Comment #5
geodaniel commentedMy solution was only a partial one, so I'm not sure it's ready to be applied to the module yet. As for when... Ankur's a busy guy, and it depends on his schedule. We can help him out by making sure that patches are pretty much ready to go in though, to make it easier to review.
Comment #6
geodaniel commentedInitial patch committed to HEAD and DRUPAL-5 branch - will backport to 4.7 too. I still haven't tackled the removal of one of the boxes when there is only one location though.
Comment #7
geodaniel commentedInitial patch committed to DRUPAL-4-7 branch too. Leaving this ticket open for the other half of it, to remove the duplicate boxes when only adding one location.
Comment #8
ankur commentedSetting to active so status it doesn't get listed as 'patch' in queue list
Comment #9
jo1ene commented+1 for fixing boxes
Where's the "add another location" in the edit form?
There is also an issue with alignment. It just looks terrible.
Comment #10
geodaniel commentedMarked http://drupal.org/node/131485 as dup of this
Comment #11
colanI'm marking this as a duplicate of http://drupal.org/node/160443 because that one's got a patch.