Skip to content

MudPicker: Add OverflowBehavior that is passed to the underlying MudPopover#9372

Merged
henon merged 1 commit intoMudBlazor:devfrom
benm-eras:dev
Jul 13, 2024
Merged

MudPicker: Add OverflowBehavior that is passed to the underlying MudPopover#9372
henon merged 1 commit intoMudBlazor:devfrom
benm-eras:dev

Conversation

@benm-eras
Copy link
Contributor

Added a single [Parameter] property to the MudPicker component in MudPicker.razor.cs that defaults to OverflowBehavior.FlipOnOpen and is simply passed to the underlying MudPopover component in MudPicker.razor where it was previously just always passing OverflowBehavior.FlipOnOpen.

Description

resolves #9371

How Has This Been Tested?

Tested visually by fiddling with the values passed to the examples on the locally hosted docs site. All existing unit tests pass, and the changes are very minimal and make use of existing behaviour in MudPopover so I don't think additional unit tests are necessary.

Type of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (fix or improvement to the website or code docs)

Checklist

  • The PR is submitted to the correct branch (dev).
  • My code follows the code style of this project.
  • I've added relevant tests.

@github-actions github-actions bot added enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library PR: needs review labels Jul 11, 2024
@codecov
Copy link

codecov bot commented Jul 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.61%. Comparing base (28bc599) to head (684867a).
Report is 335 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #9372      +/-   ##
==========================================
+ Coverage   89.82%   90.61%   +0.78%     
==========================================
  Files         412      403       -9     
  Lines       11878    12668     +790     
  Branches     2364     2448      +84     
==========================================
+ Hits        10670    11479     +809     
+ Misses        681      634      -47     
- Partials      527      555      +28     

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

@ScarletKuro ScarletKuro requested a review from henon July 13, 2024 12:20
@henon henon changed the title exposed OverflowBehavior property on MudPicker components that is passed to the underlying MudPopover MudPicker: Add OverflowBehavior parameter that is passed to the underlying MudPopover Jul 13, 2024
@henon henon changed the title MudPicker: Add OverflowBehavior parameter that is passed to the underlying MudPopover MudPicker: Add OverflowBehavior that is passed to the underlying MudPopover Jul 13, 2024
@henon henon merged commit 37a7378 into MudBlazor:dev Jul 13, 2024
@henon
Copy link
Contributor

henon commented Jul 13, 2024

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

All the picker components should have an OverflowBehavior that is passed down to the underlying MudPopover

3 participants