Closed (fixed)
Project:
Boxes
Version:
7.x-1.0-beta1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
12 Oct 2010 at 04:57 UTC
Updated:
14 Apr 2011 at 16:09 UTC
First of all, thanks for the quick and initial port of this module to D7. Awesome!
Just noticed one minor thing, which is that "Add box" should be a MENU_LOCAL_ACTION instead of a MENU_LOCAL_TASK. This would be more consistent on how other modules work.
Attached is a screenshot of what I mean, and the smallest patch ever.
| Comment | File | Size | Author |
|---|---|---|---|
| boxes-local-action.png | 14.94 KB | dixon_ | |
| boxes-local-action.patch | 671 bytes | dixon_ |
Comments
Comment #1
Grayside commentedMade this tweak manually and I now get a link to the box-adding page.
Comment #2
rvilarThis patch works for me
Comment #3
febbraro commentedThis one is in git now, thanks!