Now that we have the token browser in a dialog, it seems unnecessary that we have the automatic scrolling that happens when you click a link in the token browser. I think the original need for the auto-scrolling was caused by the user needing to scroll far down the page to find the correct token. Now that tokens are in the dialog (and typically the dialog overlays the field needing the token, rather than being below it), there is a lot less scrolling involved. To really make the dialog act like a typical dialog, we should also put a max-height on the dialog, so it doesn't end up with a 2000px high dialog after expanding items. If the dialog always fits within the current viewport, no scrolling on the main window will ever be necessary.