Skip to content

Try another solution to flickering on macOS#3388

Closed
vadi2 wants to merge 1 commit intorelease-4.6from
alternative-macos-flicker-fix
Closed

Try another solution to flickering on macOS#3388
vadi2 wants to merge 1 commit intorelease-4.6from
alternative-macos-flicker-fix

Conversation

@vadi2
Copy link
Copy Markdown
Member

@vadi2 vadi2 commented Mar 8, 2020

Brief overview of PR changes/additions

Try another solution to flickering on macOS

Motivation for adding to Mudlet

In an effort to find a good solution quicker, try an alternative to #3387

Other info (issues closed, discussion etc)

See https://forum.qt.io/topic/84262/mac-os-repainting-parented-widget-performance-seem-to-depend-on-ancestors-chain/5

@add-deployment-links
Copy link
Copy Markdown

add-deployment-links bot commented Mar 8, 2020

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

@vadi2
Copy link
Copy Markdown
Member Author

vadi2 commented Mar 8, 2020

This solution seems to work too, but it still has the strange text clipping as the other one does: https://streamable.com/ay390

@Edru2
Copy link
Copy Markdown
Member

Edru2 commented Mar 8, 2020

I cannot reproduce the text clipping issue.

The only issue with this PR (compared to #3387) what I see is that sending an empty command (just pressing enter) still causes flickering.

@vadi2
Copy link
Copy Markdown
Member Author

vadi2 commented Mar 8, 2020

Yep, good find. Let's go with your solution then! Was good to try out an alternative and compare - I think this version would have caused less of the screen to redraw if it worked.

@vadi2 vadi2 closed this Mar 8, 2020
@vadi2 vadi2 deleted the alternative-macos-flicker-fix branch March 8, 2020 13:42
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