Now that there's more space for this form, the Aggregation type element could do to be changed to radios.

At the moment, to understand what each type means, a user has to select each one in turn to see the description text.

With radios, all the descriptions can be visible all the time.

CommentFileSizeAuthor
#2 2747767-2.search_api.aggregation-type-radios.patch1.14 KBjoachim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Active » Needs review
FileSize
2747767-2.search_api.aggregation-type-radios.patch1.14 KB
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Issue tags: +release target
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented

I guess this makes sense. Let's get this in.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Status: Needs review » Reviewed & tested by the community

I guess this makes sense. Let's get this in.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Status: Reviewed & tested by the community » Fixed

OK, thanks for reviewing!
Committed.
Thanks for the patch!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.