MrDogeBro / sphinx_rtd_dark_mode

Adds a toggleable dark mode to the Read the Docs theme for Sphinx.
sphinx-rtd-dark-mode.vercel.app
MIT License
49 stars 15 forks source link

Add Option to Use Full Content Width #10

Open Zethson opened 3 years ago

Zethson commented 3 years ago

Hey,

I was waiting for this forever and I hope that you will maintain this!

Anyways,

.wy-nav-content {
    max-width: none;
}

is always required as custom css to use the full width of the screen. What do you think about making this a default?

MrDogeBro commented 3 years ago

Could you show a picture of the difference? This extension is for dark mode tho, and therefore if its not related to theming, it may not be added. We would have to see.

Zethson commented 3 years ago

It goes slightly beyond theming, so I guess not. I just patch my own css

MrDogeBro commented 3 years ago

Ok, could you show a picture tho? There is a potential that it could always just be added as an extra option that can be enabled in the config.

Zethson commented 3 years ago

@MrDogeBro look at this for example: https://sc-toolbox.readthedocs.io/en/latest/installation.html

It uses the full width of the screen. The default of the theme only uses like 400 pixels or something (I forgot)

MrDogeBro commented 3 years ago

Yeah I see — thats what I thought you were talking about. What do you think about just a bool config option for toggling in (off by default)?

Zethson commented 3 years ago

Yeah sure, why not.