[ 
https://issues.apache.org/jira/browse/SHINDIG-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12771315#action_12771315
 ] 

Benjamin McCann commented on SHINDIG-1188:
------------------------------------------

I have a JS implementation that creates the preferences dialog using the 
functions in this library:
http://code.google.com/p/doctype/source/browse/trunk/goog/dom/dom.js

However, it does not support the user preference of type list.  Instead of 
giving the add and x buttons that you get in iGoogle, you just get an input of 
type text.  This basically means you can only input one item (unless you know 
to separate them using a pipe).  It'd probably be pretty easy to extend this to 
accept commas and then replace them with pipes when submitting the form.  Given 
that the current implementation seems to be worse 
(https://issues.apache.org/jira/browse/SHINDIG-1212), I'm not sure list support 
is that big of an issue.  If including the library above and having somewhat 
bad list support is okay, then I can clean it up and submit a patch.

> Implement Gadget Prefs UI
> -------------------------
>
>                 Key: SHINDIG-1188
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-1188
>             Project: Shindig
>          Issue Type: New Feature
>          Components: Javascript 
>         Environment: 
> http://svn.apache.org/repos/asf/incubator/shindig/trunk/javascript/container/gadgets.js
>            Reporter: Benjamin McCann
>             Fix For: 1.1-BETA4
>
>
> [Original description below]
> Current shindig code uses www.gmodules.com to generate UserPrefs UI.  Shindig 
> should have code that implements this that does not depend on that host.
> ---- 
> It takes several seconds to open the gadgets dialog.
> In gadgets.js the following code should be replaced:
> http://gmodules.com/ig/gadgetsettings
> With a call not requiring a 301 redirect:
> http://www.gmodules.com/ig/gadgetsettings
> You can use a redirect checker to verify this change should be made:
> http://www.internetofficer.com/seo-tool/redirect-check/
> Thanks to Carla Geisser, Google SRE for suggesting this change.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to