Skip to content

[9.1] [Security Solution] Fix initial data view flash (#225675)#226207

Merged
kibanamachine merged 6 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-225675
Jul 15, 2025
Merged

[9.1] [Security Solution] Fix initial data view flash (#225675)#226207
kibanamachine merged 6 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-225675

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

## Summary

Solves the initial UI "flash" when new data view picker is enabled and
the application is starting.

Compared to previous approach, we are not doing default data view
provider and just check the loading state before telling the user no
indices are matched.

## Testing
Head to timelines page or explore pages... should not see the data flash
with this feature flag on.
Brief blank page could occur, as the proper loading screens would be
required.

```
xpack.securitySolution.enableExperimental: ['newDataViewPickerEnabled']
```

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 209c298)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 2, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 2, 2025 13:30
@kibanamachine kibanamachine merged commit 698cf89 into elastic:9.1 Jul 15, 2025
11 of 12 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 7838 7839 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 9.8MB 9.8MB +933.0B

History

cc @lgestc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants