Issue 22150: deprecated-removed directive is broken in Sphinx 1.2.2 (original) (raw)
Created on 2014-08-06 04:20 by berker.peksag, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | |||
|---|---|---|---|
| File name | Uploaded | Description | Edit |
| deprecated-removed.diff | berker.peksag,2014-08-06 04:20 | ||
| broken-deprecated-removed.png | berker.peksag,2014-08-06 04:21 | ||
| broken-deprecated-removed-2.png | berker.peksag,2014-08-06 04:22 |
| Messages (8) | ||
|---|---|---|
| msg224902 - (view) | Author: Berker Peksag (berker.peksag) *  |
Date: 2014-08-06 04:20 |
| The attached patch fixes the problem for me. Tested with Firefox 30.0 and Chrome 35.0.1916.153 on Ubuntu. | ||
| msg224904 - (view) | Author: Ezio Melotti (ezio.melotti) * |
Date: 2014-08-06 04:54 |
| Does this fix the extra margin visible in broken-deprecated-removed-2.png? Is there something in the HTML that causes the extra margin (e.g. an empty ? If that is the reason I think fixing the HTML is better. | ||
| msg224908 - (view) | Author: Berker Peksag (berker.peksag) * |
Date: 2014-08-06 05:02 |
| > Does this fix the extra margin visible in broken-deprecated-removed-2.png? Yes. > Is there something in the HTML that causes the extra margin (e.g. an empty ? I checked the HTML output first, but it was fine. Here's the HTML output of the deprecate-removed directive without the patch: https://gist.github.com/berkerpeksag/e303f36d69e02f926676 | ||
| msg224919 - (view) | Author: Ezio Melotti (ezio.melotti) * |
Date: 2014-08-06 09:42 |
| If the HTML output was fine, what is the purpose of the change in pyspecific.py? | ||
| msg224927 - (view) | Author: Berker Peksag (berker.peksag) * |
Date: 2014-08-06 11:50 |
| Sorry, I forgot to explain that part of the patch. See broken-deprecated-removed.png. The "Deprecated since version 3.5, will be removed in 3.6" paragraph should be in the first line, not in the second line (like the deprecated directive). | ||
| msg224958 - (view) | Author: Ezio Melotti (ezio.melotti) * |
Date: 2014-08-06 18:31 |
| LGTM. | ||
| msg225676 - (view) | Author: Roundup Robot (python-dev)  |
Date: 2014-08-22 15:25 |
| New changeset b834b1fa2aab by Berker Peksag in branch '3.4': Issue #22150: Fix deprecated-removed directive in Sphinx 1.2. http://hg.python.org/cpython/rev/b834b1fa2aab New changeset 8f635f77e7e9 by Berker Peksag in branch 'default': Issue #22150: Fix deprecated-removed directive in Sphinx 1.2. http://hg.python.org/cpython/rev/8f635f77e7e9 | ||
| msg225677 - (view) | Author: Berker Peksag (berker.peksag) * |
Date: 2014-08-22 15:27 |
| Thank you for your review, Ezio! |
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:06 | admin | set | github: 66348 |
| 2014-08-22 15:27:29 | berker.peksag | set | status: open -> closedresolution: fixedmessages: + stage: commit review -> resolved |
| 2014-08-22 15:25:11 | python-dev | set | nosy: + python-devmessages: + |
| 2014-08-07 10:03:44 | berker.peksag | set | assignee: berker.peksag |
| 2014-08-06 18:31:50 | ezio.melotti | set | messages: + stage: patch review -> commit review |
| 2014-08-06 11:50:32 | berker.peksag | set | messages: + |
| 2014-08-06 09:42:50 | ezio.melotti | set | messages: + |
| 2014-08-06 05:02:26 | berker.peksag | set | messages: + |
| 2014-08-06 04:54:33 | ezio.melotti | set | nosy: + ezio.melottimessages: + |
| 2014-08-06 04:22:04 | berker.peksag | set | files: + broken-deprecated-removed-2.png |
| 2014-08-06 04:21:56 | berker.peksag | set | files: + broken-deprecated-removed.png |
| 2014-08-06 04:20:43 | berker.peksag | create |