Custom Common JavaScript inclusion #108

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Member

fgrande commented Jun 22, 2012

Hi guys.
This is just an idea, that's why I made it in this repo.

What about including application-wide javascript files through sugar_config ?

Ciao
FabioG

Contributor

jmertic commented Jul 9, 2012

Couldn't you do the same thing by putting whatever in the custom/themes/default/js/style.js file?

Member

fgrande commented Jul 10, 2012

You are right, but, please, consider also concurrency on packages : two
different packages working on custom style.js file would do a mess.
Maybe, an alternative, would be the integration in the Ext framework....

On 07/09/2012 09:32 PM, John Mertic wrote:

Couldn't you do the same thing by putting whatever in the custom/themes/default/js/style.js file?


Reply to this email directly or view it on GitHub:
#108 (comment)

Contributor

jmertic commented Jul 10, 2012

That would be a good approach. Could you put together a pull around that?

Thanks!

John Mertic
jmertic@sugarcrm.com

On 7/10/12 2:44 AM, "Fabio GRANDE"
<reply+i-5215195-8fddcd68b491d68b47f251b47d4caae9d0db66bb-296737@reply.gith
ub.com> wrote:

You are right, but, please, consider also concurrency on packages : two
different packages working on custom style.js file would do a mess.
Maybe, an alternative, would be the integration in the Ext framework....

On 07/09/2012 09:32 PM, John Mertic wrote:

Couldn't you do the same thing by putting whatever in the
custom/themes/default/js/style.js file?


Reply to this email directly or view it on GitHub:
#108 (comment)


Reply to this email directly or view it on GitHub:
#108 (comment)

jmertic closed this Feb 28, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment