Closed zhhyu closed 2 years ago
@lyndaidaii , what do you think?
@lyndaidaii , what do you think?
I think we could evaluate how much effort we want to fix this, if require a lot engineering time. I don't feel strongly to fix this in manage package page since readme is mutable. few people notice on manage package page. if this is simple fix, I think we could do it.
Popular C# projects on GitHub very often use tables in their READMEs. Here is a list of the top 10 most popular affected projects:
GitHub project | NuGet README preview | Stars |
---|---|---|
microsoft/PowerToys | Preview | 57582 |
PowerShell/PowerShell | Preview | 27894 |
dotnet-architecture/eShopOnContainers | Preview | 18232 |
dotnet/roslyn | Preview | 14697 |
DapperLib/Dapper | Preview | 13946 |
Unity-Technologies/ml-agents | Preview | 11411 |
dotnet/maui | Preview | 10751 |
aspnetboilerplate/aspnetboilerplate | Preview | 9716 |
App-vNext/Polly | Preview | 9444 |
lionsoul2014/ip2region | Preview | 9435 |
Markdig has a Bootstrap extension that adds the .table
class to table elements. We should be able to add this extension to Markdig's pipeline here:
This also affects block quotes (like > Quote
) and images, so we should verify these aren't broken by this change.
Repro:
Test package: https://int.nugettest.org/packages/license-expression-test1
Add this markdown table into a readme file:
The manage package page shows:
The package detail page shows:
Could the rendering be better, and share the same appearance between two pages? Tweaking Bootstrap, CSS may be helpful.