twinstone / tdi

Turbocharged DOM Infusion
Apache License 2.0
6 stars 1 forks source link

TDI infusing parameter #16

Open centi opened 7 years ago

centi commented 7 years ago

Shouldn't we rename the "_infuse" parameter to "_tdi"?

My suggestion is to remove it with a possibility to force the old behavior by some JS parameter/configuration.

_infuse does no longer mean just infusion. And this may lead to confusion.

mishal commented 7 years ago

What is old the behavior?

Could this be setup with something like TDI.Config.setOption() ?

The config object could be used to configure more options for TDI... Another option could be the loading class name for involved elements(which is now hardcoded).

centi commented 7 years ago

This issue is a copy&paste from the original tracker. We developed TDI for our internal system called "Infusion". This is the reason for the GET parameter name. Old behaviour means, that we should have some way to maintain backward compatibility for our internal system (set the parameter name back to _infuse).

I give a low priority for this.