Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

[PATCH] Allow to set dataType in jquery-ujs

#4603

Currently there is no way to set dataType in jquery-ujs, it's set to 'script' by default. It would be nice to have ability to do that by setting data-type HTML attribute with default set to 'script'.

Reported by Piotr Sarnacki · May 15th, 2010 @ 04:52 PM

State: invalid
Milestone: none
Assigned to: José Valim José Valim
Importance: Low

Activity

  1. Santiago Pastorino
    Santiago Pastorino
    • Tag changed from jquery-ujs patch to jquery-ujs patch, bugmash

    May 16th, 2010 @ 12:14 AM

  2. Santiago Pastorino
    Santiago Pastorino
    • Milestone cleared.
    • Tag changed from jquery-ujs patch, bugmash to jquery-ujs patch
    • State changed from new to open

    +1

    May 16th, 2010 @ 12:16 AM

  3. Santiago Pastorino
    Santiago Pastorino

    Hehe, +1 for the idea the patch is wrong

    May 16th, 2010 @ 12:18 AM

  4. Piotr Sarnacki
    Piotr Sarnacki

    What's wrong with it? I've used it in my app, works great :)

    May 16th, 2010 @ 12:26 AM

  5. Santiago Pastorino
    Santiago Pastorino
    • Milestone cleared.
    • State changed from open to invalid

    Sorry i should be a bit asleep this is for http://github.com/rails/jquery-ujs
    You should report this kind of issues in this project issue page.
    Also is already reported http://github.com/rails/jquery-ujs/issues#issue/3 you can provide your patch there.
    Thanks.

    May 16th, 2010 @ 12:27 AM

  6. Santiago Pastorino
    Santiago Pastorino

    I've added your patch to the comments there

    May 16th, 2010 @ 12:30 AM

  7. Piotr Sarnacki
    Piotr Sarnacki

    I think I was a bit asleep too ;-) I haven't noticed issue 3 and added my own: http://github.com/rails/jquery-ujs/issues#issue/7, but then I asked Jose Valim about where to put such tickets. He said that here. Anyway, sorry for the confusion :)

    May 16th, 2010 @ 08:54 AM