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

consistently render, or omit, labels for tables #824

Closed
zygoloid opened this issue Jul 12, 2016 · 5 comments
Closed

consistently render, or omit, labels for tables #824

zygoloid opened this issue Jul 12, 2016 · 5 comments
Assignees

Comments

@zygoloid
Copy link
Member

We currently render the labels for concept tables, but for no other tables:

Table 21 — MoveConstructible requirements [moveconstructible]

If this is useful, we should do it for all tables. If not, we should remove it from the concept tables.

We will also need to make sure that there are no name collisions between table labels and section labels if we choose to show them, to avoid confusion. Perhaps we could render the 'tab:' prefix.

@jwakely
Copy link
Member

jwakely commented Jul 12, 2016

Table labels don't have a consistent style, e.g. tab:HashRequirements uses camel case. If we start showing them we should review the labels and change some of them.

@AlisdairM
Copy link
Contributor

Table numbers are not consistent, as new tables get inserted (or removed) so having some kind of name associated with each table is helpful when filing issues.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 13, 2016

The table stable name doesn't have to be the same as the label, though for the sake of editing the text, stable table labels would be nice. And yes, we can rename them until we start displaying them.

@jensmaurer
Copy link
Member

It seems to me we don't really have to render labels for tables. Usually, a table is associated with a single paragraph that we can reference. The labels shown for the "concept tables" look actually somewhat disgusting to me.

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Dec 13, 2016
@jensmaurer
Copy link
Member

Editorial meeting consensus: There are no other mentions of the labels in the document. The labels for the tables are not used in the standard. Remove the table labels.

@zygoloid zygoloid removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Mar 2, 2017
jensmaurer added a commit to jensmaurer/draft that referenced this issue Mar 29, 2017
@jensmaurer jensmaurer self-assigned this Mar 29, 2017
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

No branches or pull requests

5 participants