Skip to content

Commit

Permalink
Merge pull request #5063 from nextcloud/bugfix/noid/hide-cursor-print
Browse files Browse the repository at this point in the history
fix: Hide cursor in print output
  • Loading branch information
mejo- authored Nov 30, 2023
2 parents ae92cf9 + 4c13b38 commit 72f1ff7
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cypress/e2e/print.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ describe('Open print.md and compare print view', function() {

cy.getEditor().should('be.visible')
cy.getContent()
// Ensure cursor is not displayed to prevent flaky tests (flashing input cursor)
.invoke('css', 'caret-color', 'transparent')
.get('h1:not(.hidden-visually)').should('contain', 'Print test')
.should('be.visible')

Expand All @@ -56,6 +58,8 @@ describe('Open print.md and compare print view', function() {
.then(() => {
cy.getEditor().should('be.visible')
cy.getContent()
// Ensure cursor is not displayed to prevent flaky tests (flashing input cursor)
.invoke('css', 'caret-color', 'transparent')
.get('h1:not(.hidden-visually)').should('contain', 'Print test')
.should('be.visible')

Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified cypress/snapshots/base/print.spec.js/print view in viewer.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
5 changes: 5 additions & 0 deletions src/css/print.scss
Original file line number Diff line number Diff line change
Expand Up @@ -127,4 +127,9 @@
display: none;
}
}

.collaboration-cursor__caret,
.collaboration-cursor__label {
display: none;
}
}

0 comments on commit 72f1ff7

Please sign in to comment.