Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unreliable minimap redraw #99

Open
mchakravarty opened this issue Feb 12, 2024 · 0 comments
Open

Unreliable minimap redraw #99

mchakravarty opened this issue Feb 12, 2024 · 0 comments
Labels
architecture Requires changes that affect the general architecture bug Something isn't working

Comments

@mchakravarty
Copy link
Owner

mchakravarty commented Feb 12, 2024

Getting the minimap to redraw properly in case of changes affecting a larger area has proven difficult and OS-specific. It depends on timing and concurrency issues in the text system and I haven't been able to find a reliable way of tackling this problem.

At the moment, we have some invalidation/relayout kludges that vary somewhat across macOS, iOS, and visionOS — see also [Note Minimap Redraw Voodoo].

The easiest solution may be to use a custom view for the minimap instead of re-purposing {NS|UI}TextView. Given that we can (hopefully) reuse all other components of TextKit 2, this shouldn't be too difficult. See also the TextKit 2 video and example code from WWDC 2021.

@mchakravarty mchakravarty added bug Something isn't working architecture Requires changes that affect the general architecture labels Feb 12, 2024
This was referenced Feb 12, 2024
mchakravarty added a commit that referenced this issue Feb 12, 2024
Contributes to #99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architecture Requires changes that affect the general architecture bug Something isn't working
Projects
Development

No branches or pull requests

1 participant