Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
31 Aug 2009 at 17:27 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sign commentedreviewed, tested works fine
screens attached
Comment #2
gábor hojtsyIt does not hurt to wait for the testbot too :)
Comment #3
pp commentedI think it is a good idea!
The patch works fine, but I didn't tested because Drupal head have problem now. Wait for testbot.
On thing: I don't understand why use weight = 2?
pp
Comment #4
gábor hojtsyOh, looks like the List tab was left there even though not used. We take advantage of that. Also, the weight was 2 before, so my rollback includes that. We can set it to whatever is > 0, since the List tab has 0.
Comment #5
pp commentedOh..
I found it:
The "list tab" is the default local task
The "add tab" was the weight 1 local task, but now it is an local action
The "settings tab" was the weight 2 local task, and now(before the patch) it isn't tab.
I think it is little bit inconsistent. I suggest set settings tab's weight to 1
pp
Comment #6
gábor hojtsySetting the weight to 1.
Comment #7
Bojhan commentedTook to long Mr. Bot!
Comment #8
webchickCommitted to HEAD. Thanks!
Needs docs.
Comment #9
webchickComment #10
samuelsov commentedDocumented in a new child page to main http://drupal.org/update/modules/6/7 page to list all path changes from D6 to D7 : http://drupal.org/node/719612
#d7csmtl