Closed (fixed)
Project:
Flag
Version:
7.x-2.x-dev
Component:
Flag core
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
5 Aug 2010 at 02:46 UTC
Updated:
16 Sep 2010 at 02:40 UTC
Jump to comment: Most recent file
Drupal 7 core has settled on using "action" links for "Add a new one of these". Compare the Flag and Content admin screens:

I'm used to looking in the upper-left for this navigation, so I missed where to create a flag at first.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | add-as-local-action.diff | 1.59 KB | mooffie |
Comments
Comment #1
mooffie commentedHere's the patch.
- Should I do "Add new flag" instead of "Add a new flag"? (I see the convention is not to use articles... "add vocabulary").
- hook_help() it used so the help appears above the link, as in other places in Drupal. Will be backported to D6.
- It's a pity we don't have an brief introductory sentence ("Flags are ...") to precede "This page lists ..." but I couldn't think of any.
Comment #2
mooffie commentedNate? Are you ok with this patch?
Comment #3
quicksketchYep, looks good to me.
Comment #4
mooffie commentedCommitted.
http://drupal.org/cvs?commit=415140
http://drupal.org/cvs?commit=415142