Re: respec: nested data-include usage

On 26/06/2019 05:21, Marcos Caceres wrote:

Hi David,



On Jun 25, 2019, at 8:13 PM, David Carlisle <davidc@nag.co.uk><mailto:davidc@nag.co.uk> wrote:



The include filter runs recursively so that included content that contains data-include attributes will work



so I think it's supposed to work?



I checked the code and it doesn't seem we support recursion

That was my reading of the code (but wasn't sure if I was looking in enough places:-)

I think also the data-include-sync attribute mentioned in the documentation doesn't exist in the current code?


- so apologies if the documentation is incorrect:

https://github.com/w3c/respec/blob/develop/src/core/data-include.js


It should be trivial to add support for recursion, however: just need to walk check for more `data-include`s after included. It's not something I can work on at the moment, however - but if you want to have a go at adding support, I can walk you through it over on GitHub.



OK thanks, Not sure I have time to get my head around respec internals just at the moment (one reason for wanting to switch was so I could stop being on the critical path for maintaining the mathml xmlspec derived pipeline:-)

If It's Ok with you I may open an issue in the respec github but not actively look at fixing it in the short term, I could ping again on that issue if I find time to look at it seriously.


David

Disclaimer

The Numerical Algorithms Group Ltd is a company registered in England and Wales with company number 1249803. The registered office is:
Wilkinson House, Jordan Hill Road, Oxford OX2 8DR, United Kingdom.

This e-mail has been scanned for all viruses and malware, and may have been automatically archived by Mimecast Ltd, an innovator in Software as a Service (SaaS) for business.