Skip to content

Commit

Permalink
Merge pull request #148 from DFE-Digital/fix/print-page-csp
Browse files Browse the repository at this point in the history
fix(print): Move Javascript to <script> element due to CSP
  • Loading branch information
jimwashbrook authored Aug 28, 2024
2 parents 168db3d + e08f936 commit 132bd08
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 2 deletions.
27 changes: 26 additions & 1 deletion src/Dfe.ContentSupport.Web/Views/Content/CsIndex.cshtml
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,29 @@
<partial name="_Feedback" />
}
</div>
</div>
</div>

@section BodyEnd{
@if (Model.HasPrint)
{
<script nonce="@Context.Items["nonce"]" defer>
/**
* Adds functionality for printing a page to the Print Page button (_Print.cshtml)
*/
const printPage = () => window.print();
const addPrintButtonEventListener = () => {
const printButton = document.getElementById("print-link");
if(!printButton){
return;
}
printButton.addEventListener('click', printPage);
}
addPrintButtonEventListener();
</script>
}
}
2 changes: 1 addition & 1 deletion src/Dfe.ContentSupport.Web/Views/Shared/_Print.cshtml
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
<div class="print-button">
<button class="govuk-link print-link-button" data-module="print-link" onclick="window.print()">Print this page</button>
<button class="govuk-link print-link-button" data-module="print-link" id="print-link">Print this page</button>
</div>

0 comments on commit 132bd08

Please sign in to comment.