Skip to content
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.

Sign up for GitHub

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

Jump to bottom

URL cut off when it contains % symbol #82

Closed
chs20 opened this issue Aug 21, 2023 · 0 comments
Closed

URL cut off when it contains % symbol #82

chs20 opened this issue Aug 21, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@chs20
Copy link

chs20 commented Aug 21, 2023

Hi, when an url contains a % symbol, it is treated as a comment and is cut off. For example

\url{https://www.example.com/hello%20world}

becomes

\url{https://www.example.com/hello
@jponttuset jponttuset added the bug Something isn't working label Aug 21, 2023
dylduhamel added a commit to dylduhamel/arxiv-latex-cleaner that referenced this issue Jan 17, 2024
@dylduhamel
Fixed issue google-research#82: Broke text into segments to ignore an…
6cf3e7c 
…y % inside of \url{} link.
dylduhamel added a commit to dylduhamel/arxiv-latex-cleaner that referenced this issue Jan 17, 2024
@dylduhamel
Fixed issue google-research#82: Broke text into segments to ignore an…
01267e5 
…y % inside of \url{} link.
dylduhamel added a commit to dylduhamel/arxiv-latex-cleaner that referenced this issue Jan 17, 2024
@dylduhamel
Fixed issue google-research#82: % inside of url link is not treated a…
03e0bb0 
…s comment
@dylduhamel dylduhamel mentioned this issue Jan 17, 2024
Merged
jponttuset pushed a commit that referenced this issue Jan 17, 2024
@dylduhamel
Fixed issue #82: % inside of url link is not treated as comment (#87)
20a98ee
jponttuset added a commit that referenced this issue Jan 17, 2024
@jponttuset
Revert "Fixed issue #82: % inside of url link is not treated as comme…
4b7168b 
…nt (#87)"

This reverts commit 20a98ee.
@dylduhamel dylduhamel mentioned this issue Jan 17, 2024
Merged
jponttuset pushed a commit that referenced this issue Jan 17, 2024
@dylduhamel
Issue 82 fix (#88)
ada0c67 
* Fixed issue #82: % inside of url link is not treated as comment

* test_remove_comments_inline tests pass
@merajhashemi merajhashemi mentioned this issue Mar 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jponttuset @chs20