Skip to content

ToggleGroup: Improvements#7975

Merged
henon merged 9 commits intoMudBlazor:devfrom
mckaragoz:ToggleGroupImprovements
Dec 30, 2023
Merged

ToggleGroup: Improvements#7975
henon merged 9 commits intoMudBlazor:devfrom
mckaragoz:ToggleGroupImprovements

Conversation

@mckaragoz
Copy link
Member

@mckaragoz mckaragoz commented Dec 30, 2023

Description

Continuation of PRs #7309 and #7948

Todo:

  • fix the bug with rounded + outline:
    image

  • make the icon 4px smaller when ShowText is true

  • ShowText && ShowIcon && Icon!=null || SelectedIcon!=null: balance the space taken up by the icon on the left with padding on the right so that the text doesn't shift right when the selected icon shows up:
    image
    image

  • Outline: make sure the outline's corner radius is a tiny bit smaller than the toggle item's so that there is no visible space between them:
    image

How Has This Been Tested?

Types 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)

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 bug Unexpected behavior or functionality not working as intended PR: needs review labels Dec 30, 2023
@codecov
Copy link

codecov bot commented Dec 30, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (db7057f) 87.41% compared to head (8747ac2) 87.40%.

Files Patch % Lines
...MudBlazor/Components/Toggle/MudToggleItem.razor.cs 57.14% 0 Missing and 3 partials ⚠️
...rc/MudBlazor/Components/Toggle/MudToggleItem.razor 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #7975      +/-   ##
==========================================
- Coverage   87.41%   87.40%   -0.01%     
==========================================
  Files         395      395              
  Lines       11745    11745              
  Branches     2373     2373              
==========================================
- Hits        10267    10266       -1     
+ Misses        950      949       -1     
- Partials      528      530       +2     

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

@henon
Copy link
Contributor

henon commented Dec 30, 2023

The text is centered now and doesn't jump around when the checkmark is added
image

@henon
Copy link
Contributor

henon commented Dec 30, 2023

Readded the CheckMark bool param, so now we can have checkmark + icon content via render fragment
image

@henon
Copy link
Contributor

henon commented Dec 30, 2023

  • Outline: make sure the outline's corner radius is a tiny bit smaller than the toggle item's so that there is no visible space between them:
    image

We decided to fix this little render bug later. It is an experimental component after all

@henon henon merged commit 16cb82f into MudBlazor:dev Dec 30, 2023
biegehydra pushed a commit to biegehydra/MudBlazor that referenced this pull request Apr 26, 2024
* Fix Rounded
* Fix None Outline Borders

---------

Co-authored-by: Meinrad Recheis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Unexpected behavior or functionality not working as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants