Here are a few small changes to text and titles to tidy up:

  1. The title of the menu item should be simply 'Scheduler' instead of 'Scheduler module settings'. Most other modules just have their name so for standards it is best to match them, and the words 'module settings' are not required.
  2. Change the description to actually say what can be done, because 'allow site admins to configure scheduler' gives no real help.
  3. Now that we have tabs, we do not need to set the titles of the pages. Other modules do not have different titles for the tabs so remove the call to drupal_set_title()
  4. Scheduler's own cron is referred to by several names, 'internal', 'light weight' and 'lightweight'. Most commonly it is the single word 'Lightweight' so for consistency, I suggest we change to that

patch to follow

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jonathan1055’s picture

Status: Active » Needs review
FileSize
25.25 KB
19.71 KB
2.4 KB

Here's the patch, against 1.1 dev of 20th April

Before:

After:

I've not done screen shots of the other textual changes. This was from some old old changes I made to D6 and the patch has been sitting around and I never submitted it. Not very exciting, but it's good to tidy things up now and again.

rickmanelius’s picture

Status: Needs review » Reviewed & tested by the community

Works for me. If anyone has any objections, open the ticket back up and we'll discuss. I'll commit this into the 7.x-1.2 version.

rickmanelius’s picture

Version: 7.x-1.1 » 7.x-1.x-dev
Status: Reviewed & tested by the community » Fixed
jonathan1055’s picture

As this is only cosmetic, I do not think we need to port to D6.

Status: Fixed » Closed (fixed)

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