Skip to content

ISSN hyperlink#5

Open
Edinbro wants to merge 5 commits into
matthieu-vergne:masterfrom
Edinbro:master
Open

ISSN hyperlink#5
Edinbro wants to merge 5 commits into
matthieu-vergne:masterfrom
Edinbro:master

Conversation

@Edinbro
Copy link
Copy Markdown

@Edinbro Edinbro commented Feb 18, 2021

I'm sorry for the 1,307 additions and 1,305 deletions, but the code should work, I tried on a couple of my projects. I haven't tried to find a solution though for the case that there is a comma between two or more ISSNs.

Cheers,
Frederic

@matthieu-vergne
Copy link
Copy Markdown
Owner

matthieu-vergne commented Feb 18, 2021

This is mainly because of changes in the white spaces (space, new line, tab, etc.). If you ignore them, there is really not much:
https://github.com/matthieu-vergne/LaTeX/pull/5/files?diff=unified&w=1

Since there is also a character which is badly encoded in your version, I assume this is because you modified the files with a tool which does a bad job, especially with no management of UTF8 characters. It might be you edited the files with an editor in Microsoft Windows, which may have make a mess of the newline characters, thus making a mess of the whole file.

If you understand what I mean, I would be glad that you fix it.

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