LIVE FROM THE MINNESOTA SEARCH SPRINT, a little usability patch that gets it's help from the type argument, so that node search displays one help message and user search displays another. The hard coded HTML <UL> is also replaced with theme('item_list')

The idea for this patch comes from #247482, however we take a different approach here.

CommentFileSizeAuthor
#6 256678_6.patch5.19 KBlilou
#5 256678.patch5.2 KBgpk
#4 256678.patch5.18 KBdouggreen
#2 searchhelp.patch5.06 KBSouvent22
searchhelp.patch3.97 KBdouggreen

Comments

douggreen’s picture

douggreen commented

Please review the verbage in user_help 'search#noresults'. If you have ideas for clearer wording here, please let us know. Thanks!

Souvent22’s picture

Souvent22 commented
StatusFileSize
new searchhelp.patch5.06 KB

Updated patch with a simple test case that tests that a module which implements hook_search implements the hook_help for search#noresults.

robertdouglass’s picture

robertdouglass commented

Tracking

douggreen’s picture

douggreen commented
StatusFileSize
new 256678.patch5.18 KB

Updating the test case.

gpk’s picture

gpk commented
StatusFileSize
new 256678.patch5.2 KB

Have updated the 2nd line of verbage in user_help 'search#noresults', hope it's clearer.

lilou’s picture

lilou commented
StatusFileSize
new 256678_6.patch5.19 KB

Re-roll #5 against HEAD.

appel’s picture

appel commented

Subscribing.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

The last submitted patch failed testing.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs work » Closed (duplicate)

This is a duplicate of #569398: Make 'no results' configurable text