I think it's better to describe mix-width property in media queries and the value as unset as this PR.
Reasons
min-width and max-width of div.body are defined as theme options of basic theme. Because Alabaster inherit basic theme, end user can change the value like bellow. If static value is defined in alabaster.css, these options will be meaningless.
I think this sphinx feature (and basic theme) is assumed to be browsed on only PC browser and there is no need to have fixed value to these property on smartphone.
About other changes
When I tested this changes, I noticed that the sidebar width is not fit to widow size.
I think padding value of sidebar should be 30px (same of outer margin). Unfortunately, I couldn't understand why the width is 102.5%, but It looks good also to be 100% at my testing.
solve https://github.com/bitprophet/alabaster/issues/139 and improve https://github.com/bitprophet/alabaster/pull/140
I noticed the bug and investigated implementation of Sphinx. Related feature was added in Sphinx v1.7.0 and implemented on https://github.com/sphinx-doc/sphinx/pull/4376 .
I think it's better to describe
mix-width
property in media queries and the value asunset
as this PR.Reasons
min-width and max-width of div.body are defined as theme options of basic theme. Because Alabaster inherit basic theme, end user can change the value like bellow. If static value is defined in alabaster.css, these options will be meaningless.
I think this sphinx feature (and basic theme) is assumed to be browsed on only PC browser and there is no need to have fixed value to these property on smartphone.
About other changes
When I tested this changes, I noticed that the sidebar width is not fit to widow size.