smartypants: closing double quote not recognized

alexander smishlajev avataralexander smishlajev created an issue

The smartypants module does not recognize closing double quote immediately preceded with a roled text and followed by a punctuation character, like comma or closing bracket.

Example:

A quote from 'SPECTRUM': "The word :term:`say`
is the same as the word :term:`tell`".

Comments (5)

  1. Matt Williams

    I can confirm this with 1.0b2. The closing quote is correctly outside of the <em></em> (caused by the :term: role) but it is displayed as an opening quote.

    The same thing happens if you do:

    A quote from 'SPECTRUM': "The word *say*
    is the same as the word *tell*".
    
  2. Georg Brandl
    • changed status to open
    • changed component to html
    • changed milestone to 1.x

    Yes, unfortunately. smartypants is not applied across node boundaries, so it only sees a piece of text starting with a quote. I'm not sure how to fix that.

  3. Log in to comment
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.