Conversation
Those are all of the names I can think of. All of them would work. Not sure what is consistent with anything similar. |
|
Hi. This shouldn't be in |
|
Could literally call it |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #9478 +/- ##
==========================================
+ Coverage 89.82% 90.55% +0.72%
==========================================
Files 412 406 -6
Lines 11878 12751 +873
Branches 2364 2472 +108
==========================================
+ Hits 10670 11547 +877
+ Misses 681 642 -39
- Partials 527 562 +35 ☔ View full report in Codecov by Sentry. |
|
I would prefer |
Can this be merged? It's currently named as |
|
I didn't make myself totally clear it seams. Currently it would be
|
No it wouldn't. It would be
Please, no. I want to get get away from partial classes in next major releases and use namespaces instead (I explained the reason why partial makes it hard to extend the icons via additional packages, there will be a conflict) and this will only interfere it. Well I want to remove svg in general, and I won't be able to support |
This reverts commit 04bf541.
|
OK |
@ScarletKuro I am sure we can keep empty as an svg icon. We'll have to keep the possibility to use svg icons anyway, so I don't see a problem with extending the font icons with a couple svg icons. |

Description
Add void icon to act as empty space, see screenshot below.
Wondering if Void is the right name to use, maybe Transparent is better?
Also, this icon is only added in the Filled class, does it need to be in the other variants as well?
Use like this:
Other example & usecase:

How Has This Been Tested?
Tested locally with the mudblazor repo
Type of Changes
Checklist
dev).