Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
At the 'Configure facet display' page I have no way of knowing which facet I am working with, short of peering at the URL (such as admin/config/search/facetapi/apachesolr%40solr/block/ss_brand/edit) and trying to figure it out from that.
Comment | File | Size | Author |
---|---|---|---|
#4 | admin-page-titles-1557784-4.patch | 1.97 KB | cpliakas |
Comments
Comment #1
cpliakas CreditAttribution: cpliakas commentedMakes sense to me. Would love to see a patch to resolve the issue.
Thanks for reporting,
Chris
Comment #2
joachim CreditAttribution: joachim commentedI'm completely new to this module, so not sure where to best put this info or indeed how much there is to convey:
- in the page title or in the form as text?
- how much of the pieces of 'apachesolr%40solr/block/ss_brand' need to be given and how can they be converted to human strings?
Comment #3
cpliakas CreditAttribution: cpliakas commentedAfter taking a quick look through the code, the facetapi_facet_display_form() function is where this should be attacked. You can see where the "drupal_set_title()" is executed, and you could also add a header if you want as well. I will leave it up to you, and would love to see a patch for this.
Thanks,
Chris
Comment #4
cpliakas CreditAttribution: cpliakas commentedAttached patch makes the suggested change.
Comment #6
cpliakas CreditAttribution: cpliakas commentedLet's try testing against the correct branch...
Comment #7
cpliakas CreditAttribution: cpliakas commented#4: admin-page-titles-1557784-4.patch queued for re-testing.
Comment #8
cpliakas CreditAttribution: cpliakas commentedBump to first page.
Comment #9
cpliakas CreditAttribution: cpliakas commentedCommitted to all branches of Facet API: