Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Improve minifyability #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

meghuizen
Copy link

PgwModal now gives problems when minifying in ASP.NET Bundles. This is because ASP.NET bundles also change the function variable names.

However in 1 function you use $compile, which is detected by AngularJS by name. Because AngularJS detects the $compile service by name, the code stops working when the minifyer changes the variable name to, let's say 'r'.

AngularJS adds the possibility to name the parameter using a string (which the minifyer leaves unharmed).
So applying this change allows the code to be minified correctly.

PgwModal now gives problems when minifying in ASP.NET Bundles. This is because ASP.NET bundles also change the function variable names. 

However in 1 function you use $compile, which is detected by AngularJS by name. Because AngularJS detects the $compile service by name, the code stops working when the minifyer changes the variable name to, let's say 'r'. 

AngularJS adds the possibility to name the parameter using a string (which the minifyer leaves unharmed). 
So applying this change allows the code to be minified correctly.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant