profile
viewpoint
Jay Cambron cambronjay @offerbright United States of America www.jaycambron.com Self-taught engineer, serial entrepreneur, and avid inventor. I solve complex problems with simple solutions.

cambronjay/enterprise-angular-generator 1

A collection of angular generators for enterprise applications.

cambronjay/enterprise-angular-template 1

An enterprise level angular template that implements best practices and state management for application scale.

cambronjay/gitit 1

A sample project built with Ionic 2 which consumes the GitHub API via search.

cambronjay/location-utilities 1

Coordinate formatting, location calculation utilities, and NMEA sentence parser.

cambronjay/CalendarGenerator 0

An Angular 6 and Angular Material application that generates calendars.

cambronjay/capacitor-firebase-plugin 0

Firebase SDK wrapped with capacitor.

cambronjay/CodeChallenge 0

A front-end and back-end code challenge using ionic, angular, javascript, typescript, and node.js

cambronjay/cookie 0

HTTP server cookie parsing and serialization

cambronjay/cordova-firebase-plugin 0

A Cordova plugin that wraps the Firebase SDK with an easy to use api.

cambronjay/ionic 0

Advanced HTML5 mobile development framework and SDK. Build incredible mobile apps with web technologies you already know and love. Best friends with AngularJS.

PublicEvent

issue commentautoNumeric/autoNumeric

Trailing comma, spacing, and incorrect parentheses causes build issues in an application

Essentially wherever settings.decimalPlacesRawValue or currentSettings.originalDecimalPlacesRawValue is used with Math.max there is a trailing comma and there is an issue with missing parentheses to make the order of operations more clear.

cambronjay

comment created time in 3 months

issue commentautoNumeric/autoNumeric

Trailing comma causes build issues in an application

Also there appears to be a bad spacing in the same code block. It should be this without space: settings.decimalPlacesRawValue = Math.max(Math.max(settings.decimalPlacesShownOnBlur, settings.decimalPlacesShownOnFocus) + additionalDecimalPlacesRawValue, Number(settings.originalDecimalPlacesRawValue) + additionalDecimalPlacesRawValue);

cambronjay

comment created time in 3 months

issue openedautoNumeric/autoNumeric

Trailing comma causes build issues in an application

<!-- IMPORTANT: If you do not provide a reproducible example (ie. via codepen.io), we likely won't be able to help you to the best of our abilities, and could automatically close this issue. -->

Current behavior

<Please describe here what faulty behavior you are experiencing> Trailing comma causes build issues

Expected behavior

No Trailing comma should exist in math.max

Steps to reproduce the problem

This code block in the AutoNumeric.js file has a trailing comma after the last additionalDecimalPlacesRawValue variable as bolded below settings.decimalPlacesRawValue = Math.max(Math.max(settings.decimalPlacesShownOnBlur, settings.decimalPlacesShownOnFocus) +additionalDecimalPlacesRawValue, Number(settings.originalDecimalPlacesRawValue) + additionalDecimalPlacesRawValue, );

created time in 3 months

more