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

Invalid HTML generated by website #1170

Open
5 of 8 tasks
Tracked by #1266
, #1511
...
chalin opened this issue Feb 24, 2022 · 0 comments
Open
5 of 8 tasks
Tracked by #1266
, #1511
...

Invalid HTML generated by website #1170

chalin opened this issue Feb 24, 2022 · 0 comments
Labels
bug Something isn't working e0-minutes Effort: < 60 min p2-medium

Comments

@chalin
Copy link
Contributor

chalin commented Feb 24, 2022 •
edited
Loading

For example, consider the following:

opentelemetry.io/content/en/vendors/_index.md

Lines 7 to 14 in 7e86a7f

<section class="row td-box td-box--1 position-relative td-box--gradient td-box--height-auto">
<div class="container text-center td-arrow-down">
<h1>Vendors Supporting OpenTelemetry</h1>
<span class="h4 mb-0">
<p>Distributions and vendors who natively support OpenTelemetry in their commercial products.</p>
</span>
</div>
</section>

Notice how the <span> contains a <p>, which isn't permitted. There are more such occurrences, search for <\w+ class="h4 for example.

Related:
@chalin chalin added bug Something isn't working e0-minutes Effort: < 60 min p2-medium labels Feb 24, 2022
This was referenced Feb 24, 2022
Merged
Merged
Merged
Closed
Merged
@svrnm svrnm mentioned this issue Mar 18, 2022
Closed
@chalin chalin mentioned this issue Apr 2, 2022
Closed
24 tasks
@chalin chalin mentioned this issue Jul 5, 2022
Closed
32 tasks
@chalin chalin mentioned this issue Jun 27, 2023
Merged
@chalin chalin changed the title Some pages contain invalid HTML Invalid HTML generated by website Jun 27, 2023
This was referenced Jun 27, 2023
Merged
Merged
carlosalberto pushed a commit to open-telemetry/opentelemetry-specification that referenced this issue Jun 28, 2023
@chalin
[editorial] Data model appendix: table-entry markdown and typo fixes (#…
c07f4a2 
…3571)

- Contributes to
open-telemetry/opentelemetry.io#1170, because
of OTel spec pages
- Replaces `attribute["client.address"]` by
`Attributes["client.address"]`, since `attribute` appears to be a
**typo**
- Fixes `<td>` spread across multiple lines with intervening blank
lines, since that generates **invalid HTML** in some cases.
  - In most cases I replaced line breaks with `<br>`.
- Replaces code tick markdown syntax in two places since those entries
are in "HTML-only" format.
  For example: replaces \`Debug\` by `<code>Debug</code>`

/cc @svrnm @cartermp
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this issue Oct 31, 2024
@chalin
[editorial] Data model appendix: table-entry markdown and typo fixes (o…
d3776f9 
…pen-telemetry#3571)

- Contributes to
open-telemetry/opentelemetry.io#1170, because
of OTel spec pages
- Replaces `attribute["client.address"]` by
`Attributes["client.address"]`, since `attribute` appears to be a
**typo**
- Fixes `<td>` spread across multiple lines with intervening blank
lines, since that generates **invalid HTML** in some cases.
  - In most cases I replaced line breaks with `<br>`.
- Replaces code tick markdown syntax in two places since those entries
are in "HTML-only" format.
  For example: replaces \`Debug\` by `<code>Debug</code>`

/cc @svrnm @cartermp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working e0-minutes Effort: < 60 min p2-medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chalin