Enhancement suggestion: Show NotFound when a control panel is not found
Currently, when accessing a non-existent control panel URL (e.g., /controlpanel/does-not-exist), the Controlpanel component returns an empty <div /> if the panel is not found. This provides no visual feedback to the user about the error.
Problem
- Users who type an incorrect URL or try to access a removed panel receive no indication that the panel does not exist.
- This can be confusing, as the screen remains blank with no error message.
Proposal
Change the component to explicitly render the NotFound screen (<NotFound />) when the panel is not found, instead of returning an empty <div />.
Benefits:
- Provides clear feedback to the user when a panel does not exist.
- Improves the user experience and usability of the system.
hello sir. Is this issue is still open ?
@stevepiercy I think no one is working on this issue. Can i start working on it?
@VibhuGupta-dev If don't have an open PR and no one is assigned to the issue, you can simply start working on it and open a PR. No request is necessary.
@VibhuGupta-dev But in this case, there is already an open PR: https://github.com/plone/volto/pull/7218
However, the issue is not referenced in the PR description; I will fix it.
You can talk to @shashank0470
Hey @shashank0470
Since this PR has been inactive for a while and the requested changes seem to be pending because the updated code wasn't pushed, I wanted to check whether you're still working on this.
I have implemented a complete working fix for the same issue in PR #7663 (including the routing loading condition and the NotFound behavior). I'm happy to collaborate with you — we can merge the approaches if needed.
If you prefer, I can also take over this PR and push the required changes here so that it can move forward. Just let me know what works best for you.
Thanks!
Thanks for checking! @VibhuGupta-dev
I haven’t been able to complete the pending updates.
If you’d like to commit the required changes directly to this PR, please go ahead — that works for me.
Thanks for helping move it forward!
I noticed the issue is still present. I’ve implemented a fix on my side, would it be helpful if I opened a PR?
@VibhuGupta-dev can help you on this if he is not working on it.
We have 3 open PRs for this issue now. @iyush05 please read and follow First-time contributors, especially Things not to do, Contributing to Plone, and Contributing to Volto.