Allow display:inline-block on spans style by yoavbls · Pull Request #180498 · microsoft/vscode

@yoavbls

DrJume, Aaron-Pool, xlboy, tizmagik, GauBen, rivy, mtdvlpr, sandrooco, vteivans, tommykw, and 45 more reacted with thumbs up emoji alon710, joaopedrodcf, wridgeu, mbtools, and DanSM-5 reacted with laugh emoji

@gjsjohnmurray

Pinging @mjbvz who might have an opinion about this change.

@yoavbls

@mjbvz Hey Matt,
I'm sorry I'm pinging you about it, but almost a year has passed, and I would be pleased if you could take a look at it and provide some clarification on the desired solution 🙏🏼

aminya

@aminya

This oneliner fixes the only missing feature of ts-pretty-errors. Could someone take a look? Looks like one year should have been enough time for reviewing this simple fix.

Code-Victor, miraclemenikelechi, joaopedrodcf, yoavbls, ChiriVulpes, StevenVroomGravity, frammiie, adriangalilea, sandrooco, CristianMain, and 2 more reacted with thumbs up emoji

@xenostar

Hello @mjbvz and @gjsjohnmurray. It has been nearly 1.5 years since this PR was opened. Are there any further reservations about merging this in? This enables a fix for #180496, which will enable a massive developer experience improvement for the Pretty TS Errors plugin.

What are the steps we can take to get this prioritized and possibly merged? This would enable a huge improvement to the DX for nearly one million developer using this plugin.

@AndreMaz

Any chance of merging this PR @mjbvz ?

@alanmoyano

Hello! @mjbvz could you please check this out? This would be awesome for empowering extension developers.

@mbtools

There's now a little conflit, yoavbls. Let's resolve it and give it another bump. TY! :-)

@yoavbls

@yoavbls yoavbls changed the title Allow display prop on spans style Allow display:inline-block on spans style

Mar 29, 2025

@yoavbls

@mbtools updated, now allowing only inline-block to make it even safer

@mbtools

@alanmoyano

@hexrw

@danielfoxp2

WoW, not even a reply to say they hated the solution or that the solution conflicts with their personal preferences and will not be merged?

This can easily impact millions of developers (me included) to a better experience and not a single word for years?

Shame.

@ouardito

Since Microsoft / The Assignee / The Reviewer has been silent on this pr for a long time ...

Does anyone know how to, instead, remove the original errors, and only show the Pretty TypeScript Errors ?