ProfessionalWiki / ModernTimeline

resources/modernTimeline.js

( function ( mw ) {

    modernTimelineLog( 'Timeline script' );

    function modernTimelineLog( message ) {
        if( mw.config.get( 'debug' ) ) {
            console.log( message );

console.log looks like debug code. Are you sure you do not want to remove it?

        }
    }

    mw.loader.using( [ 'ext.modern.timeline' ] ).done( function () {
        modernTimelineLog( 'Loaded timeline JS' );

        for( var timelineId in window.modernTimeline ) {
            if( window.modernTimeline.hasOwnProperty( timelineId ) ) {
                modernTimelineLog( 'Initializing timeline "' + timelineId + '"' );

                var timelineJson = window.modernTimeline[timelineId];
                new TL.Timeline( timelineId, timelineJson, timelineJson.options );

The variable TL seems to be never declared. If this is a global, consider adding a /** global: TL */ comment.

The object created with new TL.Timeline(timelineId, timelineJson, timelineJson.options) is not used but discarded. Consider invoking another function instead of a constructor if you are doing this purely for side effects.

                modernTimelineLog( 'Done initializing timeline "' + timelineId + '"' );
            }
        }
    } );

}( mediaWiki ) );

The variable mediaWiki seems to be never declared. If this is a global, consider adding a /** global: mediaWiki */ comment.