Hi Cory,
Thanks for sharing your first Omeka plugin with the rest of the
community! I ran into a few problems that I want to discuss. First,
a few minor/general things:
* There should be no space in the AccessKeys folder name.
* You can remove the views and models directories since you're not
using them.
The big problem is that it's not working properly in my browser. I
reviewed the code and I see what you're doing -- using the
public_theme_header hook to insert content into the page. Note that I
have no experience with using accesskeys.. so I'm wondering: is it
possible to add the markup for accesskeys in the header of a document,
have it still validate and work? Right now the HTML is inserted
within the <head> element and it looks a little funky in my text
editor.
As a test, I tried modifying your code to use the public_theme_footer
hook instead, but I still had no luck getting my accesskeys to work.
Yes, I can see that the markup is being sent to the page properly, but
my browser (Firefox 3.5) isn't working w/ the access keys. Any ideas?
Otherwise, I like what you've done with the plugin configuration and
saving options to the database.. very nice!
Dave