ah my bad.. max 100 is probably fine to leave as-is


On Tue, May 13, 2014 at 4:39 PM, Andy Tuba <notifications@github.com> wrote:

> In lib/modules/context.js:
>
> > +			}
> > +			if ((RESUtils.pageType() === 'inbox') && !isNaN(this.options.defaultContext.value) && this.options.defaultContext.
> value >= 0) {
> > +				this.setDefaultContext();
> > +			}
> > +		}
> > +	},
> > +	addViewFullContext: function() {
> > +		if(location.search !== "?context=10000") {
> > +			var pInfobar = document.querySelector(".infobar:not(#searchexpando) p");
> > +			if(pInfobar) {
> > +				pInfobar.innerHTML += '&nbsp;<a href="?context=10000">view the full context</a>&nbsp;→';
> > +			}
> > +		}
> > +	},
> > +	setDefaultContext: function() {
> > +		$(".bylink").each(function() {
>
> well, this function is only called on inbox. so max .. 100? but it
> wouldn't hurt to chunk
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/honestbleeps/Reddit-Enhancement-Suite/pull/1052/files#r12610307>
> .
>