[MYFACES-2858] pointless oamsubmit inline rendering (patch available)

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.2-SNAPSHOT
• Fix Version/s: 2.0.2
• Assignee: Jakob Korherr
• Reporter: Werner Punz
• Created: 04/Aug/10 10:56
• Updated: 16/Sep/10 04:11
• Resolved: 23/Aug/10 10:11

Description

We have had several functions rendered inline for ages, namely appendHiddenInput oamSubmit, the autoscroll stuff etc...
I personally think the rendering of those functions as inline scripts is pointless, blows up the browsers tremendously and
prevents that the affected scripts can be browser cached.
A quick look at the code revealed that there is basically nothing which would prevent to externalize the scripts. My main problem is where to we handle the auto append code.
My personal guess is we probably simply should add it as a resource definitions to the commandLink, Button etc.. renderers, any ideas regarding this?