Make stack-usage display inline, same as opt-view

[OfekShilon]

Following the intention declaration here:

[RubenRBS]

Looks great :)

[RubenRBS]

Random thought: Should we give this some ofther font?
Maybe so it looks like a codelens text? https://microsoft.github.io/monaco-editor/playground.html?source=v0.48.0#example-extending-language-services-codelens-provider-example

[OfekShilon]

Interesting! Does this look better?

For opt-remarks I would be interested in making part of some comments text link to source, and I'm not sure if that is possible with InjectedText. Does codeLensProvider enable this?

[mattgodbolt]

Ooh I prefer the font there? But also I'm cool with merging this and improving/tweaking later :)

[OfekShilon]

@mattgodbolt sorry, I don't understand: which font do you prefer?

[mattgodbolt]

LGTM! thank you Ofek!

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ TypeError: Cannot read properties of undefined (reading 'replace') forEach(panes/stack-usage-view) View Issue

_{Did you find this useful? React with a 👍 or 👎}

[partouf]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ TypeError: Cannot read properties of undefined (reading 'replace') forEach(panes/stack-usage-view) View Issue

Did you find this useful? React with a 👍 or 👎

Never seen this kind of sentry comment, very interesting.

I think if we did || 'None' here

compiler-explorer/static/panes/stack-usage-view.ts

Line 156 in 9cdd3a0

suClass: rem.Qualifier,

the problem would be fixed

[partouf]

Perhaps also add ? here

compiler-explorer/static/panes/stack-usage-view.interfaces.ts

Line 40 in 9cdd3a0

Qualifier: 'static' | 'dynamic' | 'dynamic,bounded';

[OfekShilon]

Just pushed 930ee1d , in hope of learning more before applying a permanent solution