Skip to content

fix: correct expand-all icon SVG in HTML report#5105

Merged
thomhurst merged 1 commit intothomhurst:mainfrom
slang25:fix-report-expand-all-icon
Mar 8, 2026
Merged

fix: correct expand-all icon SVG in HTML report#5105
thomhurst merged 1 commit intothomhurst:mainfrom
slang25:fix-report-expand-all-icon

Conversation

@slang25
Copy link
Contributor

@slang25 slang25 commented Mar 8, 2026

Summary

  • Fixed the "Expand all groups" icon SVG in the HTML report where the top-left and top-right corner paths had broken coordinates drawing outside the 0-16 viewBox, resulting in a wonky/asymmetric icon
  • Corrected by properly mirroring the working bottom corner paths vertically

Test plan

  • Generate an HTML report and visually verify the expand-all icon renders symmetrically with all four corner brackets visible

🤖 Generated with Claude Code

The top-left and top-right corners of the expandAll SVG had broken
coordinates that drew outside the 0-16 viewBox, resulting in a wonky
icon. Fixed by properly mirroring the working bottom corners vertically.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@slang25
Copy link
Contributor Author

slang25 commented Mar 8, 2026

Before / After

image

The Before icon has the top-left corner bracket oversized/misshapen and the top-right similarly broken — both had SVG path coordinates drawing outside the 0–16 viewBox. The After shows all four corner brackets rendering symmetrically as intended.

