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.

CommentFileSizeAuthor
boxes-local-action.png14.94 KBdixon_
boxes-local-action.patch671 bytesdixon_

Comments

Grayside’s picture

Grayside commented

Made this tweak manually and I now get a link to the box-adding page.

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
commented
Status: Needs review » Reviewed & tested by the community

This patch works for me

febbraro’s picture

febbraro commented
Status: Reviewed & tested by the community » Closed (fixed)

This one is in git now, thanks!