NotificationsYou must be signed in to change notification settings
Fork32.4k
Star67.9k

Commit6ef6915

authored

gh-101845: pyspecific: Fix i18n for availability directive (GH-101846)

pyspecific: Fix i18n for availability directiveIf the directive has content, the previous code would nest paragraphnodes from that content inside a general paragraph node, which confusesSphinx and leads it to drop the content when translating. Instead, use acontainer node for the body.Also use set_source_info so that any warnings have location info.

1 parentdfc2e06 commit6ef6915Copy full SHA for 6ef6915

File tree

1 file changed

+11

-9

lines changed

Doc/tools/extensions
- pyspecific.py

1 file changed

+11

-9

lines changed

`‎Doc/tools/extensions/pyspecific.py`

Lines changed: 11 additions & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`	`fromsphinx.environmentimportNoUri`
`29`	`29`	`fromsphinx.localeimport_assphinx_gettext`
`30`	`30`	`fromsphinx.utilimportstatus_iterator,logging`
	`31`	`+fromsphinx.util.docutilsimportSphinxDirective`
`31`	`32`	`fromsphinx.util.nodesimportsplit_explicit_title`
`32`	`33`	`fromsphinx.writers.textimportTextWriter,TextTranslator`
`33`	`34`
`@@ -119,7 +120,7 @@ def run(self):`
`119`	`120`
`120`	`121`	`# Support for documenting platform availability`
`121`	`122`
`122`		`-classAvailability(Directive):`
	`123`	`+classAvailability(SphinxDirective):`
`123`	`124`
`124`	`125`	`has_content=True`
`125`	`126`	`required_arguments=1`
`@@ -139,18 +140,19 @@ class Availability(Directive):`
`139`	`140`
`140`	`141`	`defrun(self):`
`141`	`142`	availability_ref=':ref:`Availability <availability>`: '
	`143`	`+avail_nodes,avail_msgs=self.state.inline_text(`
	`144`	`+availability_ref+self.arguments[0],`
	`145`	`+self.lineno)`
`142`	`146`	`pnode=nodes.paragraph(availability_ref+self.arguments[0],`
`143`		`-classes=["availability"],)`
`144`		`-n,m=self.state.inline_text(availability_ref,self.lineno)`
`145`		`-pnode.extend(n+m)`
`146`		`-n,m=self.state.inline_text(self.arguments[0],self.lineno)`
`147`		`-pnode.extend(n+m)`
	`147`	`+'',avail_nodes,avail_msgs)`
	`148`	`+self.set_source_info(pnode)`
	`149`	`+cnode=nodes.container("",pnode,classes=["availability"])`
	`150`	`+self.set_source_info(cnode)`
`148`	`151`	`ifself.content:`
`149`		`-self.state.nested_parse(self.content,self.content_offset,pnode)`
`150`		`-`
	`152`	`+self.state.nested_parse(self.content,self.content_offset,cnode)`
`151`	`153`	`self.parse_platforms()`
`152`	`154`
`153`		`-return [pnode]`
	`155`	`+return [cnode]`
`154`	`156`
`155`	`157`	`defparse_platforms(self):`
`156`	`158`	`"""Parse platform information from arguments`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit6ef6915

File tree

1 file changed

1 file changed

`‎Doc/tools/extensions/pyspecific.py`

0 commit comments