-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
navbar_title missing for list view #8169
Comments
Hi ! Do you want to provide a PR ? |
Could do, its not very nice code, pretty sure I've stripped out necessary things :/ Probably better of someone familiar with the codebase having a go? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The navbar_title on list view pages is blank which makes it a harder to know where you are. We have many admin entities and it would be helpful to use this space with a sensible heading. I know the breadcrumbs show where you are but the absence of this seems inconsistent and trying to trace the code through, looks like we are expecting something to be displayed there.
Environment
Latest Symfony/Sonata on linux nginx docker.
Sonata packages
show
Symfony packages
show
PHP version
Subject
Steps to reproduce
browse to /admin/app/test/list and see there is no navbar_title:
Other pages such as edit or show have an appropriate navbar_title:
Expected results
Nav title should not be blank, there is a appropriate title in breadcrubms and html title, not sure why it is missing from navbar_title?
Actual results
Seems this is the code responsible for the title and navbar_title, admin.isChild is false so not executed leaving it blank. Not sure how Title has the appropriate text and navbar_title ends up blank :/
Fix / Workaround
I have overridden the
@SonataAdmin/CRUD/list.html.twig
template with:And now my nav title works as expected.
Obviously not a real fix as no translations and removed the possibly needed isChild test etc.
The text was updated successfully, but these errors were encountered: