DataViews: open the filters from the column table consistently

[hbhalodia]

What?

Closes #72986

Why?

• Fix the inconsistent behavior for the DataViews filter.

How?

• Add the useEffect to check for the filters and update the state as needed.

Testing Instructions

1. Navigate to storybook,
2. On page load, without any filter applied, click the "Categories" column and "Add filter".
3. Notice it would open now.
4. Remove that filter
5. Repeat step Add/local build #2.
6. Notice it would open again, so this is a consistent behavior.

Testing Instructions for Keyboard

• None

Screenshots or screencast

Screen.Recording.2025-11-05.at.4.06.44.PM.mov

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: hbhalodia <hbhalodia@git.wordpress.org>
Co-authored-by: StevenDufresne <dufresnesteven@git.wordpress.org>
Co-authored-by: ntsekouras <ntsekouras@git.wordpress.org>
Co-authored-by: oandregal <oandregal@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[StevenDufresne]

My instinct is that using a ref would be safer and more performant, seeing that setPrevFiltersLength won't be used in a different context. We can avoid rerenders and having prevFiltersLength in the dependency array, and then updating its value opens the door for infinite loops.

[hbhalodia]

This is now updated.

[ntsekouras]

I'm wondering if a better fix would be to just call this in table column header's add filter here. WDYT @oandregal ?

[hbhalodia]

Hi @ntsekouras, I tried that but setting explicitly there I thought was not better, even in filter's if you log the isShowingFilter, it is correctly showing the state, but only on initial filter check even it logs true, still it does not opens the filter panel.

[oandregal]

Yeah, updating the state in the table layout sounds better. Was there any issue with that approach, @hbhalodia ? I see the setIsShowingFilter function is already in the context, so available in the layout as well.

[hbhalodia]

The issue was not something with approach, We can set that explicitly in the column header, but my concern was, it was not working on first refresh, but thereafter it was working as expected, so updating the state in column header component wouldn't be a solution just to add a fix for this and explicitly adding to that component.

I can update the PR with tha approach (it was my first approach) but then thought of this, hence changed to that.

Updating the PR with the setting state via column header.

[hbhalodia]

Hi @oandregal, This is now updated, to use context within column-header.

[hbhalodia]

Hi @oandregal @ntsekouras, Can you please take a look on PR. Updated it with the required changes.

Thank You 🙇,

[oandregal]

Thanks for working on this @hbhalodia