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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start using new DisplayLayer APIs #474

Merged
merged 10 commits into from
Apr 21, 2016

Conversation

as-cii
Copy link
Contributor

@as-cii as-cii commented Apr 20, 2016

These changes are an attempt at switching to the new DisplayLayer text decoration facilities (in a backward compatible manner), which have superseded some deprecated APIs, such as tokenizedLineForScreenRow. More informations in the related PR.

@abe33: what do you think? I haven't tested this extensively, but definitely feel free to make any change you think would be needed to this branch (it's a fork, so I have given you collaborator access on my version of the repository).

Thanks! 馃檱

@abe33
Copy link
Contributor

abe33 commented Apr 20, 2016

Thank you @as-cii for taking care of that! I'll give it a look as soon as I come back home.

@abe33
Copy link
Contributor

abe33 commented Apr 20, 2016

I mostly fixed issues related to the current version of Atom, I found that some lines disappear when scrolling up (it was reported before but I couldn't reproduce on my config), so I'll investigate that part as well. Anyway, thanks again for the kick off on this.

@abe33 abe33 merged commit ae7b9bc into atom-minimap:master Apr 21, 2016
@as-cii as-cii deleted the use-new-display-layers-apis branch April 21, 2016 21:39
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.

None yet

2 participants