[Property grid]: don't hide non empty values

[JonasDov]

closes #1715

[Copilot]

Pull request overview

This PR addresses issue #1715 where the Property Grid “Hide null values” feature was incorrectly hiding boolean switches (e.g., false) and merged properties by refining the null-value filtering logic and expanding coverage with targeted tests.

Changes:

• Replace the “non-empty” value filter with a “non-null” value filter that uses underlying primitive value presence (instead of displayValue truthiness).
• Update PropertyGridContent to use the new non-null filterer when showNullValues is disabled.
• Add regression tests to ensure false boolean primitives and merged primitives are not filtered out.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
packages/itwin/property-grid/src/property-grid-react/components/FilteringPropertyGrid.tsx	Renames/updates the filterer to treat false/merged primitives as non-null.
packages/itwin/property-grid/src/property-grid-react/components/PropertyGridContent.tsx	Switches the default “hide nulls” filterer to the new non-null filterer.
packages/itwin/property-grid/src/test/components/FilteringPropertyGrid.test.tsx	Adds regression tests for false booleans and merged primitives; updates references to the renamed filterer.
change/@itwin-property-grid-react-1715-hide-null-boolean.json	Adds a patch change note for the fix.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

packages/itwin/property-grid/README.md:88

• This README comment still refers to "show/hide null values", but the example now uses ShowHideEmptyValuesSettingsMenuItem. Updating the wording keeps the documentation consistent with the renamed API.

          (props) => <ShowHideEmptyValuesSettingsMenuItem {...props} persist={true} />,
        ],

        // supply an optional custom storage for user preferences, e.g. the show/hide null values used above
        preferencesStorage: new IModelAppUserPreferencesStorage("my-favorites-namespace"),

[Copilot]

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 3 comments.