Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #10363 make sphinxmaketitle ruler use \linewidth not \textwidth #10365

Merged

Conversation

LuisBL
Copy link

@LuisBL LuisBL commented Apr 18, 2022

Subject: use \linewidth not \textwidth for sphinxmaketitle ruler

  • branche Others: 4.5.x

Feature or Bugfix

  • Bugfix

Purpose

sphinxmaketitle should use \linewidth not \textwidth for his ruler

Relates

@oscargus
Copy link
Contributor

oscargus commented Apr 20, 2022

Makes sense, but you have used the wrong issue number in the commit message and title. (Not sure if the CHANGES file is auto-generated or not, but same there.)

@tk0miya
Copy link
Member

tk0miya commented May 7, 2022

We have a plan to release 5.0 soon. So it's better to merge this into the 5.x branch. Could you rebase this, please?

@jfbu
Copy link
Contributor

jfbu commented May 8, 2022

Thanks! I will address the issues raised by @oscargus by force-pushing to your branch to rebase on 5.0.0 for merge, hope you don't mind.

@jfbu jfbu mentioned this pull request May 8, 2022
@jfbu jfbu force-pushed the 10363-sphinxmaketitle-should-use-linewidth-fix branch from 35a6d11 to 6487529 Compare May 8, 2022 08:21
@jfbu jfbu changed the base branch from 4.5.x to 5.x May 8, 2022 08:22
@jfbu jfbu changed the title fix #1036 make sphinxmaketitle ruler use \linewidth not \textwidth fix #10363 make sphinxmaketitle ruler use \linewidth not \textwidth May 8, 2022
@jfbu jfbu merged commit f1061c0 into sphinx-doc:5.x May 8, 2022
@jfbu
Copy link
Contributor

jfbu commented May 8, 2022

Merged !

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants