Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Nov 30, 2023

Follow up to #5054 to make tests more stable by hiding the cursor that may be more or less transparent depending on the activity state.

Also it is not much useful in print view.

@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Nov 30, 2023
@mejo-
Copy link
Member

mejo- commented Nov 30, 2023

Thanks for the follow-up. Let me push two more commits in the hope they help:

  • Set caret-color to transparent before doing the snapshot
  • One more time update the screenshots

@mejo-
Copy link
Member

mejo- commented Nov 30, 2023

But looking at the screenshot diffs it rather seems like the whole rendering is not pixel-perfect and moves a few pixels up/down left/right between runs 🤔

@mejo- mejo- merged commit 72f1ff7 into main Nov 30, 2023
@mejo- mejo- deleted the bugfix/noid/hide-cursor-print branch November 30, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants