Update markdown.py

[Rhythmicc]

Type of changes

• Bug fix
• New feature
• Documentation / docstrings
• Tests
• Other

Checklist

• I've run the latest black with default args on new code.
• I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
• I've added tests for new code.
• I accept that @willmcgugan may be pedantic in the code review.

Description

Fix the extra blank lines that appear when displaying tables:

After modified:

[willmcgugan]

I can't reproduce this. Assuming fixed.

[TomJGooding]

@willmcgugan Here's a quick test which hopefully demonstrates this isn't fixed on master:

def test_table_with_empty_cells():
    """Test a table with empty cells is rendered without extra newlines above.
    Regression test for #3027 https://github.com/Textualize/rich/issues/3027
    """
    complete_table = Markdown(
        """\
| First Header  | Second Header |
| ------------- | ------------- |
| Content Cell  | Content Cell  |
| Content Cell  | Content Cell  |
"""
    )
    table_with_empty_cells = Markdown(
        """\
| First Header  |               |
| ------------- | ------------- |
| Content Cell  | Content Cell  |
|               | Content Cell  |
"""
    )
    result = len(render(table_with_empty_cells).splitlines())
    expected = len(render(complete_table).splitlines())
    assert result == expected

[willmcgugan]

So it is empty cells that is the issue? Could have used that in description!

[TomJGooding]

So it is empty cells that is the issue? Could have used that in description!

Sorry this isn't my PR and it has been several months. I agree this could have been clearer, this is the relevant issue:

• [BUG] Extra Space above Certain Markdown Tables #3027

[willmcgugan]

Thanks. Fixed in main.