Re: SVG AAM and mpping table script

As Michale notes, the "view as table" prior to respec snapshot is a solution because the mapping-tables script rewrites the table content as a bunch of details/summary elements, one for each row, and then hides the table. In that process, the IDs on the table rows are stored in an array, applied to the details elements, and removed from the table rows. If the respec snapshot is taken in this view, then the table rows have no IDs, and the mapping-tables script works from the assumption that the initial HTML is a table with IDs on the TRs, so when it tries to work on this snapshot DOM, it finds no IDs on the table TRs, hence the "undefined”.

Should be easy enough for the mapping-tables script to check at start where the IDs live (on the table row or on the details/summary) and build the page and mapping-tables functionality as appropriate. I can take a stab.

Jason


> On 13/04/2018, at 13:35, Michael Cooper <cooper@w3.org> wrote:
> 
> On 12/04/2018 8:45 PM, Amelia Bellamy-Royds wrote:
>> >   the fix is to activate the "View as a single table" button for each table before saving a snapshot from Respec.
>> 
>> How feasible is it to get Core-AAM,
> First, what are the broken links you have to Core-AAM? I just checked Core-AAM and it seems I did apply the fix when publishing, so the ids are there on <tr>s. And I just checked SVG-AAM in the master branch (using a Respec snapshot) and don't find broken links to Core-AAM (nor to the others).
> 
> If there is a broken link that is the result of a publication fault, I can ask the webmaster for permission to fix. If the fix is beyond what we are allowed to do in place for a Recommendation, it would have to wait for a First Public Working Draft of Core-AAM 1.2. That might happen in the next few months but I can't really say more specifically right now.
>> HTML-AAM,
> That's maintained by Web Platform, so you'd have to ask them. I expect they'd be willing to publish a new snapshot of a WD.
>> and Graphics-AAM
> I wouldn't want to touch a Candidate Recommendation (though I can ask the webmaster for permission to fix, as for Core-AAM). We plan to transition to Proposed Recommendation soon. I do see that in this one I forgot to apply the workaround when making the snapshot, which is a bit surprising since a broken internal link report generally reminds me. But, I don't see broken links to this from my own link check of SVG-AAM.
>> all republished with that fix?  Because a WD build of SVG-AAM has broken links to the TR versions of all three, as well as the broken internal links.
>> 
>> My guess is that republishing three specs is enough of a bother that it's probably worth fixing the JS code first. But then, I haven't done a review of the code to identify the problem.
> As far as I can tell when I looked at this, it's a "feature" not a "bug" that the script removes the IDs, as part of changing the table to a set of twistie ties. I don't know if it's inherently necessary to do it that way, but if it is, the only script fix would be to get Respec to disable that script or activate the "view as table" buttons first when exporting a snapshot. Which I imagine is possible but beyond my current skills. If I ever find I have time for a deep dive into ARIA maintenance scripts I was going to look into this, but it's really out of my skill set and not front burner unfortunately, I just deal with the workaround (which I obviously need some system to ensure I remember to use).
> 
> Michael
>> 
>> ~ Amelia
>> 
>> 
>> 
>> On 12 April 2018 at 18:01, Michael Cooper <cooper@w3.org> wrote:
>> If this is the problem I'm aware of, the fix is to activate the "View as a single table" button for each table before saving a snapshot from Respec.
>> 
>> This is of course not ideal. I don't know if it's possible to adjust the script to not require this workaround. I'll put on editors' agenda to discuss, though we will only make progress if someone knowledgeable about the script attends. Otherwise continuing discussion on the issue you referenced seems to make sense. Sorry I hadn't seen that issue before - GitHub doesn't reliably notify me of new issues filed even though it's set to.
>> 
>> Michael
>> 
>> On 12/04/2018 7:44 PM, Liam R. E. Quin wrote:
>>> SVG AAM is blocked in pubrules because of a script,
>>> mapping-tables.js, that seems to be breaking links
>>> 
>>> https://github.com/w3c/aria-common/issues/7
>>> 
>>> 
>>> Could this be on an agenda? Or should i dive in and try and fix the
>>> JavaScript?
>>> 
>>> or is this the wrong list?
>>> 
>>> Thanks!
>>> 
>>> Liam
>>> 
>>> 
>> 
>> 
>

Received on Friday, 13 April 2018 01:57:30 UTC