Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Let productions users table reflow better #3376

Merged
merged 1 commit into from
Nov 27, 2020
Merged

Conversation

squaremo
Copy link
Member

It's a pain to add to the production users table, as marked down,
since it involves reflowing all the subsequent entries.

Instead, divide the entries into three groups with roughly the same
number of entries, and use HTML markup to put them into three columns,
with line breaks instead of rows. This means adding an entry is
easier; if the columns need to be rebalanced, you can just move the
cell divider </td><td>.

(It doesn't look quite as nice, since there's no alternating row background; re-introducing that would probably mean going back to manually reflowing everything ..)

@squaremo squaremo requested a review from hiddeco November 27, 2020 13:36
Copy link
Member

@dholbach dholbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done! ♥

@squaremo
Copy link
Member Author

squaremo commented Nov 27, 2020

Looks a bit better as lists within the cells:

EDIT: github-flavored markdown strips out style attributes, so use the deprecated but still respected valign attribute.

It's a pain to add to the production users table, as marked _down_,
since it involves reflowing all the subsequent entries.

Instead, divide the entries into three groups with roughly the same
number of entries, and use HTML markup to put them into three columns,
with line breaks instead of rows. This means adding an entry is
easier; if the columns need to be rebalanced, you can just move the
cell divider `</td><td>`.

Signed-off-by: Michael Bridgen <[email protected]>
@squaremo squaremo merged commit 85ce925 into master Nov 27, 2020
@squaremo squaremo deleted the reflowable-table branch November 27, 2020 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants