Skip to content

feat(IDocTags): add content wrapping for handling whitespace#489

Merged
PeterStaar-IBM merged 2 commits intomainfrom
add-content-wrapper
Jan 23, 2026
Merged

feat(IDocTags): add content wrapping for handling whitespace#489
PeterStaar-IBM merged 2 commits intomainfrom
add-content-wrapper

Conversation

@vagenas
Copy link
Member

@vagenas vagenas commented Jan 23, 2026

Also:

  • fix caption serialiazation
  • consolidate indendation via pretty_indentation param

Also:
- fix caption serialiazation
- consolidate indendation via `pretty_indentation` param

Signed-off-by: Panos Vagenas <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2026

DCO Check Passed

Thanks @vagenas, all your commits are properly signed off. 🎉

@dosubot
Copy link

dosubot bot commented Jan 23, 2026

Related Documentation

Checked 7 published document(s) in 0 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@mergify
Copy link

mergify bot commented Jan 23, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

🟢 Require two reviewer for test updates

Wonderful, this rule succeeded.

When test data is updated, we require two reviewers

  • #approved-reviews-by >= 2

Signed-off-by: Panos Vagenas <[email protected]>
Copy link
Member

@PeterStaar-IBM PeterStaar-IBM left a comment

Choose a reason for hiding this comment

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

awesome!

@codecov
Copy link

codecov bot commented Jan 23, 2026

Codecov Report

❌ Patch coverage is 91.42857% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
docling_core/experimental/idoctags.py 91.42% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Member

@Matteo-Omenetti Matteo-Omenetti left a comment

Choose a reason for hiding this comment

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

all good

@PeterStaar-IBM PeterStaar-IBM merged commit fdcdfd1 into main Jan 23, 2026
13 checks passed
@PeterStaar-IBM PeterStaar-IBM deleted the add-content-wrapper branch January 23, 2026 14:20
@vagenas vagenas mentioned this pull request Jan 23, 2026
34 tasks
PeterStaar-IBM pushed a commit that referenced this pull request Jan 26, 2026
* feat(IDocTags): add content wrapping for handling whitespace

Also:
- fix caption serialiazation
- consolidate indendation via `pretty_indentation` param

Signed-off-by: Panos Vagenas <[email protected]>

* fix content tag value

Signed-off-by: Panos Vagenas <[email protected]>

---------

Signed-off-by: Panos Vagenas <[email protected]>
Signed-off-by: Peter Staar <[email protected]>
PeterStaar-IBM added a commit that referenced this pull request Jan 26, 2026
…ng_boxes (#490)

* feat(IDocTags): add content wrapping for handling whitespace (#489)

* feat(IDocTags): add content wrapping for handling whitespace

Also:
- fix caption serialiazation
- consolidate indendation via `pretty_indentation` param

Signed-off-by: Panos Vagenas <[email protected]>

* fix content tag value

Signed-off-by: Panos Vagenas <[email protected]>

---------

Signed-off-by: Panos Vagenas <[email protected]>
Signed-off-by: Peter Staar <[email protected]>

* feat: add `minimal` parameter to get_row_bounding_boxes and get_column_bounding_boxes

Add option to return uniform bounding boxes instead of minimal ones:
- When minimal=False, rows have uniform horizontal extent (same x0/x1)
- When minimal=False, columns have uniform vertical extent (same y0/y1)

Also propagate the parameter to TableVisualizer via new params:
- minimal_row_bboxes
- minimal_col_bboxes

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Signed-off-by: Peter Staar <[email protected]>

* updated the color of the row-section

Signed-off-by: Peter Staar <[email protected]>

---------

Signed-off-by: Panos Vagenas <[email protected]>
Signed-off-by: Peter Staar <[email protected]>
Co-authored-by: Panos Vagenas <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>
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.

3 participants