@thomhurst thomhurst merged commit 4ce8731 into thomhurst:main Mar 8, 2026
2 checks passed
This was referenced Mar 9, 2026
This was referenced Mar 16, 2026
PhilippNaused pushed a commit to PhilippNaused/Bshox that referenced this pull request Mar 18, 2026
Updated [TUnit.Core](https://github.com/thomhurst/TUnit) from 1.19.0 to
1.19.74.

<details>
<summary>Release notes</summary>

_Sourced from [TUnit.Core's
releases](https://github.com/thomhurst/TUnit/releases)._

## 1.19.74

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.74 -->

## What's Changed
### Other Changes
* feat: per-hook activity spans with method names by @​thomhurst in
thomhurst/TUnit#5159
* fix: add tooltip to truncated span names in HTML report by @​thomhurst
in thomhurst/TUnit#5164
* Use enum names instead of numeric values in test display names by
@​Copilot in thomhurst/TUnit#5178
* fix: resolve CS8920 when mocking interfaces whose members return
static-abstract interfaces by @​lucaxchaves in
thomhurst/TUnit#5154
### Dependencies
* chore(deps): update tunit to 1.19.57 by @​thomhurst in
thomhurst/TUnit#5157
* chore(deps): update dependency gitversion.msbuild to 6.6.1 by
@​thomhurst in thomhurst/TUnit#5160
* chore(deps): update dependency gitversion.tool to v6.6.1 by
@​thomhurst in thomhurst/TUnit#5161
* chore(deps): update dependency polyfill to 9.20.0 by @​thomhurst in
thomhurst/TUnit#5163
* chore(deps): update dependency polyfill to 9.20.0 by @​thomhurst in
thomhurst/TUnit#5162
* chore(deps): update dependency polyfill to 9.21.0 by @​thomhurst in
thomhurst/TUnit#5166
* chore(deps): update dependency polyfill to 9.21.0 by @​thomhurst in
thomhurst/TUnit#5167
* chore(deps): update dependency polyfill to 9.22.0 by @​thomhurst in
thomhurst/TUnit#5168
* chore(deps): update dependency polyfill to 9.22.0 by @​thomhurst in
thomhurst/TUnit#5169
* chore(deps): update dependency coverlet.collector to 8.0.1 by
@​thomhurst in thomhurst/TUnit#5177

## New Contributors
* @​lucaxchaves made their first contribution in
thomhurst/TUnit#5154

**Full Changelog**:
thomhurst/TUnit@v1.19.57...v1.19.74

## 1.19.57

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.57 -->

## What's Changed
### Other Changes
* fix: use unique artifact names to avoid collisions in matrix builds by
@​thomhurst in thomhurst/TUnit#5132
* fix: resolve IndexOutOfRangeException with MethodDataSource<T> on
class (#​5118) by @​thomhurst in
thomhurst/TUnit#5137
* fix: prevent StringEqualsAssertion from matching non-string types by
@​thomhurst in thomhurst/TUnit#5156
### Dependencies
* chore(deps): update tunit to 1.19.22 by @​thomhurst in
thomhurst/TUnit#5117
* chore(deps): update dependency fsharp.core to 10.0.104 by @​thomhurst
in thomhurst/TUnit#5119
* chore(deps): update dependency microsoft.entityframeworkcore to 10.0.4
by @​thomhurst in thomhurst/TUnit#5120
* chore(deps): update dependency fsharp.core to v11 by @​thomhurst in
thomhurst/TUnit#5128
* chore(deps): update dependency microsoft.templateengine.authoring.cli
to v10.0.200 by @​thomhurst in
thomhurst/TUnit#5122
* chore(deps): update dependency dotnet-sdk to v10.0.200 by @​thomhurst
in thomhurst/TUnit#5123
* chore(deps): update dependency microsoft.sourcelink.github to 10.0.200
by @​thomhurst in thomhurst/TUnit#5121
* chore(deps): update dependency system.commandline to 2.0.4 by
@​thomhurst in thomhurst/TUnit#5125
* chore(deps): update microsoft.extensions to 10.0.4 by @​thomhurst in
thomhurst/TUnit#5127
* chore(deps): update microsoft.build to 18.4.0 by @​thomhurst in
thomhurst/TUnit#5129
* chore(deps): update microsoft.aspnetcore to 10.0.4 by @​thomhurst in
thomhurst/TUnit#5126
* chore(deps): update dependency
microsoft.templateengine.authoring.templateverifier to 10.0.200 by
@​thomhurst in thomhurst/TUnit#5124
* chore(deps): update microsoft.extensions to 10.4.0 by @​thomhurst in
thomhurst/TUnit#5130
* chore(deps): update dependency
opentelemetry.instrumentation.aspnetcore to 1.15.1 by @​thomhurst in
thomhurst/TUnit#5136
* chore(deps): update dependency vogen to 8.0.5 by @​thomhurst in
thomhurst/TUnit#5133
* chore(deps): update dependency npgsql to 10.0.2 by @​thomhurst in
thomhurst/TUnit#5139
* chore(deps): update dependency microsoft.sourcelink.github to 10.0.201
by @​thomhurst in thomhurst/TUnit#5141
* chore(deps): update dependency microsoft.entityframeworkcore to 10.0.5
by @​thomhurst in thomhurst/TUnit#5140
* chore(deps): update dependency microsoft.templateengine.authoring.cli
to v10.0.201 by @​thomhurst in
thomhurst/TUnit#5142
* chore(deps): update dependency
microsoft.templateengine.authoring.templateverifier to 10.0.201 by
@​thomhurst in thomhurst/TUnit#5143
* chore(deps): update dependency npgsql.entityframeworkcore.postgresql
to 10.0.1 by @​thomhurst in thomhurst/TUnit#5145
* chore(deps): update dependency dotnet-sdk to v10.0.201 by @​thomhurst
in thomhurst/TUnit#5144
* chore(deps): update dependency system.commandline to 2.0.5 by
@​thomhurst in thomhurst/TUnit#5146
* chore(deps): update microsoft.aspnetcore to 10.0.5 by @​thomhurst in
thomhurst/TUnit#5147
* chore(deps): update dependency testcontainers.kafka to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5149
* chore(deps): update microsoft.extensions to 10.0.5 by @​thomhurst in
thomhurst/TUnit#5148
* chore(deps): update dependency testcontainers.postgresql to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5150
* chore(deps): update dependency testcontainers.redis to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5151
* chore(deps): update dependency stackexchange.redis to 2.12.1 by
@​thomhurst in thomhurst/TUnit#5153


**Full Changelog**:
thomhurst/TUnit@v1.19.22...v1.19.57

## 1.19.22

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.22 -->

## What's Changed
### Other Changes
* Remove redundant prefixes from tracing timeline spans by @​thomhurst
in thomhurst/TUnit#5111
* Render span tags and events on separate lines by @​thomhurst in
thomhurst/TUnit#5113
* Demote granular internal logs from Debug to Trace level by @​thomhurst
in thomhurst/TUnit#5116
### Dependencies
* chore(deps): update tunit to 1.19.16 by @​thomhurst in
thomhurst/TUnit#5109


**Full Changelog**:
thomhurst/TUnit@v1.19.16...v1.19.22

## 1.19.16

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.16 -->

## What's Changed
### Other Changes
* Truncate exceptions in GitHub summary tables by @​thomhurst in
thomhurst/TUnit#5108
### Dependencies
* chore(deps): update tunit to 1.19.11 by @​thomhurst in
thomhurst/TUnit#5106
* chore(deps): bump dompurify from 3.3.0 to 3.3.2 in /docs by
@​dependabot[bot] in thomhurst/TUnit#5096
* chore(deps): bump svgo from 3.2.0 to 3.3.3 in /docs by
@​dependabot[bot] in thomhurst/TUnit#5086


**Full Changelog**:
thomhurst/TUnit@v1.19.11...v1.19.16

## 1.19.11

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.11 -->

## What's Changed
### Other Changes
* Fix HTML report sort to also reorder groups by @​thomhurst in
thomhurst/TUnit#5103
* fix: correct expand-all icon SVG in HTML report by @​slang25 in
thomhurst/TUnit#5105
* Avoid some redundant list allocations by @​SimonCropp in
thomhurst/TUnit#4963
* avoid some redundant enumerable alloc by @​SimonCropp in
thomhurst/TUnit#4972
* use char instead of string for joins by @​SimonCropp in
thomhurst/TUnit#4989
### Dependencies
* chore(deps): update dependency nunit to 4.5.1 by @​thomhurst in
thomhurst/TUnit#5097
* chore(deps): update tunit to 1.19.0 by @​thomhurst in
thomhurst/TUnit#5099
* chore(deps): update dependency humanizer to 3.0.10 by @​thomhurst in
thomhurst/TUnit#5101
* chore(deps): update dependency nunit.analyzers to 4.12.0 by
@​thomhurst in thomhurst/TUnit#5102


**Full Changelog**:
thomhurst/TUnit@v1.19.0...v1.19.11

Commits viewable in [compare
view](thomhurst/TUnit@v1.19.0...v1.19.74).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=TUnit.Core&package-manager=nuget&previous-version=1.19.0&new-version=1.19.74)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore <dependency name> major version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's major version (unless you unignore this specific
dependency's major version or upgrade to it yourself)
- `@dependabot ignore <dependency name> minor version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's minor version (unless you unignore this specific
dependency's minor version or upgrade to it yourself)
- `@dependabot ignore <dependency name>` will close this group update PR
and stop Dependabot creating any more for the specific dependency
(unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore <dependency name>` will remove all of the ignore
conditions of the specified dependency
- `@dependabot unignore <dependency name> <ignore condition>` will
remove the ignore condition of the specified dependency and ignore
conditions


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
PhilippNaused pushed a commit to PhilippNaused/Bshox that referenced this pull request Mar 19, 2026
Updated [TUnit](https://github.com/thomhurst/TUnit) from 1.19.0 to
1.19.74.

<details>
<summary>Release notes</summary>

_Sourced from [TUnit's
releases](https://github.com/thomhurst/TUnit/releases)._

## 1.19.74

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.74 -->

## What's Changed
### Other Changes
* feat: per-hook activity spans with method names by @​thomhurst in
thomhurst/TUnit#5159
* fix: add tooltip to truncated span names in HTML report by @​thomhurst
in thomhurst/TUnit#5164
* Use enum names instead of numeric values in test display names by
@​Copilot in thomhurst/TUnit#5178
* fix: resolve CS8920 when mocking interfaces whose members return
static-abstract interfaces by @​lucaxchaves in
thomhurst/TUnit#5154
### Dependencies
* chore(deps): update tunit to 1.19.57 by @​thomhurst in
thomhurst/TUnit#5157
* chore(deps): update dependency gitversion.msbuild to 6.6.1 by
@​thomhurst in thomhurst/TUnit#5160
* chore(deps): update dependency gitversion.tool to v6.6.1 by
@​thomhurst in thomhurst/TUnit#5161
* chore(deps): update dependency polyfill to 9.20.0 by @​thomhurst in
thomhurst/TUnit#5163
* chore(deps): update dependency polyfill to 9.20.0 by @​thomhurst in
thomhurst/TUnit#5162
* chore(deps): update dependency polyfill to 9.21.0 by @​thomhurst in
thomhurst/TUnit#5166
* chore(deps): update dependency polyfill to 9.21.0 by @​thomhurst in
thomhurst/TUnit#5167
* chore(deps): update dependency polyfill to 9.22.0 by @​thomhurst in
thomhurst/TUnit#5168
* chore(deps): update dependency polyfill to 9.22.0 by @​thomhurst in
thomhurst/TUnit#5169
* chore(deps): update dependency coverlet.collector to 8.0.1 by
@​thomhurst in thomhurst/TUnit#5177

## New Contributors
* @​lucaxchaves made their first contribution in
thomhurst/TUnit#5154

**Full Changelog**:
thomhurst/TUnit@v1.19.57...v1.19.74

## 1.19.57

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.57 -->

## What's Changed
### Other Changes
* fix: use unique artifact names to avoid collisions in matrix builds by
@​thomhurst in thomhurst/TUnit#5132
* fix: resolve IndexOutOfRangeException with MethodDataSource<T> on
class (#​5118) by @​thomhurst in
thomhurst/TUnit#5137
* fix: prevent StringEqualsAssertion from matching non-string types by
@​thomhurst in thomhurst/TUnit#5156
### Dependencies
* chore(deps): update tunit to 1.19.22 by @​thomhurst in
thomhurst/TUnit#5117
* chore(deps): update dependency fsharp.core to 10.0.104 by @​thomhurst
in thomhurst/TUnit#5119
* chore(deps): update dependency microsoft.entityframeworkcore to 10.0.4
by @​thomhurst in thomhurst/TUnit#5120
* chore(deps): update dependency fsharp.core to v11 by @​thomhurst in
thomhurst/TUnit#5128
* chore(deps): update dependency microsoft.templateengine.authoring.cli
to v10.0.200 by @​thomhurst in
thomhurst/TUnit#5122
* chore(deps): update dependency dotnet-sdk to v10.0.200 by @​thomhurst
in thomhurst/TUnit#5123
* chore(deps): update dependency microsoft.sourcelink.github to 10.0.200
by @​thomhurst in thomhurst/TUnit#5121
* chore(deps): update dependency system.commandline to 2.0.4 by
@​thomhurst in thomhurst/TUnit#5125
* chore(deps): update microsoft.extensions to 10.0.4 by @​thomhurst in
thomhurst/TUnit#5127
* chore(deps): update microsoft.build to 18.4.0 by @​thomhurst in
thomhurst/TUnit#5129
* chore(deps): update microsoft.aspnetcore to 10.0.4 by @​thomhurst in
thomhurst/TUnit#5126
* chore(deps): update dependency
microsoft.templateengine.authoring.templateverifier to 10.0.200 by
@​thomhurst in thomhurst/TUnit#5124
* chore(deps): update microsoft.extensions to 10.4.0 by @​thomhurst in
thomhurst/TUnit#5130
* chore(deps): update dependency
opentelemetry.instrumentation.aspnetcore to 1.15.1 by @​thomhurst in
thomhurst/TUnit#5136
* chore(deps): update dependency vogen to 8.0.5 by @​thomhurst in
thomhurst/TUnit#5133
* chore(deps): update dependency npgsql to 10.0.2 by @​thomhurst in
thomhurst/TUnit#5139
* chore(deps): update dependency microsoft.sourcelink.github to 10.0.201
by @​thomhurst in thomhurst/TUnit#5141
* chore(deps): update dependency microsoft.entityframeworkcore to 10.0.5
by @​thomhurst in thomhurst/TUnit#5140
* chore(deps): update dependency microsoft.templateengine.authoring.cli
to v10.0.201 by @​thomhurst in
thomhurst/TUnit#5142
* chore(deps): update dependency
microsoft.templateengine.authoring.templateverifier to 10.0.201 by
@​thomhurst in thomhurst/TUnit#5143
* chore(deps): update dependency npgsql.entityframeworkcore.postgresql
to 10.0.1 by @​thomhurst in thomhurst/TUnit#5145
* chore(deps): update dependency dotnet-sdk to v10.0.201 by @​thomhurst
in thomhurst/TUnit#5144
* chore(deps): update dependency system.commandline to 2.0.5 by
@​thomhurst in thomhurst/TUnit#5146
* chore(deps): update microsoft.aspnetcore to 10.0.5 by @​thomhurst in
thomhurst/TUnit#5147
* chore(deps): update dependency testcontainers.kafka to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5149
* chore(deps): update microsoft.extensions to 10.0.5 by @​thomhurst in
thomhurst/TUnit#5148
* chore(deps): update dependency testcontainers.postgresql to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5150
* chore(deps): update dependency testcontainers.redis to 4.11.0 by
@​thomhurst in thomhurst/TUnit#5151
* chore(deps): update dependency stackexchange.redis to 2.12.1 by
@​thomhurst in thomhurst/TUnit#5153


**Full Changelog**:
thomhurst/TUnit@v1.19.22...v1.19.57

## 1.19.22

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.22 -->

## What's Changed
### Other Changes
* Remove redundant prefixes from tracing timeline spans by @​thomhurst
in thomhurst/TUnit#5111
* Render span tags and events on separate lines by @​thomhurst in
thomhurst/TUnit#5113
* Demote granular internal logs from Debug to Trace level by @​thomhurst
in thomhurst/TUnit#5116
### Dependencies
* chore(deps): update tunit to 1.19.16 by @​thomhurst in
thomhurst/TUnit#5109


**Full Changelog**:
thomhurst/TUnit@v1.19.16...v1.19.22

## 1.19.16

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.16 -->

## What's Changed
### Other Changes
* Truncate exceptions in GitHub summary tables by @​thomhurst in
thomhurst/TUnit#5108
### Dependencies
* chore(deps): update tunit to 1.19.11 by @​thomhurst in
thomhurst/TUnit#5106
* chore(deps): bump dompurify from 3.3.0 to 3.3.2 in /docs by
@​dependabot[bot] in thomhurst/TUnit#5096
* chore(deps): bump svgo from 3.2.0 to 3.3.3 in /docs by
@​dependabot[bot] in thomhurst/TUnit#5086


**Full Changelog**:
thomhurst/TUnit@v1.19.11...v1.19.16

## 1.19.11

<!-- Release notes generated using configuration in .github/release.yml
at v1.19.11 -->

## What's Changed
### Other Changes
* Fix HTML report sort to also reorder groups by @​thomhurst in
thomhurst/TUnit#5103
* fix: correct expand-all icon SVG in HTML report by @​slang25 in
thomhurst/TUnit#5105
* Avoid some redundant list allocations by @​SimonCropp in
thomhurst/TUnit#4963
* avoid some redundant enumerable alloc by @​SimonCropp in
thomhurst/TUnit#4972
* use char instead of string for joins by @​SimonCropp in
thomhurst/TUnit#4989
### Dependencies
* chore(deps): update dependency nunit to 4.5.1 by @​thomhurst in
thomhurst/TUnit#5097
* chore(deps): update tunit to 1.19.0 by @​thomhurst in
thomhurst/TUnit#5099
* chore(deps): update dependency humanizer to 3.0.10 by @​thomhurst in
thomhurst/TUnit#5101
* chore(deps): update dependency nunit.analyzers to 4.12.0 by
@​thomhurst in thomhurst/TUnit#5102


**Full Changelog**:
thomhurst/TUnit@v1.19.0...v1.19.11

Commits viewable in [compare
view](thomhurst/TUnit@v1.19.0...v1.19.74).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=TUnit&package-manager=nuget&previous-version=1.19.0&new-version=1.19.74)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore <dependency name> major version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's major version (unless you unignore this specific
dependency's major version or upgrade to it yourself)
- `@dependabot ignore <dependency name> minor version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's minor version (unless you unignore this specific
dependency's minor version or upgrade to it yourself)
- `@dependabot ignore <dependency name>` will close this group update PR
and stop Dependabot creating any more for the specific dependency
(unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore <dependency name>` will remove all of the ignore
conditions of the specified dependency
- `@dependabot unignore <dependency name> <ignore condition>` will
remove the ignore condition of the specified dependency and ignore
conditions


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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