Improve search settings help text

Patch #22 applied successfully and it's working as expected.

Thanks for the interest but this still needs usability review.

Still waiting for UX review. But looking again at the issue summary not clear what is being updated. Think the issue summary should be updated for what the test changes are and why. Screenshots couldn't hurt.

@eric@erubino.net:

Thanks for the updated screenshots and for rewriting the issue summary.

I am making some further updates: looking at the patch from #22, I do not think it makes any changes to the table. And I want to be a little more specific about where the text is updated.

I think the biggest change is replacing the text

Search pages that use an index may use the default index provided by the Search module, or they may use a different indexing mechanism. These settings are for the default index. Changing these settings will cause the default search index to be rebuilt to reflect the new settings. Searching will continue to work, based on the existing index, but new content won't be indexed until all existing content has been re-indexed.

The default settings should be appropriate for the majority of sites.

with

Changing these settings will cause the default search index to be rebuilt to reflect the new settings. Searching will continue to work, based on the existing index, but new content will not be indexed until all existing content has been re-indexed.

I have to think a bit about whether we will miss any of the deleted text.

Usability review

We discussed this issue at 📌 Drupal Usability Meeting 2023-07-28 Fixed . That issue has a link to a recording of the meeting. For the record, the attendees at the usability meeting were @AaronMcHale, @Emma Horrell, @benjifisher, @lukas.fischer, and @rkoller.

We agreed that the proposed changes are an improvement. We especially considered the text that is removed (as I pointed out in #38) and we decided that removing it is a good idea.

I already updated the issue description when I added #38, so I am removing the tag for an update.

#22 still applies for 11.x

So believe this can be moved to RTBC

I'm triaging RTBC issues → .

I read the issue summary and the comments. Thank you @eric@erubino.net for the issue summary update, it has made this task much easier. Also, thanks to @smustgrave for asking that it was done. Although, I do think that item 3 in the UI changes "content ranking is now sortable" is incorrect.

There has been a Usability review confirming that changes are OK. It is unfortunate that the patch is re-testing on 9.5 but this has been recently tested on 11.x. I have not found anything else that needs to be done here.

Leaving at RTBC.

xjm → credited AaronMcHale → .

xjm → credited Emma Horrell → .

xjm → credited lukasfischer → .

xjm → credited rkoller → .

Saving credits, and attributing usability review meeting participants

I reviewed the changes carefully with a word diff, and manually tested the page with the patch applied.

When I first reviewed the diff, I was concerned about losing any reference to cron. (Site owners often don't realize that they need to configure cron, so referencing it is important.) I was particularly concerned about losing the link to the cron page. However, when manually testing, I see that the cron link is still mentioned at the top of the page. It's still clear in context, so that's okay.

Overall I think the tradeoff is worthwhile. I'm linking a couple related UX issues for this and related admin pages that would make good followups.

As a set of string changes, this issue is minor-version only, so I committed it to 11.x and did not backport. Thanks everyone!

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