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

Stop bolding multi-line table headers by default #33

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Conversation

pcraig3
Copy link
Collaborator

@pcraig3 pcraig3 commented Nov 8, 2024

Summary

It used to be that we would just bold all table headings. But sometimes we have more complex table headings with several lines, typically the first line would be bold and then the ones after wouldn't be.

Here's an example:

Screenshot 2024-11-08 at 6 29 09 PM

The update in this PR is to stop bolding paragraph tags inside of table headings. Mechanically, how the app works is that we leave the content of table headings in paragraph tags if there are hard returns in there.

Here are some examples:

<th>Heading 1</th> <!-- bold -->
<th>
  <p>Heading 1:</p>  <!-- not bold -->
  <p>This is the first heading</p> <!-- not bold -->
</th>
<th>
   Heading 1:  <!-- bold -->
  <p><strong>Important:</strong> This is the first heading</p> <!-- bold, then not bold -->
</th>

Really we are just removing bolds from paragraphs inside of table cells.

It's a bit weird because table cells can be multi-line and single line
in the same table.

In this case, we just have an inconsistent table. I plan to take a 'wait and see'
approach to this.

Here are some examples:

```html
<th>Heading 1</th> <!-- bold -->
```

```html
<th>
  <p>Heading 1:</p>  <!-- not bold -->
  <p>This is the first heading</p> <!-- not bold -->
</th>
```

```html
<th>
   Heading 1:  <!-- bold -->
  <p><strong>Important:</strong> This is the first heading</p> <!-- bold, then not bold -->
</th>
```
I've made lots of changes, so let's version the app again.
@pcraig3 pcraig3 merged commit b85a49c into main Nov 8, 2024
1 check passed
@pcraig3 pcraig3 deleted the tables-not-bold branch November 13, 2024 16:23
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.

1 participant