Beamer frame attributes not always output when using pandoc-ling filter about pandoc-ling HOT 5 OPEN

It looks like maybe the frame attribute classes are getting removed from first level headings at line 219:

I don't know enough about Lua to suggest a fix, but I think any solution would probably need to take into account lines 230-238 as well:

from pandoc-ling.

The change in my fork seems to work? Again, I don't know much Lua, but the changes I made:

1. Add a "restart" class to the header instead of overwriting existing ones
2. Only remove the inserted "restart" class when it's done instead of existing classes.

The removal code is based on the example for minted here:

https://github.com/pandoc/lua-filters/blob/2aa98bfda556c7d4dfb8e30c20b318b6fd1f5091/minted/minted.lua#L237-L248

Running on the modified filter on my example maintains the beamer frame attributes, and I tried it with a few examples from the readme here that didn't seem to break anything. But, again, I don't know much about Lua. :)

from pandoc-ling.

I just realized I probably broke the intent of tracking the headers by accidental removing the check for "restart". :)

from pandoc-ling.

I did some further tests, and I guess I didn't actually seem to break it. But something doesn't seem quite right about skipping the check.

from pandoc-ling.

Ok. I figured out the reason it still worked was because only the first level headers were being converted to divs.

The second commit in my fork (somelinguist@4e4da4e) adds back the check and is probably more efficient than my first attempt. It just inserts the "restart" class at index 1 so that it's easy to check later and remove.

from pandoc-ling.