Opened 14 years ago
Closed 9 years ago
#16125 closed enhancement (fixed)
Tag cloud widget: hide dropdown when there's only one taxonomy
Reported by: | GautamGupta | Owned by: | GautamGupta |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | 3.1 |
Component: | Widgets | Keywords: | has-patch |
Focuses: | Cc: |
Description (last modified by )
We should use [20285]get_taxonomies()
instead of get_object_taxonomies()
. Attaching a patch.
When there's only one taxonomy to select from, we shouldn't show the dropdown at all.
Attachments (5)
Change History (28)
#4
@
14 years ago
This ticket is 3 months old and its way before wordpress-3.1 was released, still this issue is not addressed in this new version. Hope this gets addressed in the next update to come.
#6
@
14 years ago
I would love to hide the dropdown (make it input:hidden so it still saves) when there is only one taxonomy that supports a tag cloud. Less UI for the win.
Would like a patch to implement this.
#7
@
14 years ago
Attached another patch based on nacin's recommendations.
Should we return 'post_tag'
in _get_current_taxonomy()
if no taxonomy was found saved in the options?
#8
@
13 years ago
- Keywords needs-testing added
Would be helpful for bbPress 2.0 to inherit this instead of including it's own duplicate widget.
Added the needs testing keyword as it's been 6 months since it's had attention.
#10
@
13 years ago
- Keywords needs-refresh added
I kicked the tires on this patch and here is some feedback:
- Works nicely as I remove taxonomies till none are available.
- The text displayed in the widget when none are available could do with some copy editing to improve it.
- The whole patch could do with some work to match coding standards for things like spacing inside of function calls, around !, the whole switch statement.
- (Personal Taste) I'm not a big fan of the way the switch statement ends up laid out with all the switching back and forth between php and html. I wonder if it might read easier to use echo's instead.
Adding 'needs-refresh' to get the patch updated.
#11
@
13 years ago
I've attached a new patch based on westi's recommendations. I've also removed the fallback to the post tag taxonomy if no taxonomies are found as if it was hid due to some reason from the tag cloud, we should be acknowledging that.
#13
@
13 years ago
- Keywords dev-feedback added
If this is making it into 3.3, it needs to be committed now, as we have passed freeze. If it's not ready to be committed, punt to next cycle.
#18
@
13 years ago
- Description modified (diff)
- Summary changed from Tag cloud widget should allow any taxonomy (with show_tagcloud arg) to be shown to Tag cloud widget: hide dropdown when there's only one taxonomy
Trying to salvage this ticket somehow.
#19
follow-up:
↓ 20
@
12 years ago
Grammatical error in tagcloud.v3.diff --currently:
'The tag cloud would not be displayed since their are no taxonomies...
should be 'there':
'The tag cloud would not be displayed since there are no taxonomies...
Any chance of getting at least the one-line patch from [20285] into 3.4? Without that, taxonomies cannot even use the standard Tag Cloud widget.
Related: #11612. This is an additional enhancement past that.
I would love to hide the dropdown (make it input:hidden so it still saves) when there is only one taxonomy that supports a tag cloud. Less UI for the win.