Skip to content

Folding Improvements - #5534

Merged
greenbonebot merged 10 commits into
mainfrom
folding-improvements
Aug 18, 2026
Merged

Folding Improvements#5534
greenbonebot merged 10 commits into
mainfrom
folding-improvements

Conversation

@bjoernricks

Copy link
Copy Markdown
Contributor

What

Folding Improvements

  • Separate code into own modules
  • Convert withFoldToggle HOC into a useFoldToggle hook
  • Add tests

Why

  • Separation of concerns
  • Ensure the behavior of the code
  • Hooks are easier to understand and to use then HOCs

Checklist

  • Tests

@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA f80aec4.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.65079% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.09%. Comparing base (cdfea48) to head (f80aec4).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
src/web/components/folding/useFoldToggle.ts 89.74% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5534      +/-   ##
==========================================
+ Coverage   83.99%   84.09%   +0.10%     
==========================================
  Files        1290     1293       +3     
  Lines       33760    33766       +6     
  Branches    10089    10056      -33     
==========================================
+ Hits        28356    28395      +39     
+ Misses       4853     4821      -32     
+ Partials      551      550       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Ensure the behavior of the HOC.
Convert the wrapper component inside the HOC from a class component to a
function component. Function components are easier to understand and to
maintain.
@bjoernricks
bjoernricks force-pushed the folding-improvements branch from 12f5600 to 2ec1380 Compare August 18, 2026 11:15
The HOC got replaced with the useFoldToggle hook.
Keep things separated. With using an own module the HOC code will be
easier to find.
With the refactoring of the Section component the types of the props
passed to the Section are fixed now.
@bjoernricks
bjoernricks force-pushed the folding-improvements branch from 2ec1380 to d8127ef Compare August 18, 2026 11:20
@bjoernricks
bjoernricks marked this pull request as ready for review August 18, 2026 11:38
@bjoernricks
bjoernricks requested a review from a team as a code owner August 18, 2026 11:38
@greenbonebot
greenbonebot enabled auto-merge (rebase) August 18, 2026 11:38
@bjoernricks
bjoernricks force-pushed the folding-improvements branch from 64cdf37 to 5e6396d Compare August 18, 2026 12:19
@bjoernricks
bjoernricks force-pushed the folding-improvements branch from 5e6396d to 93df629 Compare August 18, 2026 12:21
@greenbonebot
greenbonebot merged commit d836486 into main Aug 18, 2026
23 checks passed
@greenbonebot
greenbonebot deleted the folding-improvements branch August 18, 2026 12:39
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.

3 participants