Skip to content

Conversation

@maihcx
Copy link
Contributor

@maihcx maihcx commented Oct 1, 2025

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

Issue Number: #1538, #1546

What is the new behavior?

Fix CardExpander has inconsistent CornerRadius when opened and closed

maihcx added 2 commits October 1, 2025 13:33
To split the corner radius
Use CornerRadiusSplitConverter for CardExpander
@maihcx maihcx requested a review from pomianowski as a code owner October 1, 2025 06:48
@github-actions github-actions bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request dotnet release labels Oct 1, 2025
maihcx added 2 commits October 3, 2025 08:51
Eliminate redundant declarations
- Added class SmoothScrollBehavior.cs
- Apply SmoothScrollBehavior for DynamicScrollViewer.xaml
@maihcx maihcx changed the title Fix CardExpander has inconsistent CornerRadius Fix CardExpander has inconsistent CornerRadius and Smooth Scrolling for Page Oct 17, 2025
@maihcx
Copy link
Contributor Author

maihcx commented Oct 22, 2025

@Nuklon also check about this

Removed CornerRadius template binding from Border.
Removed corner radius from ContentPresenterBorder and ToggleButtonBorder.
@pomianowski pomianowski changed the title Fix CardExpander has inconsistent CornerRadius and Smooth Scrolling for Page fix(controls): CardExpander has inconsistent CornerRadius and Smooth Scrolling for Page Nov 7, 2025
@pomianowski pomianowski changed the title fix(controls): CardExpander has inconsistent CornerRadius and Smooth Scrolling for Page feat(controls): Add Smooth Scrolling for Page and fix CardExpander has inconsistent CornerRadius Nov 7, 2025
pomianowski and others added 3 commits November 7, 2025 13:24
Updated the CardExpander.xaml file by reordering and modifying namespaces, adjusting control templates, and refining trigger actions for better functionality and maintainability.
@Nuklon
Copy link
Collaborator

Nuklon commented Nov 11, 2025

Please split this up in two PRs. I'm not sure if we should enable smooth scrolling by default.

@maihcx
Copy link
Contributor Author

maihcx commented Nov 13, 2025

I will split it into 2 PR

@maihcx maihcx closed this Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

controls Changes to the appearance or logic of custom controls. dotnet PR Pull request release styles Topic is related to styles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants