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

Fix directx issues #2440

Closed
wants to merge 5 commits into from
Closed

Fix directx issues #2440

wants to merge 5 commits into from

Conversation

k-takata
Copy link
Member

@k-takata k-takata commented Dec 13, 2017

This PR fixes the following issues in the latest directx renderer:

  1. Make scroll slightly faster when scrlines:1.

  2. Fix y-position of displayed text.
    I hope that this fixes the issue reported at here: https://groups.google.com/d/msg/vim_dev/2AaZMN1T6K8/mnrR9ZJrBgAJ

  3. Fix directx with non-utf8 encoding.
    After 8.0.1343, nothing is displayed when 'enc' is not "utf8" and the directx renderer is selected.
    It should have fallen back to the GDI renderer.

  4. Add COLOR_EMOJI option and also update INSTALLpc.txt.

Note that the following issues are not fixed yet:

@codecov-io
Copy link

codecov-io commented Dec 14, 2017

Codecov Report

Merging #2440 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2440      +/-   ##
==========================================
+ Coverage   74.48%   74.51%   +0.02%     
==========================================
  Files          92       92              
  Lines      132744   132744              
  Branches    29123    29123              
==========================================
+ Hits        98878    98911      +33     
+ Misses      33835    33802      -33     
  Partials       31       31
Impacted Files Coverage Δ
src/os_unix.c 54.23% <0%> (-0.19%) ⬇️
src/channel.c 82.68% <0%> (-0.13%) ⬇️
src/if_xcmdsrv.c 84.17% <0%> (ø) ⬆️
src/screen.c 73.87% <0%> (+0.02%) ⬆️
src/gui_gtk_x11.c 47.65% <0%> (+0.09%) ⬆️
src/gui.c 47.45% <0%> (+0.15%) ⬆️
src/window.c 81.77% <0%> (+0.55%) ⬆️
src/gui_beval.c 63.09% <0%> (+7.29%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6d4849...64e7441. Read the comment docs.

The color emoji support requires Windows SDK 8.1 or later.  DirectX
support could not be enabled after 8.0.1343 without it.
Add COLOR_EMOJI option to be able to use older compilers without newer
Windows SDK.

Also fix indentation.
Also update INSTALLpc.txt.
@k-takata
Copy link
Member Author

@brammool brammool closed this in 7f88b65 Dec 14, 2017
@k-takata k-takata deleted the fix-directx-issues branch December 14, 2017 13:02
adizero pushed a commit to adizero/vim that referenced this pull request May 19, 2018
…s off

Problem:    DirectX scrolling can be slow, vertical positioning is off.
Solution:   Make scroll slightly faster when using "scrlines:1". Fix y
            position of displayed text. Fix DirectX with non-utf8 encoding.
            (Ken Takata, closes vim#2440)
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