Skip to content

Commit e833b3f

Browse files
authored
Fixing incorrect substring indexes (#431)
Substring indexes were all off by 1, creating an incorrect output. Also removed unnecessary `toUpperCase()` calls.
1 parent 8ffe9ce commit e833b3f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Conversions/HexToRGB.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
function hexStringToRGB (hexString) {
2-
var r = (hexString.substring(1, 3)).toUpperCase()
3-
var g = hexString.substring(3, 5).toUpperCase()
4-
var b = hexString.substring(5, 7).toUpperCase()
2+
var r = hexString.substring(0, 2)
3+
var g = hexString.substring(2, 4)
4+
var b = hexString.substring(4, 6)
55

66
r = parseInt(r, 16)
77
g = parseInt(g, 16)
@@ -11,4 +11,4 @@ function hexStringToRGB (hexString) {
1111
return obj
1212
}
1313

14-
console.log(hexStringToRGB('javascript rock !!'))
14+
console.log(hexStringToRGB('ffffff'))

0 commit comments

Comments
 (0)