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.
On the display configuration of facet's, there's a setting for Pretty path alias.
Here, the option's title and field are ripped apart, depending on the viewport size:
The title is in the 4th col, while the input field (and help text) is in the 1st.
Not sure, maybe Facetapi is to blame?
Comment | File | Size | Author |
---|---|---|---|
#6 | facetapi_pretty_paths-7.x-1.x-2629206-6-facetapi_facet_display_form-fieldset.patch | 2.27 KB | donquixote |
#3 | fapi-global-html.jpg | 39.32 KB | sirtet |
pretty-path-alias-UI.jpg | 130.18 KB | sirtet |
Comments
Comment #2
dasjoThe layout is controlled by facetapi, feel free to reopen and move this to the facet api queue if needed :)
Comment #3
sirtetOk i looked at the html, and it seems like there's a
<div class="facetapi-global-setting">
missing around the pretty-path settings:Before i move it, can you confirm this should be added by facetapi, and not by the module providing these settings?
Or, to compare, do you know of any other module adding elements there?
The GLOBAL SETTINGS part of the facetapi UI is very confusing anyways.
The floats are causing confusion.
Maybe i need to do a new ticket for that or re-focus this once it's moved, but wanted confirmation first that this problem here really is facetapi's job.
Comment #4
sirtetThis is from facetapi. It wraps all its global- items in that div. I think it can't do that for other modules after they add their stuff with form alter, no?
So pre- and suffix should be added here too i think.
Comment #5
dasjook i see. well, we could just add the same prefix / suffix in facetapi_pretty_paths_form_facetapi_facet_display_form_alter then. if you agree, could you do a patch for that?
or we move this to the facetapi queue and suggest to remove the floats there
Comment #6
donquixote CreditAttribution: donquixote as a volunteer commentedThis patch moves the extra settings from facetapi_pretty_paths into a separate fieldset.
The form elements still have 'global[..]' as their name, and not 'facetapi_pretty_paths[..]', so we don't have to change everything else. Thanks to
'#parents' => ['global']
on the new fieldset.Comment #7
donquixote CreditAttribution: donquixote as a volunteer commentedThe other solution would be to do like facetapi itself:
In facetapi_facet_display_form() this is added to each component down in the "Global settings" fieldset.
I don't really like this, but this is how it is implemented there.