You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Table headers are measured separately from table body content. The exact reason is unknown, yet my PR seems to have triggered this error.
Originally spotted during my PR development - ignored due to buggy dev environment. Yet, it is now confirmed to be the problem thanks to @Lourens-Rich.
Possible solution
If you remove display: block from table CSS, that is _markdown.less#L243, the problem goes away. But it makes table 100% wide in parent container. The solution to that is width: auto instead of width: 100% on the next line, that is also the way it is done in Semantic UI (collapsing class on table).
If maintainers confirm that is the way to go, I can create PR with this fix.
Description
Table headers are measured separately from table body content. The exact reason is unknown, yet my PR seems to have triggered this error.
Originally spotted during my PR development - ignored due to buggy dev environment. Yet, it is now confirmed to be the problem thanks to @Lourens-Rich.
Possible solution
If you remove
display: block
from table CSS, that is _markdown.less#L243, the problem goes away. But it makes table 100% wide in parent container. The solution to that iswidth: auto
instead ofwidth: 100%
on the next line, that is also the way it is done in Semantic UI (collapsing
class on table).If maintainers confirm that is the way to go, I can create PR with this fix.
Screenshots
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: