On /admin/content/node --
Check the 'Select all' checkbox in the non-floating header, the floating header remains unchecked.
Check the 'Select all' checkbox in the floating header, uncheck it in the non-floating header, the floating header remains checked.
Check the 'Select all' checkbox in either header, uncheck any other checkbox, the non-floating header is unchecked (correctly), the floating header remains checked.

Files: 
CommentFileSizeAuthor
#22 core_js_select_all_sticky-218470-22.patch780 bytesAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
#19 core-js-select-all-sticky-218470-17-D7.patch783 bytesAlbert Volkman
PASSED: [[SimpleTest]]: [MySQL] 38,277 pass(es).
[ View ]
#17 core-js-select-all-sticky-218470-17-D7.patch783 bytesnod_
PASSED: [[SimpleTest]]: [MySQL] 38,281 pass(es).
[ View ]
#17 core-js-select-all-sticky-218470-17.patch803 bytesnod_
PASSED: [[SimpleTest]]: [MySQL] 34,583 pass(es).
[ View ]
#14 tableselect-tableheader.patch919 bytescasey
PASSED: [[SimpleTest]]: [MySQL] 23,291 pass(es).
[ View ]
#5 table-sa2.patch815 bytesredndahead
Passed: 11624 passes, 0 fails, 0 exceptions
[ View ]
#2 table-sa.patch819 bytesredndahead
Failed: Failed to apply patch.
[ View ]

Comments

Version:6.x-dev» 7.x-dev

Confirmed. Same happens in 7.x-dev.

Status:Active» Needs review
StatusFileSize
new819 bytes
Failed: Failed to apply patch.
[ View ]

Here is a patch to include the cloned header on update. One thing that this patch will miss is when the select all box is checked on page load. The issue is the cloned table header is loaded after the update of the select all box on the initial load. Not sure when this will come up.

EDIT: Nevermind on the initial load the js will clone the checked state also.

i wonder if it's related but i'd noticed that the header isn't fully cloned when it start floating. if you set a class on the fixed header once it starts floating the formatting is lost until it stops floating.

Status:Needs review» Needs work

The last submitted patch failed testing.

Status:Needs work» Needs review
StatusFileSize
new815 bytes
Passed: 11624 passes, 0 fails, 0 exceptions
[ View ]

Now improved with a reroll and 4 less bytes. Get it now while it's hot.

Status:Needs review» Needs work

The last submitted patch failed testing.

Status:Needs work» Needs review

Status:Needs review» Needs work

The last submitted patch failed testing.

Status:Needs work» Needs review

Hmm don't see how this patch could make the file naming test fail. Maybe bad result? Moving back to review

I have submitted the patch for re-testing.

Status:Needs review» Needs work

The last submitted patch failed testing.

Status:Needs work» Needs review

Status:Needs review» Needs work

I can't see the select all checkbox in the floating table header at all (ff3/ubuntu) - with or without the patch.

Status:Needs work» Needs review
StatusFileSize
new919 bytes
PASSED: [[SimpleTest]]: [MySQL] 23,291 pass(es).
[ View ]

Even smaller patch.

I do see the select-all checkbox (FF3/XP).

Still applies. anyone?

#14: tableselect-tableheader.patch queued for re-testing.

Version:7.x-dev» 8.x-dev
Status:Needs review» Reviewed & tested by the community
StatusFileSize
new803 bytes
PASSED: [[SimpleTest]]: [MySQL] 34,583 pass(es).
[ View ]
new783 bytes
PASSED: [[SimpleTest]]: [MySQL] 38,281 pass(es).
[ View ]

just a reroll, don't credit me.

Works fine, 'nuf said :)

Version:8.x-dev» 7.x-dev
Status:Reviewed & tested by the community» Patch (to be ported)
Issue tags:+needs backport to D7

Status:Patch (to be ported)» Needs review
StatusFileSize
new783 bytes
PASSED: [[SimpleTest]]: [MySQL] 38,277 pass(es).
[ View ]

Reuploading nod_'s D7 patch from #17 for testbot.

Status:Needs review» Reviewed & tested by the community

Version:7.x-dev» 6.x-dev
Status:Reviewed & tested by the community» Patch (to be ported)

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/efd12ac

Moving to 6.x for possible backport.

Status:Patch (to be ported)» Needs review
StatusFileSize
new780 bytes
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

D6 backport.