Copy table header is wrong - and questionable practice

[Dan Scott]

* Evergreen master

Currently there is a <td headers='copy_header_date_format'> entry in copy_table.tt2 that doesn't actually link to an associated table header with an ID of that value. It should be <td headers='copy_header_create_date'>

However, per http://webaim.org/techniques/tables/data the current recommendation is to avoid the use of the headers approach in favour of the much simpler scope attribute for headers (one of the reasons being that as things change, they get out of alignment and errors are unlikely to be noticed... as happened here. Heh.)

[Dan Scott (denials)]

For the conservative "just fix the broken ID reference" approach, see http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbs/lp1305958_fix_copy_table_header

[Dan Scott (denials)]

And for the "use recommended practices (scope='col') approach", see http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbs/lp1305958_th_scope

[Dan Wells (dbw2)]

I took the "conservative" fix, but retargeting to 2.6.1 to get some eyes on the better fix. (To any potential testers, remember you'll need to revert the other branch, or do some minor massaging to get the second branch to apply.) Thanks, Dan!

[Chris Sharp (chrissharp123)]

Copy table loads correctly on a rebased master using the "better practice" branch. Signoff branch here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1305958_th_scope_signoff

[Ben Shum (bshum)]

I took csharp's signoff and pushed it to master and rel_2_7. rel_2_6 has a conflict and needs some massaging to apply if we wish to backport there too?