Background: http://drupal.org/node/537828

Here is the patch for updating the Tracker module's Help function. Contains reformatting and some rewrites.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

jhodgdon’s picture

Issue tags: +#d7ux

A few comments:

a) All the lines should have the same indentation.

b) This line should have .= in it: $output = '<h3>' . t('About') . '</h3>';

c) This is a bug in the Tracker module in general, and not in the help file specifically, but I thought we were no longer using the term "posts" when referring to content? So why is the tracker page still called "Recent posts", and why does it display "no posts" when there aren't any yet?????????

jhodgdon’s picture

I filed (c) as a separate issue: #637930: Tracker screens still using "post"

batigolix’s picture

Assigned: Unassigned » batigolix

ok. i shall wait for the outcome of #637930: Tracker screens still using "post" before redoing the helptext

arianek’s picture

i wouldn't wait on it, there are blocks still using the name posts as well... they'll all get changed eventually, but we shouldn't hold out on it.

batigolix’s picture

Status: Needs work » Needs review
FileSize
33.49 KB
30.71 KB
2.38 KB

worked around the "posts" problem

Status: Needs review » Needs work

The last submitted patch failed testing.

jhodgdon’s picture

The tracker module issue was done, so this patch needs an update.

marvil07’s picture

Status: Needs work » Needs review
FileSize
2.57 KB

trying to male the patch apply

arianek’s picture

overhauled this one - added some more content, cleaned up and "drupalized" the language. needs reviewing!

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

I like it, and the patch looks clean to me. :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

One minor omission: There's also a "My recent content" tab where a user can track their own content. This is one of the most frequently accessed pages on Drupal.org, so it's worth mentioning in the help, methinks. :)

So after "...occur in the text, or when new comments are added." I added: "The My recent content tab limits the list to the currently logged-in user." I would completely understand if this horrifies someone and they'd like to create a follow-up patch to un-geekify this a bit. :P

Also, because I'm totally pedantic like that, I removed the extra space before "For more information..." ;)

Committed to HEAD!

arianek’s picture

i think it's tolerably geeky ;-) thanks, missed that bit!

Status: Fixed » Closed (fixed)
Issue tags: -#d7ux, -d7help

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