wpf icon indicating copy to clipboard operation
wpf copied to clipboard

Hyphen based ligatures don't work in WPF

Open clairernovotny opened this issue 6 years ago • 65 comments

See https://github.com/tonsky/FiraCode/issues/259 for the full issue description.

Would be good to get this fixed for .NET Core 3.

clairernovotny avatar Dec 06 '18 15:12 clairernovotny

This makes Visual Studio very sad. -> is a very common symbol in C++, and I don't get pretty arrows for it.

DrPizza avatar Mar 28 '19 16:03 DrPizza

I've tracked this bug through various different platforms now, and it would be really great if it could get fixed. It's a small thing, but it would make everyone's programming life a little more fun.

Fell avatar Apr 05 '19 10:04 Fell

Problem persists in Visual Studio 2019.1

limbusdev avatar Apr 10 '19 12:04 limbusdev

in vs2017 too, but not in vs code

HolyChen avatar Sep 20 '19 05:09 HolyChen

I suspect there's something wrong in TextItemizer that it breaks out hyphens (-) as individual run.

be5invis avatar Oct 29 '19 05:10 be5invis

I understand this originally was probably by design, but considering that hyphen based ligatures are a desired feature in some fonts, that original "by design" is not appropriate anymore and needs to be revisited. I would have contributed by researching the implementation and proposing a fix, but considering the text engine is going to stay closed source this really needs to be done by MS.

weltkante avatar Oct 29 '19 14:10 weltkante

~Windows Terminal, VS Code and JetBrains all render the text better than Visual Studio, which uses the WPF renderer. Text rendered by WPF is less crisp than other technologies; horizontal lines are thinner, and vertical lines are thicker.~

image

~I heard that WPF used by Visual Studio uses ~2010 version of DWrite, and there was no demand to update the version of WPF used in .NET Framework. I would like this comment to serve as proof of demand for a fix on behalf of Visual Studio Editor's users and developers.~

~Some of the user feedback: https://www.reddit.com/r/VisualStudio/comments/ev5b2u/is_there_an_editor_text_readability_fix_already/ https://developercommunity.visualstudio.com/content/problem/44061/some-programming-ligatures-do-not-work.html https://developercommunity.visualstudio.com/content/problem/899361/editor-readability.html?childToView=904574#comment-904574~

AmadeusW avatar Feb 04 '20 23:02 AmadeusW

Edit: I moved the previous comment to appropriate location: #2025

AmadeusW avatar Feb 18 '20 17:02 AmadeusW

Just to be clear, the last two comments are off-topic and have nothing to do with hyphen-based ligatures (which is what this issue is about). The text layouting is unrelated to the font rendering.

weltkante avatar Feb 18 '20 18:02 weltkante

Thanks for clarification @weltkante. I thought that setting layout and rendering are handled in the same piece of code. I will move my comments to #2025

AmadeusW avatar Feb 18 '20 21:02 AmadeusW

Now both are closed. I think one should stay open, or not?

limbusdev avatar Apr 27 '20 14:04 limbusdev

#2025 has received some fix and this issue is still open because its not fixed yet. Not sure what exactly you mean?

weltkante avatar Apr 27 '20 14:04 weltkante

Any progress on this issue?

FluorineDog avatar Oct 14 '20 16:10 FluorineDog

Given Visual Studio .Net Core is open source.. is it possible to get into the source code and find and fix the bug for them and submit a pull request? Particularly if it's a WPF bug, seems like it might be possible.

FlogramMatt avatar Nov 14 '20 14:11 FlogramMatt

Neither Visual Studio is open source, nor the part of WPF that does text processing. I already mentioned this above, Microsoft has to fix this bug, the community cannot help.

weltkante avatar Nov 14 '20 18:11 weltkante

Any progress ?

CoderTyn avatar Jul 31 '21 06:07 CoderTyn

Any progress ?

Obviously not. WPF might be open source but repo got owned by WinUI team and they don't know much about C# or WPF. For new features only chance is a community pull request. And the last pull request which added a new feature was ignored for months, then closed without merging.

Symbai avatar Jul 31 '21 07:07 Symbai

Any progress ?

Obviously not. WPF might be open source but repo got owned by WinUI team and they don't know much about C# or WPF. For new features only chance is a community pull request. And the last pull request which added a new feature was ignored for months, then closed without merging.

thank u for your reply. there is no hope . ms does not care about it.

CoderTyn avatar Aug 05 '21 06:08 CoderTyn

I've mentioned it in VS2022 YouTube Chat, and they reacted to the comment: https://www.youtube.com/watch?v=lsInDJdx6As

jogibear9988 avatar Sep 21 '21 19:09 jogibear9988

I've mentioned it in VS2022 YouTube Chat, and they reacted to the comment: https://www.youtube.com/watch?v=lsInDJdx6As

Its at 22:14 btw for the benefit of everyone

geocine avatar Oct 09 '21 04:10 geocine

Status update, this is still an issue. Is there any plan to fix this issue?

julian-a-avar-c avatar Jan 04 '22 05:01 julian-a-avar-c

three years passed...

CoderTyn avatar Jan 05 '22 02:01 CoderTyn

See tonsky/FiraCode#259 for the full issue description.

Would be good to get this fixed for .NET Core 3.

.NET 6 released...

CoderTyn avatar Feb 09 '22 02:02 CoderTyn

Can you please fix this? Its been on the bug list for years (literaly).

flundstrom2 avatar Mar 04 '22 09:03 flundstrom2

Sup?

conioh avatar Mar 16 '22 07:03 conioh

I agree. It's time to fix this. What's so special about hyphens anyways?

Fell avatar Mar 16 '22 08:03 Fell

What's so special about hyphens anyways?

They take part in word breaking / line wrapping so probably have special code handling them that didn't have ligatures in mind when being written. Unfortunately its in a closed source blob thats linked into WPF, so I can just guess.

weltkante avatar Mar 16 '22 09:03 weltkante

Would be good to get this fixed for .NET Core 3.

3 years later, .NET 7 is on the horizon and this remains unfixed.

ifarbod avatar Apr 24 '22 15:04 ifarbod

See tonsky/FiraCode#259 for the full issue description.

Would be good to get this fixed for .NET Core 3.

I am not sure wether it is fixed or not in .NET Core 3, but Visual Studio seems to use .Net Framework 4.8?

CoderTyn avatar May 11 '22 04:05 CoderTyn

This is still broken in the latest version (16.11.15) of VS2019

cmacmillan avatar May 21 '22 02:05 cmacmillan