Make YUI global name configurable about yui3 HOT 16 OPEN

var MyNameSpace = YUI;

from yui3.

what if I already have something called YUI on my page?

Then when I load the YUI library, it will override it.

One possible use case is being able to support multiple versions of YUI on the same page.

from yui3.

I guess I can hack it by globally replacing 'YUI' with 'MyNamespace' in the source code. But it's still a hack.

from yui3.

You can use shifter to make a custom build:

cd yui3/src
shifter --walk --replace-yuiglobalvar=FOO

Then you can do one of two things:

Hand modify the global in build/yui/yui.js

OR

<script>
var exports = {};
</script>
<script src="/path/to/your/yui.js"></script>
<script>
var FOO = exports.YUI;
</script>

from yui3.

How is shifter going to find @YUIGLOBALVAR@? I can't find this place holder anywhere in the source code.

from yui3.

When shifter builds the lib with that command, it will wrap the code
in YUI.add. YUI is a variable in shifter.

from yui3.

Gotcha.

@YUIGLOBALVAR@ is in shifter, and the remaining offender is src/yui/js/yui.js

Will you consider a patch to make the YUI global variable configurable?

Anyway, thank you for your help.

from yui3.

Yup, we should make that configurable.

There may be some edge cases in other modules that use the YUI var too.

I'll also keep that in mind with the seed rewrite I'm working on.

from yui3.

Hi Dave,

I started the work on making YUI global variable configurable.

Can you let me know if I am headed down the right road? Thanks.

trungpham@c939272

from yui3.

Changing them from YUI to Y is not the proper thing. Changing them from YUI to @YUIGLOBALVAR is the best thing.

from yui3.

Is it possible to use an alternative marker to indicate a place holder variable? Because using the @ sign confuses most IDEs since variable name cannot start with the @ sign.

Perhaps, $YUIGLOBALVAR$?

from yui3.

Just hold off on this for now, we discussed this on a Hangout on
Friday. I think I have the proper way to fix this lined out in my new
seed changes.

from yui3.

Awesome!

from yui3.

@davglass hi Dave, is there any update on this?

from yui3.

Any update on this ?

from yui3.

@alaindresse, @trungpham

It sucks this issue was not addressed, and left to rot. Unfortunately, I don't believe there are any current plans for this feature. However, this may be resolved next sprint (starting Nov. 15th).

from yui3.