Skip to content

Add footer formatter to print example#578

Merged
wenzhixin merged 1 commit intodevelopfrom
feature/add-print-footer
May 3, 2026
Merged

Add footer formatter to print example#578
wenzhixin merged 1 commit intodevelopfrom
feature/add-print-footer

Conversation

@wenzhixin
Copy link
Copy Markdown
Owner

Summary

  • Add data-show-footer and data-footer-formatter to the print extension example to display a price total in the footer

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the print extension example to demonstrate footer support by enabling the table footer and adding a footer formatter for the price column. It fits into the examples site by showing how the print plugin can render summarized footer data alongside printed table content.

Changes:

  • Enable the footer in extensions/print.html with data-show-footer="true".
  • Add a data-footer-formatter to the price column.
  • Introduce a priceFormatter function intended to compute and display the total price in the footer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread extensions/print.html Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wenzhixin wenzhixin force-pushed the feature/add-print-footer branch from bbd9e2e to fdeab99 Compare May 3, 2026 03:26
@wenzhixin wenzhixin merged commit 8bae62f into develop May 3, 2026
1 check passed
@wenzhixin wenzhixin deleted the feature/add-print-footer branch May 3, 2026 03:32
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.

2 participants