Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix Children Pres. for math #1172

Closed
wants to merge 1 commit into from

Conversation

WilcoFiers
Copy link
Contributor

@WilcoFiers WilcoFiers commented Jan 25, 2020

According to section 7.1, math is intended to have "children presentational: True". This PR fixes that. Alternatively, 7.1 could be updated, but this seemed like the right way this was supposed to work.


Preview | Diff

According to section 7.1, math is intended to have
children presentational: True This PR fixes this.
Copy link
Member

@jnurthen jnurthen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was changed by #425
Good catch that we missed removing from the list in 7.1.
It may be easiest to close this PR and create a new one for that.

@carmacleod
Copy link
Contributor

Heh, #425 reads like a suspense novel... will math children be presentational... or not? Of course, I already know the ending, but I read the story anyhow so that I could follow the plot twists. 😄

So, I've updated the list of roles with children presentational in 7.1 to remove math and add meter (which was missing). I committed the fix to PR #1100, which already contains changes to 7.1 and 7.2, and really needs to be merged.

This PR (#1172) can serve as the corresponding "issue" to update the list of roles in 7.1.

@jnurthen
Copy link
Member

jnurthen commented Mar 2, 2020

Lets close this one then

@jnurthen jnurthen closed this Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has PR PR exists that will close this issue WR comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants