Skip to content

Remove fillRegion() and Region.subtract block in CTabFolderRenderer#3218

Merged
HeikoKlare merged 1 commit intomasterfrom
remove-fillRegion-subtract
Apr 10, 2026
Merged

Remove fillRegion() and Region.subtract block in CTabFolderRenderer#3218
HeikoKlare merged 1 commit intomasterfrom
remove-fillRegion-subtract

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 10, 2026

With square tab corners, the Region.subtract block in drawTabArea that painted parent background into curved-corner cutouts was always empty. Remove it along with the now-unused fillRegion() helper.

With square tab corners, the Region.subtract block in drawTabArea that
painted parent background into curved-corner cutouts was always empty.
Remove it along with the now-unused fillRegion() helper.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

  170 files  ±0    170 suites  ±0   25m 3s ⏱️ - 1m 4s
4 679 tests ±0  4 658 ✅ ±0   21 💤 ±0  0 ❌ ±0 
6 592 runs  ±0  6 437 ✅ ±0  155 💤 ±0  0 ❌ ±0 

Results for commit 1d72629. ± Comparison against base commit c83ed3c.

@HeikoKlare HeikoKlare merged commit 16e501f into master Apr 10, 2026
25 checks passed
@HeikoKlare HeikoKlare deleted the remove-fillRegion-subtract branch April 10, 2026 11:10
@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 10, 2026

Fast you are Jedi master @HeikoKlare, I was still testing...

But thanks for the merge, up to the next split out...

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