Closed (fixed)
Project:
Facetapi Select
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
8 Oct 2014 at 01:04 UTC
Updated:
16 Sep 2015 at 13:51 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rooby commentedComment #2
PCateNumbersUSA commentedWould love this as well, right now the theming options are very limited without this.
Comment #3
julien commentedInstead of a theme function, i did in the meantime a patch so we can choose to display ('.$item['#count'].')' in the facet configuration. See patch attached.
Comment #4
julien commentedComment #5
julien commentedpatch is duplicate with the one here: https://www.drupal.org/node/2283177
Comment #6
rooby commentedVery nice to have that option but it will definitely need a theme function also.
Themers need to be able to override the markup regardless of what has been selected in the settings.
For example they might not want the count wrapped in parentheses or they might want additional spans and classes for styling.
Comment #7
julien commentedYou mean spans and classes outside the select itself? or are you talking about the select options
Comment #8
julien commentedAdded a theme function to the select inside the form
Comment #9
julien commentedit's 4am, i will finish that patch work later on
Comment #10
rooby commentedOops I mistook this for a different issue. Now that I have properly refreshed my memory...
Markup is not relevant here since we're dealing with select options, so it's just for changing the actual text, like the parentheses, or possibly you want the count at the start or something like that.
Comment #11
rooby commentedOverriding the themeing of the whole select is a bit out of the scope of this issue. This one is just for options.
So something to change this:
With something along these lines:
And then a theme function to print out something like this:
EDIT: Not sure why DRupal.org is messing with the code format.
Comment #13
grndlvl commentedI agree with rooby as you may now theme the whole form using '#theme' from within a hook_form_alter() now that we have #2136831: Use drupal_get_form() when retrieving the select options form in place.
Attached is a patch of the changes made that will be committed http://drupalcode.org/project/facetapi_select.git/commit/97388ea.