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
This is another issue that may be more of a topic for discussion than a bug. It involves some unanticipated consequences to how tables are formatted.
When large tables are made, the large padded tables are ungainly for two reasons:
A single large cell (for example, a cell with a link or two in it) spreads the table so wide the padding is useless
A single-line change can have non-local changes on every other line
The second issue is the worst, since it effectively means two simultaneous changes to any table will always conflict in GitHub (even if it's to different rows).
In general, to avoid merge conflicts, it'd be great if markdownfmt has only "local" changes, i.e. changes to one line don't affect many other lines in the doc.
Suggested fix: No longer pad table cells with whitespace. Alternately never pad tables if the width will be more than a fixed size, like 80 or 120 chars.
The text was updated successfully, but these errors were encountered:
Your observations make sense. There's probably some room for improvement. The current algorithm is very simple and consistent. It'll take some thinking to come up with something better.
This is another issue that may be more of a topic for discussion than a bug. It involves some unanticipated consequences to how tables are formatted.
When large tables are made, the large padded tables are ungainly for two reasons:
The second issue is the worst, since it effectively means two simultaneous changes to any table will always conflict in GitHub (even if it's to different rows).
In general, to avoid merge conflicts, it'd be great if markdownfmt has only "local" changes, i.e. changes to one line don't affect many other lines in the doc.
Suggested fix: No longer pad table cells with whitespace. Alternately never pad tables if the width will be more than a fixed size, like 80 or 120 chars.
The text was updated successfully, but these errors were encountered: