#603 ✓resolved
Bryan Larsen

IE7: select-many inside input-many causes problems

Reported by Bryan Larsen | January 16th, 2010 @ 05:56 PM | in Hobo 1.0 - Final

To reproduce:

  • have a select-many inside of an input-many
  • display the new-page in IE7
  • click on "Create" without hitting the "+" button -- ie create the item with zero elements

You should get validations errors. Looking at the log, you will notice that the select-many submits a blank string. None of the other fields in the input-many submit anything.

Comments and changes to this ticket

  • Bryan Larsen

    Bryan Larsen January 16th, 2010 @ 06:04 PM

    Here's a possible fix:

    diff --git a/hobo/rails_generators/hobo_rapid/templates/hobo-rapid.js b/hobo/rails_generators/hobo_rapid/templates/hobo-rapid.js
index 80d99e1..c1d1ca2 100755
--- a/hobo/rails_generators/hobo_rapid/templates/hobo-rapid.js
+++ b/hobo/rails_generators/hobo_rapid/templates/hobo-rapid.js
@@ -571,7 +571,7 @@ new HoboBehavior("ul.input-many", {
 
   initialize: function(ul) {
       // disable all elements inside our template, and mark them so we can find them later.
-      $$(".input-many-template input:enabled, .input-many-template select:enabled, .input-many-template textarea:enabled, .input-many-template button:enabled").each(function(input) {
+      $$(".input-many-template input:enabled, .input-many-template select:enabled, .input-many-template textarea:enabled, .input-many-template button:enabled, .input-many-template input.proto-hidden").each(function(input) {
           input.disabled = true;
           input.addClassName("input_many_template_input");
       });

    This fix is very specific -- there's probably a more generic fix. The question is why prototype.js doesn't capture input.proto-hidden in IE7. Does it consider hidden elements to be disabled?

  • Bryan Larsen

    Bryan Larsen January 16th, 2010 @ 09:52 PM

    • State changed from “new” to “resolved”

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

People watching this ticket

Tags

Referenced by

Pages