Remove code that adjusts navigation margin

[armsnyder]

Hi. Thanks for this beautiful theme. I however noticed something odd, which is that this adjustForScrollbar function does not seem to be doing anything under regular usage, but it is getting in my way when I try to customize my home page. I'll explain.

The code adds a ~15px margin to the right navigation arrow then the #body-inner height is near the same height as #body. The intent appears to be to detect when the scrollbar is present on-screen, but because #body is a div that contains #body-inner, and therefore content added to #body-inner affects the height of both divs, this condition will continue to always evaluate to false. I checked this on pages that required scrolling, and sure enough the margin of the right-nav was still 0.

So for the above reason I believe the code can be removed, since it doesn't seem to be serving its intended purpose. Please let me know if I misevaluated the purpose.

The reason I came across this is that I am trying to customize my home page in a way that it creates a banner that is flush with the top of the page. In doing so I have removed the 83px margin, which is now triggering this code to add a margin to the left nav, and it looks wrong.

[matalo33]

I agree, this doesn't seem to be providing much benefit. I think it can come out too.

Thanks for your submission.

[McShelby]

With the last patch there are no more references to getScrollBarWidth() left in the code. So codewise this function could be removed, too.

The only thing that speaks against it is the fact that this function wasn't wrapped in an IIFE and therefore could be accessed by any user. In this case, the function should be deprecated and been removed in 3.0.