Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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:

Redraw directly instead of using WM_PAINT.
Some characters like 'y' and 'g' are cut off on some fonts.
@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