I posted a 7.x version which passes 'coder' module tests and will install & work as far as kiosk functionality. It needs some tweaks to fit better with the new 7.x dashboard or menu system, and is really only usable if you have the Admin Menu module enabled. Currently it puts the 'Kiosk' menu as a top-level item, but it should probably go under the 'Configuration' item, or possibly even the 'Appearance' item. I'd like to open it up to discussion: ?

Comments

Crell’s picture

Crell commented

I'd say the Kiosk configuration page goes under Configuration. Is Appearance even a full section anymore? :-)

I don't know why this module would be affected by admin_menu or dashboard, though; at least not any more than any other module. It's just adding template variables.

tylerwalts’s picture

tylerwalts commented
Assigned: Unassigned » tylerwalts

Sorry for being vague above. I meant to refer to the administration section of the Kiosk module is hard to use when I first installed it on the Drupal 7.0 alpha2 release. Straight out of the box, I had trouble accessing the 'Set computer' menu option (to set a cookie), as all I could see on the dashboard menu was the parent link for just 'Kiosk mode' (which is the page that gives you the textarea to enter in your list of kiosks). Only after I installed the Admin Menu option was I able to get the child menu item to show up under the kiosk item. The only problem is that it shows up as it's own top-level menu item, and it looks like its because they've changed the menu paths to /admin/config/* rather than the existing /admin/settings/*

The main issue for me was that I needed to get up to speed on the changes to the menu system in Drupal 7 and the differences from Drupal 6, but now I'm pretty sure that if we update the paths inside the _menu hook then we can get it to show up in the right parent menu items. I'm taking a look at it now to see if I can make it play nice in the menu with a fresh D7 install.

Crell’s picture

Crell commented

Ah, right right. Yeah, the best practices for where to put things in the menu system changed entirely in D7. I still don't really understand them all myself. :-) We should probably ping someone from the UX team for suggestions here.

Bojhan’s picture

Bojhan commented

It needs to live on /admin/config under a chosen category. Preferably this is system, since its a setting that affects the whole Drupal site and is not specific to a given activity.