Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recently accessed items do a full page reload #64629

Closed
flash1293 opened this issue Apr 28, 2020 · 8 comments · Fixed by #77446
Closed

Recently accessed items do a full page reload #64629

flash1293 opened this issue Apr 28, 2020 · 8 comments · Fixed by #77446
Assignees
Labels
REASSIGN from Team:Core UI Deprecated label for old Core UI team Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc

Comments

@flash1293
Copy link
Contributor

Kibana version: master

Describe the bug: When clicking the links in the "recently accessed" list of the side nav, a full page reload is performed. The navigation to these items should use the flow of application.navigateToApp if applicable to perform the navigation to avoid reloading the page (basically in the same way the actual nav links work).

This is currently not a problem because most features using this feature are in the legacy platform in a single "app" anyway, but when cutting over to the new platform, this won't be true anymore.

Not sure whether this is a Core UI or a platform concern.

@flash1293 flash1293 added the REASSIGN from Team:Core UI Deprecated label for old Core UI team label Apr 28, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core-ui (Team:Core UI)

@Dosant
Copy link
Contributor

Dosant commented Apr 29, 2020

I think, ideally, this needs: #58751

@myasonik
Copy link
Contributor

Recently accessed items will also be going away soon so not sure if this is worth the effort... @ryankeairns thoughts?

@flash1293
Copy link
Contributor Author

flash1293 commented Apr 29, 2020

If it’s on the way out I’m +1 for not putting any effort in it - if #58751 fixes it, good, otherwise it’s not a big deal. The list is still functional, just not the perfect experience

@ryankeairns
Copy link
Contributor

The plan, as of right now, is to display recents in the Global Search feature. We will keep an eye on this one and see if the experience becomes intolerable in the meantime.

As part of the Kibana home page/landing page project there is a chance we also provide an in-page solution again. It's all on my radar, thanks for pointing out the degraded experience!

@pgayvallet pgayvallet added the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Sep 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

@pgayvallet pgayvallet self-assigned this Sep 10, 2020
@pgayvallet pgayvallet added this to 7.11 - Tenatitive in kibana-core [DEPRECATED] Sep 10, 2020
@pgayvallet
Copy link
Contributor

@ryankeairns

The plan, as of right now, is to display recents in the Global Search feature. We will keep an eye on this one and see if the experience becomes intolerable in the meantime.

Just to be sure as it's quite old now: #72331 still have the recently accessed items in the left nav, right? Just to confirm this issue is not outdated.

@ryankeairns
Copy link
Contributor

ryankeairns commented Sep 10, 2020

@pgayvallet

The current thinking is that we ship search as-is, leave recents in the left nav, and collect customer feedback before determining the 'ideal' default state.
A separate issue was created to track the factors and suggestions which led us to this approach: #75040

kibana-core [DEPRECATED] automation moved this from 7.11 - Tenatitive to Done (7.10) Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
REASSIGN from Team:Core UI Deprecated label for old Core UI team Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
Projects
Development

Successfully merging a pull request may close this issue.

6 participants