Fix SegmentBackground not being drawn when StrokeThickness is set to 0

[Copilot]

Problem

When StrokeThickness is set to 0 on SfSegmentedControl, the SegmentBackground color disappears and becomes virtually transparent, even when explicitly set to a color.

<segmentedControl:SfSegmentedControl SegmentBackground="Red" StrokeThickness="0">
    <segmentedControl:SfSegmentedControl.ItemsSource>
        <x:Array Type="{x:Type x:String}">
            <x:String>Day</x:String>
            <x:String>Week</x:String>
            <x:String>Month</x:String>
            <x:String>Year</x:String>
        </x:Array>
    </segmentedControl:SfSegmentedControl.ItemsSource>
</segmentedControl:SfSegmentedControl>

Expected: Red background with no border stroke
Actual: Transparent background

Root Cause

In SegmentItemView.DrawRoundedRectangle(), the method had an early return condition that prevented drawing when StrokeThickness == 0:

if (itemInfo?.SegmentTemplate != null || itemInfo == null || itemInfo.StrokeThickness == 0)
{
    return; // This prevented background from being drawn!
}

Solution

Modified the drawing logic to:

1. Always draw the background when SegmentBackground is set
2. Only draw stroke when StrokeThickness > 0
3. Handle corner radius correctly for both scenarios

Key Changes

• Removed StrokeThickness == 0 from early return condition
• Separated background and stroke drawing logic
• Added conditional stroke drawing based on thickness value
• Updated corner radius calculations to handle zero stroke thickness

Before/After

Before:

// Early return prevented any drawing
if (itemInfo.StrokeThickness == 0) return;

// Background and stroke drawn together
canvas.FillRoundedRectangle(...);  // Never reached when thickness = 0

After:

// Always draw background
canvas.FillRoundedRectangle(...);

// Only draw stroke when needed
if (itemInfo.StrokeThickness > 0)
{
    canvas.DrawRoundedRectangle(...);
}

Testing

Added comprehensive test cases to verify:

• SegmentBackground property is preserved when StrokeThickness = 0
• GetSegmentBackground() helper returns correct values regardless of stroke thickness
• No regression in existing functionality

Fixes #191.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[PaulAndersonS]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).