Remove the date format and collapse depth options from the options passed to replaceComments and move them to options.js instead. Move comment header time creation out of ExtCommentComponent and into an option.
The time format option is more of a pseudo-option, in that it doesn't have a value that it uses itself, but instead uses the values of other options to format the time. For example, there may be an option to use 24 hour time, and the time format option will check this when filling in the time fields. This means that the other time related options will just be values holders that don't have to handle anything themselves.
Performance:
The collapse depth option runs in single digit milliseconds.
The time format option takes about 500ms on the ivermectin post, but this shouldn't be a net increase in lag as it just contains functionality that was previously in ExtCommentComponent.
Remove the date format and collapse depth options from the options passed to
replaceComments
and move them tooptions.js
instead. Move comment header time creation out ofExtCommentComponent
and into an option.The time format option is more of a pseudo-option, in that it doesn't have a value that it uses itself, but instead uses the values of other options to format the time. For example, there may be an option to use 24 hour time, and the time format option will check this when filling in the time fields. This means that the other time related options will just be values holders that don't have to handle anything themselves.
Performance:
ExtCommentComponent
.