hemn

hemn supports this item

Supported

861 comments found.

Unden621

Unden621 Purchased

I’m requesting a complete refund. Please see my review.

You import ALL CSS SKINS from the main css file by default! Wow. That’s what params and optional arguments are for my friend, basics. Initialize the framework with these options during use, which you haphazardly do and you are duplicating this action as welll through CSS imports and JS inits. Very unoptimized to force the default behaviour this way.

/* Import Skins */
@import url('../dark-skin/skin.css');
@import url('../light-skin/skin.css');
@import url('../parade-skin/skin.css');
@import url('../metro-black-skin/skin.css');
@import url('../metro-white-skin/skin.css');
@import url('../mac-skin/skin.css');
@import url('../smooth-skin/skin.css');
@import url('../flat-dark-skin/skin.css');

You also hard-coded parent folder paths. (-‸ლ)

Please clean up your JS code. You are dumping a great deal of info into the browser console that should NOT be there in a released codebase (JS or other). Never, ever release code that dumps debug/info into the console. At the very least, only allow this behaviour if there is something like:

debug : true

in the init options.

I could go on but I don’t want to waste my time. Please update your coding skillset before releasing software products of any type.

hemn

hemn Author

Hi,
Thanks for using iLightBox.

I’m waiting for your request.

Warm Regards

by
by
by
by
by
by