Skip to content

Fix footnotes. #188

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

Merged
merged 2 commits into from
Jul 23, 2016
Merged

Fix footnotes. #188

merged 2 commits into from
Jul 23, 2016

Conversation

bwhelm
Copy link
Contributor

@bwhelm bwhelm commented Jul 23, 2016

Commit message below. (3) is partly a matter of taste, and I'd be happy to change
or remove that if you'd like. Other comments, of course, welcome.


  1. Skip over brackets inside footnotes. Otherwise things like PCites
    and other bracketed comments wrongly match and end the footnote
    early.
  2. Allow for inline formatting inside footnotes.
  3. Change highlighting of footnotes. (Having footnotes be Comments means they
    get italicized. This is not desirable since it's possible for italics
    to show up in footnotes.)
  4. Allow for footnotes to appear in lists.

1. Skip over brackets inside footnotes. Otherwise things like PCites
   and other bracketed comments wrongly match and end the footnote
   early.

2. Allow for inline formatting inside footnotes.

3. Change highlighting of footnotes. (Having footnotes be Comments means they
   get italicized. This is not desirable since it's possible for italics
   to show up in footnotes.)

4. Allow for footnotes to appear in lists.
@fmoralesc
Copy link
Member

3 seems dependent on the colorscheme. The one I use (based on molokai), doesn't use italics for Comment, and worse, it uses color for Type. I would rather have this be a variation of Normal with less contrast (might use the functions in https://github.com/vim-pandoc/vim-pandoc-syntax/blob/master/autoload/pandoc/syntax/color.vim).

@bwhelm
Copy link
Contributor Author

bwhelm commented Jul 23, 2016

Yes -- I see it's working when I change away from Solarized theme. Shall I simply revert the styling changes?

@fmoralesc
Copy link
Member

Sure ;)

@bwhelm
Copy link
Contributor Author

bwhelm commented Jul 23, 2016

Done.

@fmoralesc fmoralesc merged commit e9fb387 into vim-pandoc:master Jul 23, 2016
@bwhelm bwhelm deleted the footnotes branch July 23, 2016 18:58
davenquinn added a commit to davenquinn/vim-pandoc-syntax that referenced this pull request Aug 10, 2018
* master: (70 commits)
  Three improvements to LaTeX highlighting (vim-pandoc#216)
  Add `pandocAmpersandEscape` syntax for &...; special chars (vim-pandoc#205)
  Fix unicode with sub- or superscripts. (vim-pandoc#202)
  Optionally disable definition lists (vim-pandoc#212)
  Add support for tab-indented code blocks (vim-pandoc#207)
  Never conceal \ in \newcommand... (vim-pandoc#197)
  BufFilePRE => BufFilePre (vim-pandoc#199)
  Fixes for parenthetical citations (vim-pandoc#189)
  Fix footnotes. (vim-pandoc#188)
  Fix reference links
  Fix automatic links by restricting characters in mail links.
  Check 'termguicolors' instead of 'nvim'
  fix typo in doc
  fix for upstream issue vim-pandoc#177
  fix typo
  Conceal char for end of code block
  Add cchar for block end
  added endash and emdash support to syntaxt
  fix for neovim
  Use custom syntax group for operators
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants