feat: implemented print shortcut for performance report page #51

Merged

addison merged 1 commit from 005-002-004-performance-report-print-shortcut into main 2026-08-29 05:04:57 -04:00

Conversation 2 Commits 1 Files changed 5 +30 -2

talha commented 2026-08-27 12:30:13 -04:00

Collaborator

Copy link

Spec 5.2.4

Changes

• Added support of title (browser's native tooltip) to the button.
• Added hook to override ctrl + p and cmd + p shortcuts to replace them with apps print call.

Testing

• Make sure you have latest code running.
• Navigate to Performance report page and hover over print button, it must show browser's native tooltip saying "Ctril + P".
• Now try apply different filters and pressing ctrl + p, print screen must be opened in new tab with filters applied.

# Spec 5.2.4 ## Changes - Added support of title (browser's native tooltip) to the button. - Added hook to override ctrl + p and cmd + p shortcuts to replace them with apps print call. ## Testing - [x] Make sure you have latest code running. - [x] Navigate to Performance report page and hover over print button, it must show browser's native tooltip saying "Ctril + P". - [x] Now try apply different filters and pressing ctrl + p, print screen must be opened in new tab with filters applied.

talha added 1 commit 2026-08-27 12:30:13 -04:00

feat: implemented print shortcut for performance report page d0c085afe1

talha requested review from addison 2026-08-27 12:36:27 -04:00

addison merged commit 8588d03c2d into main 2026-08-29 05:04:57 -04:00

addison deleted branch 005-002-004-performance-report-print-shortcut 2026-08-29 05:04:57 -04:00

addison referenced this pull request from a commit 2026-08-29 05:04:58 -04:00

feat: implemented print shortcut for performance report page (#51)

addison commented 2026-08-29 05:05:16 -04:00

Owner

Copy link

Works great! Could you also apply this shortcut to the job commissions and sales reports pages?

Works great! Could you also apply this shortcut to the job commissions and sales reports pages?

talha commented 2026-08-29 05:07:35 -04:00

Author

Collaborator

Copy link

@addison wrote in #51 (comment):

Works great! Could you also apply this shortcut to the job commissions and sales reports pages?

Yes sure

@addison wrote in https://git.kwila.cloud/kwila/office-extensions/pulls/51#issuecomment-4508: > Works great! Could you also apply this shortcut to the job commissions and sales reports pages? Yes sure

Sign in to join this conversation.

Reviewers

No reviewers

addison

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

kwila/office-extensions!51

No description provided.