#85858 Allow breadcrumbs.symbolSortOrder per language

[jzyrobert]

This PR fixes #85858
@jrieken Is this fine? I looked at how the other function uses OutlineModel, and it seems like it needs the input element, so I modified the function signature.

Example:

Javascript file without override uses default order:

Typescript files use the overriden name order:

[jrieken]

Not so nice - given that there is _setInput. Why not update/modify the filter in there

[jzyrobert]

I don't think that works.
I tried:

	protected _setInput(input: BreadcrumbElement): Promise<void> {
		const element = input as TreeElement;
		const model = OutlineModel.get(element)!;
		const textModel = model.textModel;
		this._overrideConfiguration = {
			resource: textModel.uri,
			overrideIdentifier: textModel.getLanguageIdentifier().language
		};

and

	private _getOutlineItemCompareType(): OutlineSortOrder {
		switch (this._symbolSortOrder.getValue(this._overrideConfiguration)) {

However in testing, _getOutlineItemCompareType gets called before _setInput, making the override undefined.

This makes sense as _createTree is called before _setInput, and OutlineComparator only takes a value rather than a function, so you have to have the override ready at the time of _createTree

[jrieken]

Just store the sorter in a property when _createTree is called and update it when _setInput is called

[jzyrobert]

I see what you mean, like this?

[jrieken]

looking good. thanks