roydukkey / sass-module-list

Provides more advanced list functions for Sass.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Consolidate the parameter names dealing with indexes

roydukkey opened this issue · comments

commented

Currently, I believe these are the current terms being used: $index, $from-index, $start-at, and $end-at.

As development of more functions in sass-string continues, I'm finding it $confusing to continue using $from-index, especially as I complete sass-string.last-index(). MDN does use fromIndex, but since the nature of the chosen implementation for the new Sass function searches forward not backward, I'm finding $start-at and $end-at add some necessary, at-a-glance cluing of the function's nature.

$from-index should be replaced with either $start-at or $end-at as appropriate.