Change Yes/no Ajax to a checkbox.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Looks reasonable and I understand that we shouldn't care about this SEO details for the normal user.

oresh’s picture

I'm only ok with making it a checkbox.
I think there is a reason, why that long description is there, and it makes sense to me. Ajax is something important and heavy, and not needed everywhere. If the terminology is not quite understandably, maybe just add a link for further reading?

Xano’s picture

What about "Use AJAX for paging, table sorting, and exposed filters"?

We don't have to explain terminology, but this short sentence does tell the user what AJAX will be used for.

Bojhan’s picture

@oresh I am not sure if "more text" adds heaviness - it just makes it easier to ignore.

@Xano I thought about that, but aren't there more things (e.g. additional plugins)? I dont want to give the illusion that AJAX is just for those things

dawehner’s picture

My default it's indeed just exposed form, tablesort and pager, but yeah contrib could add more.

Xano’s picture

FileSize
1.42 KB

Re-roll as per Bojhan's request.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed d5df67d and pushed to 8.x. Thanks!

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.