Tabledrag doesn't hide columns when fieldset is collapsed
Darren Oh - September 2, 2008 - 20:01
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | javascript |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Description
Tested in Firefox 3. Elements in a collapsed fieldset inherit the display: none setting, and tabledrag.js skips elements with display set to none. Skipping does not seem to be necessary.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| tabledrag.js_DRUPAL-6.patch | 744 bytes | Idle | Failed: Failed to apply patch. | View details | Re-test |
#1
Patch failed to apply. More information can be found at http://testing.drupal.org/node/14016. If you need help with creating patches please look at http://drupal.org/patch/create
#2
Bot needs to learn not to run check_plain() on issue titles.
#3
oh, so THAT's why I'm having problems trying to add tabledrag compatibility to location.
Subscribing.
#4
The header wasn't hiding either.
#5
I ran into this with a collapsed fieldset containing a draggable table. Manually applying the patch solves the problem for me at least in Safari4beta and Opera9.6.
#6
Any updates on this? WOuld really need this for Location module
#7
This is ready.
#8
New stuff and bugfixes go first to Drupal 7 to avoid regressions popping up. Needs review on D7.
#9
The last submitted patch failed testing.
#10
Patch for Drupal 7.
#11
The last submitted patch failed testing.
#12
#13
The last submitted patch failed testing.
#14
#15
#16
The last submitted patch failed testing.
#17
Marked #339105: Tabledrag not hiding columns inside collapsed fieldsets as a duplicate of this issue.
#18
This definitely needs work, as there's lots of cruft in tabledrag from getting it to work in older browsers. Please see this comment.
#19
#20
#425148: Form elements within a field group are not visible in Internet Explorer was marked as duplicate for this issue. Is it?
The symptoms mentioned here seems to be different then those mentioned there.
In addition can the above patch be applied for D6 as well or only D7?
thanks.
#21
Use the patch in #4 for Drupal 6.
#22
Keeping up to date with HEAD. Still needs review!