[Property Grid] Property value for multiple selection when `alwaysShowEditor` is true

[Emi1yZhang]

Package
@itwin/property-grid-react

Describe the bug
We have a rule 'alwaysShowEditor' to always display the editor.
We have two vertices with different radius values:

• Vertex [2-MC]: 45.0 ft
• Vertex [2-M9]: 41.0 ft

When selecting both [2-M9] and [2-MC] at the same time, the property grid displays a single value of 40.49 ft, which does not match either value.

The property records provided in our getData override appear to be correct.

{
value: {
valueFormat: 0,
},
property: {
typename: "double",
name: "pc_opnsite_AreaVertex_Radius",
displayLabel: "Radius",
kindOfQuantityName: "CivilUnits:LENGTH",
quantityType: "CivilUnits:LENGTH",
editor: {
name: "presentation-quantity-editor",
},
},
description: undefined,
isReadonly: undefined,
isDisabled: undefined,
isMerged: true,
autoExpand: false,
extendedData: undefined,
links: undefined,
imodelKey: "38e911a1-4749-485b-a677-563a1b867d2c",
}

If alwaysShowEditor is turned off, the property grid correctly shows “Varies”, indicating that the underlying data is valid. This suggests a potential issue related to alwaysShowEditor.

I also noticed that:

• Records with string values (e.g. ECInstanceId, Location) behave correctly.
• Records that use a formatter exhibit this issue when alwaysShowEditor is enabled.
  This seems to indicate a bug in how formatted numeric values are handled when alwaysShowEditor is true.

Desktop (please complete the applicable information):

• OS: Windows
• Browser: Electron 36.9.1
• iTwin.js Version 5.4.2
• @itwin/property-grid-react 1.19.5

[grigasp]

Symptoms similar to #1542 (placeholder being displayed as the actual value).

[grigasp]

@Emi1yZhang, the fix is released with @itwin/presentation-components @ 5.2.17. I'm closing the issue, please reopen it if the fix doesn't work for you.

[Emi1yZhang]

@grigasp I tried updating to 5.12.17, for multi selection, the value displayed is still not correct, I'll reopen the issue.

[grigasp]

@MartynasStrazdas fyi

[Emi1yZhang]

sorry, just realized need studio to update the version, can't just test with os+. I'll give an update in comments once studio updated to this version.