Skip to content

Fix non-self-closing <br> tags in configuration.md#1169

Merged
chrjohn merged 2 commits into
masterfrom
copilot/fix-unmatched-br-tags
Mar 20, 2026
Merged

Fix non-self-closing <br> tags in configuration.md#1169
chrjohn merged 2 commits into
masterfrom
copilot/fix-unmatched-br-tags

Conversation

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

configuration.md contained 62 instances of <br> (non-self-closing), which are invalid in XHTML/XML contexts.

Changes

  • quickfixj-core/src/main/doc/usermanual/usage/configuration.md: Global replace of <br><br/> throughout all table cells where multiple valid values are listed inline.

Before / After

`initiator`<br>`acceptor`        <!-- before -->
`initiator`<br/>`acceptor`       <!-- after -->

`Y`<br>`N`                       <!-- before -->
`Y`<br/>`N`                      <!-- after -->
Original prompt

Fix unmatched <br> tags in configuration.md

In the file quickfixj-core/src/main/doc/usermanual/usage/configuration.md, there are numerous <br> tags used inside Markdown table cells to separate multiple valid values. These are not self-closing (i.e., they use <br> instead of <br/>), which is not valid in XML/XHTML contexts.

Task

Replace all occurrences of <br> with <br/> in the file:

File: quickfixj-core/src/main/doc/usermanual/usage/configuration.md

This affects many lines throughout the file, including but not limited to:

  • Line 62: `FIX.4.4`<br>`FIX.4.3`<br>...
  • Line 71: `initiator`<br>`acceptor`
  • Line 78–80: `Y`<br>`N` and similar
  • Line 79: `SECONDS`<br>`MILLIS`<br>`MICROS`<br>`NANOS`
  • Lines 88–107: many rows with `Y`<br>`N`
  • Lines 122, 125, 138: `TCP`<br>`VM_PIPE`
  • Lines 148, 157, 161, 170, 171 and many more

Expected Change

Every <br> in the file should be replaced with <br/> to be XHTML-compliant self-closing tags.

Please do a global find-and-replace of <br><br/> within that single file only.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: Find any unmatched br tags

Assistant: After scanning the file for <br> tags, all of them are self-closing (<br>) used inside Markdown table cells as line separators between list-like values. This is a common HTML pattern in Markdown tables.

Here's a review of how they are used:

All <br> tags in the file are used as inline separators within table cells to list multiple valid values (e.g., `Y`<br>`N`). In Markdown/HTML, <br> is a void element — it does not require a closing tag, and none is expected.

No unmatched <br> tags were found. Every <br> in the file is a standalone void element used correctly. There are no instances of <br> paired with </br>, and none are left dangling in an unexpected context.

If you were looking for <br> tags that are missing their matching </br> closing counterpart in an XML-strict context (e.g., XHTML), then all occurrences would technically be "unmatched." Here's a summary of those lines:

Line Content (abbreviated)
62 `FIX.4.4`<br>`FIX.4.3`<br>...
71 `initiator`<br>`acceptor`
78 `Y`<br>`N`
79 `SECONDS`<br>`MILLIS`<br>`MICROS`<br>`NANOS`
80 `Y`<br>`N`
88–107 Many rows with `Y`<br>`N`
122, 125, 138 `TCP`<br>`VM_PIPE`
148, 157, 161, etc. `Y`<br>`N`
170, 171 Proxy type/version values

Would you like me to open a PR to convert these to <br/> (XHTML-compliant self-closing tags), or is this purely an audit?

User: Please open pr

Assistant: I'm waiting for your response to the confirmation dialog to proceed with opening the pull request.

This pull request was created from Copilot chat.


⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

Copilot AI changed the title [WIP] Fix unmatched <br> tags in configuration.md Fix non-self-closing <br> tags in configuration.md Mar 20, 2026
Copilot AI requested a review from chrjohn March 20, 2026 16:39
@chrjohn chrjohn changed the title Fix non-self-closing <br> tags in configuration.md Fix non-self-closing <br> tags in configuration.md Mar 20, 2026
@chrjohn chrjohn marked this pull request as ready for review March 20, 2026 16:47
@chrjohn chrjohn merged commit 9002416 into master Mar 20, 2026
20 checks passed
@chrjohn chrjohn added this to the QFJ 3.0.1 milestone Mar 20, 2026
@chrjohn chrjohn deleted the copilot/fix-unmatched-br-tags branch March 20, 2026 16:48
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.

2 participants