-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Partial IntelliJ IDEA support #96
Comments
Didn't work for me on 16 EAP :( |
@dmitriid, for some reason, it doesn't work for all file types. If you create a new scratch buffer, the ligatures will work. |
@ShockOne Nope :) Didn't work for the new scratch buffer either :) I guess we can call the support is "spotty". But yay for you! You got it working |
@dmitriid did you forget to add |
Nope, done that. Then launched IDEA, all fonts look as they did before. In any type of file, in any type of scratch file |
Heh. Worked for me on Idea 15 with modified idea.properties. For some On Tue, Dec 22, 2015 at 3:31 PM dmitriid [email protected] wrote:
|
This isn't actually a fully working test case just yet: there are some problems which only show up when syntax highlighting is used. |
Hello, I'm a JetBrains employee, working now on supporting ligatures in IDEA. There's a reason why Fira Code in its current state cannot be supported in IntelliJ IDEA at the moment, some details about this can be found e.g. here. But there's a 'refactoring' for the font, which can make it work. I could provide a patch, if font's source code would be available in open format, but I don't have Glyphs app, so I can only explain the idea here. Basically, DFLT script tag shouldn't be used in OpenType features mapping, tags for all scripts supported by font should be used instead. For Fira Code this means, 'DFLT dflt' script/language pair should be replaced with 'zyyy dflt' 'zinh dflt' 'cyrl dflt' 'grek dflt' ('latn' script is already declared). I've tested font modified in such a way, and didn't see any regressions in any editors on any platform so far. |
Hi @JB-Dmitry! I’ll see what I can do about it. Does removing DFLT solves the issue when ligatures only render if a letter is placed before/after it? |
Yes, I think it should solve it. |
@JB-Dmitry here’s a test build https://dl.dropboxusercontent.com/u/561580/Fonts/FiraCode/FiraCode_1.102.zip I kept Folks in this thread, can you please test it in Windows/Linux and report back? @JB-Dmitry any ideas why Idea chooses Light weight by default and how to change that? |
That works for me on Linux! You're all awesome for looking into this so much! |
That should also work, thanks.
I couldn't reproduce that. What JDK are you using to run IDEA (can be found in 'About' screen)? |
Works on OS X! Great! |
@JB-Dmitry it must be related to |
Laugh all you want, still doesn't work for me on either 15 or 16 EAP :) |
Link to my full IDEA settings in case anyone wanna take a look (perhaps some setting somewhere interferes): https://dl.dropboxusercontent.com/u/2966998/idea/Mamut-15.jar |
OS X 10.11.2 (15C50) IDEA Ultimate 15 IU-143.1559.1 |
@JB-Dmitry I'm adding the |
@dmitriid Please try next IDEA 16 EAP build (to be released). If it still won't work, attach your log file here or at https://youtrack.jetbrains.com/issue/IDEA-127539. |
@JB-Dmitry Done |
The following builds have some fixes related to support of ligatures on Mac OS: @tonsky Issue with picking of Light font version should be fixed there. @dmitriid Try one of these builds, and attach logs from them. |
@JB-Dmitry Still nothing :( https://youtrack.jetbrains.com/issue/IDEA-127539#comment=27-1266742 I removed the font, reinstalled it from the link above. Set ligatures=true, nothing :( |
I've tried installing Fira Code on my Mint Petra, to use it with PHPStorm 10.0.2, but... I got a huge performance penalty. I tried simply navigating PHP, Blade and Markdown files, but all of them makes my CPU graph go crazy. The files are not big, but are from the same project. A markdown scratch file presents the same issues. The problem does not happen with Hasklig, though. The performance is normal with it. Should I open a separate issue about IntelliJ performance? |
@igorsantos07 I have experienced the same. Reported here |
Tried on Ubuntu 14.04 with AndroidStudio 2.0 Preview 4. Also had performance problem. |
Time required for application of OpenType substitution rules in the font is approximately proportional to the number of matching rules - as, more often then not, rules don't match, we need to go over each rule for each character/glyph in input string.
(in feature file syntax) The same can be implemented using just one rule:
Most of ligatures in Fira Code are for two or three characters, so I'd expect at least 2x performance improvement after such refactoring for all rules. Not sure this will be enough for eliminating performance issues completely, but potential gain seems worth trying. Disclaimer. This is applicable for ICU layout engine. It's probably also true for other OpenType layout engines, but I've not checked that. |
👍 Thanks for all the hard work @JB-Dmitry @tonsky , really looking forward to having ligatures in 2016! (hopefully 😄 ) |
Great many thanks to everyone involved 👍 |
IDEA 145.257 still picking up Fira Code Light if ligatures are enabled on Mac OS X 10.11. |
@singalen Same here in PhpStorm 2016.1 |
I get the ugly anti-aliased version with ligatures enabled, whether I have Fira Code Light installed or not (OS X 10.11, IDEA 2016.1.1/IC-145.597). |
It’s not a light version, IDEA just switches to different font rendering On Fri, Apr 22, 2016 at 12:03 AM Ryan Winchester [email protected]
|
@tonsky thanks for the clarification. @singalen @ryanwinchester @curtosis @jessedobbelaere I think these problems may be resolved in the 2016.2 update for intellij, fingers crossed! |
In the new release of EAP 2016.2. Works pretty well. (WebStorm, MAC OS X) |
Seems fixed in the EAP 2016.2 that was released for PHPStorm yesterday too 👍 |
Looks like it's partially working in Webstorm-EAP. Italics are not working. Anyone else seeing this? |
|
Ligature of Fira Code doesn't work for me too in IntelliJ 16.1.3 |
Are ligatures working for anyone with 2016.2? I can't get them to work no matter what font I try. |
Have you enabled them in settings panel (not in property file)? On Wed, Jul 13, 2016 at 10:17 PM Peter Malone [email protected]
|
@petermalone Yes! |
Is there a way to pick font weight? I can see the various font weights in: Settings -> Appearance & Behavior -> Appearance -> Override default fonts by But cannot see the weights in: Settings -> Editor -> Colors & Fonts -> Font -> Primary font I'd love to use a lighter weight version of FiraCode... |
Same problem here. OSX 10.10, RubyMine 2016.1.2, using settings panel to enable. Ligatures on => thin weight, ugly font rendering Ligatures-on rendering is very usable on Retina display, but not on external Samsung crapdisplay. On Retina, the font weight is thinner but still well-rendered. Same issue with Hasklig. |
This was fixed for me in PhpStorm 2016.2 |
Not all ligatures works in some buffers: in java buffers for example <= works , but not >=. In example window all works perfectly. |
@allmycode please report that to tracker. They are really serious about ligatures now |
To the Intellij tracker. There's a link somewhere in the thread |
I am still getting a weird version of Fira Code in intellij. VSCode Fira Code font looks quite better |
From version of 15 EAP there is a partial support for ligatures with
editor.enable.optional.ligatures=true
inbin/idea.properties
file.Here is a proof:
As you can see, ligatures appears if there is a letter (not number) beside.
I created a test case for this:
And this is how it looks in IntelliJ IDEA 16 EAP
v144.2151.16
:As you can see, every ligature that based on 2 glyphs is working, except of any brackets
<>[]{}()
But without letter beside there is nothing as I described above:
So, basically, we can add this info in repository
readme
.The text was updated successfully, but these errors were encountered: