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

measure part of att.ranging #2342

Merged
merged 7 commits into from
Oct 22, 2022
Merged

measure part of att.ranging #2342

merged 7 commits into from
Oct 22, 2022

Conversation

bleekere
Copy link
Contributor

Made <measure> part of att.ranging and added some examples of how this element and the att.ranging attributes could be used to express a range cf. #2254 .

@peterstadler peterstadler linked an issue Oct 21, 2022 that may be closed by this pull request
@peterstadler
Copy link
Member

Release team looked at this and fixed some minor issues. From a formal perspective this seems ready to go – or was there more you wanted to do here ( @sydb) ?

@sydb
Copy link
Member

sydb commented Oct 21, 2022

Nothing substantive, but if there are no objections there are some formatting fixes to the examples that are worth making now. (They may not matter with current egXML processing, but when we fix that, we are going to wish we had these formatted better …)

Probably will make little, if any difference using current Stylesheets, but if & when we improve egXML formatting we will apprciate these minor changes.
@sydb
Copy link
Member

sydb commented Oct 21, 2022

Pushed the formatting changes I spoke of a few hours ago. IMHO this is ready to be merged. (Unless 'dev' branch needs to be merged into this branch again, first. :-)

@martinascholger martinascholger merged commit 1427c0a into dev Oct 22, 2022
peterstadler added a commit to TEIC/Stylesheets that referenced this pull request Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

<measure> should be a member of att.ranging
5 participants