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

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

Right, it's just DOM manipulation and certainly could be done without the DOM 
helper.  However, it would be extremely tedious and verbose code.  I wrote it 
like this for another project I'm working on and thought I would contribute it 
back.  You're welcome to change it to remove the dependency, but I 
unfortunately don't have time to myself.  And like I said, the resulting code 
wouldn't be fun or pretty, so I'm not sure you'd wan to.

> 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
>
>         Attachments: patch.zip
>
>
> [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