There are some problems with the overall form theming
* hook_forms() still needs to be supported, as the fix didn't make it into D7
* Link buttons are ordinary links for now.
* Ugly inclusion of forms from themes is probably the issue.
* Not all the forms up on the top are working right.
* Multiple forms on a page still needs work.
* Buttons that should be hidden are not hidden
* "Checkout at amazon" on cart page is a link; the button code has debug code in it. Make it work.

CommentFileSizeAuthor
#3 amazon_store.hide_no_js_989108_03.patch1.89 KBrfay
#1 amazon_store.linkbuttons_989108_01.patch2.94 KBrfay

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
StatusFileSize
new amazon_store.linkbuttons_989108_01.patch2.94 KB

Here's the patch and the commit for the linkbutton problems: http://drupal.org/cvs?commit=459638

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

What remains to do is
* Not all the forms up on the top are working right.
* Buttons that should be hidden are not hidden

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
StatusFileSize
new amazon_store.hide_no_js_989108_03.patch1.89 KB

This one hides elements that have amazon-store-no-js as class. Commited: http://drupal.org/cvs?commit=459646

Now, to make the top forms work. That's the last thing in this issue.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Active » Fixed

It looks to me like most of these work. Basic page behavior is OK. Anybody want to do some more significant testing?

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.