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.

Sign up for GitHub

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

Jump to bottom

thead and other table elements allow xml:id #39

Closed
ghost opened this issue Jan 29, 2018 · 0 comments
Closed

thead and other table elements allow xml:id #39

ghost opened this issue Jan 29, 2018 · 0 comments
Assignees
@tomschr
Labels
bug 🐛 Something isn't working as expected
Milestone
1.0.3

Comments

@ghost
Copy link

ghost commented Jan 29, 2018 •
edited by ghost
Loading

GeekoDoc 1.0.1 already disallows xml:id on row and entry but preety much all other table-related elements still allow xml:ids even though they should not:

  • thead
  • colspec
  • tgroup
  • tbody

(List is not exhaustive, just the few that I immediately thought of and tried.)
@ghost ghost changed the title thead and othe table elements allow xml:id thead and other table elements allow xml:id Jan 29, 2018
@tomschr tomschr added the bug 🐛 Something isn't working as expected label Feb 7, 2018
@tomschr tomschr added this to the 1.0.3 milestone Feb 7, 2018
tomschr added a commit that referenced this issue Feb 9, 2018
@tomschr
Fix #39 disallow xml:id on child elements of table
5e00877 
thead, colspec, tgroup, tbody
@tomschr tomschr self-assigned this Mar 1, 2018
@ghost ghost closed this as completed in 69ecf26 May 15, 2018
tomschr added a commit that referenced this issue Jun 22, 2018
@tomschr
Merge tag 'v1.0.3' into develop
cbbf37e 
Bugfix release 1.0.3

Geekodoc:
 * disallow xml:id on child elements of table (#39)
 * Missing "fig" value in format attribute of imagedta (#43)
 * add missing holder tag (#46)
 * integrate its:translate (#45)
 * output XSLT stylesheet from Schematron validation (#49)
 * remove action in keycombo (#50)
 * reduce content model in screen (#51)
 * remove procedure from table entry (#56)

 Others:
 * Make sure GeekoDoc is available in OBS/IBS, so we can use
   it from DC file (#33)

Novdoc:
  no changes
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomschr tomschr

Labels
bug 🐛 Something isn't working as expected
Projects
None yet
Milestone
1.0.3
Development

No branches or pull requests
1 participant
@tomschr