Skip to content

DEV-26894 Add height to table row properties#76

Merged
harvestcore merged 2 commits into
developfrom
DEV-26894
May 21, 2026
Merged

DEV-26894 Add height to table row properties#76
harvestcore merged 2 commits into
developfrom
DEV-26894

Conversation

@ins426

@ins426 ins426 commented May 18, 2026

Copy link
Copy Markdown

No description provided.

@ins426 ins426 marked this pull request as ready for review May 19, 2026 06:04
* - `value`: the height of the row.
* - `rule`: how the height is applied:
* - `'auto'`: the row height is determined by the content.
* - `'atLeast'` (default when omitted): the row height is at least the specified value.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The documentation says that 'auto' is the default option when omitted, no?
https://c-rex.net/samples/ooxml/e1/Part4/OOXML_P4_DOCX_trHeight_topic_ID0E5G4R.html?hl=trheight

Image

*/
height?: null | {
value: Length;
rule?: null | 'auto' | 'atLeast' | 'exact';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If the default is “auto,” I would never set it to “null.”

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Discussed with the team via chat and "null" is fine.

@luiisgallego luiisgallego left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please update the 'rule' (default value) documentation before merging.

@harvestcore harvestcore merged commit 3799e82 into develop May 21, 2026
2 checks passed
@harvestcore harvestcore deleted the DEV-26894 branch May 21, 2026 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants