Skip to content
This repository has been archived by the owner on Jun 3, 2020. It is now read-only.

Luna (catalogue) links on Talk not working #276

Open
VVH opened this issue Jan 4, 2016 · 12 comments
Open

Luna (catalogue) links on Talk not working #276

VVH opened this issue Jan 4, 2016 · 12 comments

Comments

@VVH
Copy link
Contributor

VVH commented Jan 4, 2016

Links from images to the Folger catalogue (viewable under 'i' icon in the metadata on Talk) are not working.The links work fine in the transcription interface, but not from Talk. I think @camallen and @brian-c might know what's up, but flagging @simoneduca as well.
screen shot 2016-01-04 at 16 21 33

@camallen
Copy link
Contributor

camallen commented Jan 4, 2016

@simoneduca you fixed this in the project right?

@simoneduca
Copy link
Contributor

@camallen that rings a bell, but I think it was a PFE problem, not of SW. Something to do with a html parser rendering ~ as superscript. Let me check.

@brian-c
Copy link

brian-c commented Jan 4, 2016

This is the markdown-it automatic linker conflicting with the subscript syntax (~) plugin. I wonder if it can be fixed by messing with the plugin's activation order.

@simoneduca
Copy link
Contributor

@brian-c I can have a look if you point me to the right code bit :)

@brian-c
Copy link

brian-c commented Jan 4, 2016

Here's where the plugin is being included:

https://github.com/zooniverse-ui/markdownz/blob/master/src/components/markdown.jsx#L11

Looks like auto-linking is built-in (not a plugin) though, so there might not be a way to change the order these things happen in.

Might be worth checking if https://github.com/markdown-it/markdown-it-sub would accept a PR, or if it can be fixed in https://github.com/markdown-it/markdown-it

@simoneduca
Copy link
Contributor

what's the markdown-it-sub plugin needed for?

@brian-c
Copy link

brian-c commented Jan 4, 2016

That just allows people to mark text as subscript.

@VVH
Copy link
Contributor Author

VVH commented Feb 15, 2016

Hey guys, where did we get on this? If it's not fixable in PFE I will ask Folger to give us differently formatted links for the next batch of data. @simoneduca @brian-c?

@simoneduca
Copy link
Contributor

@VVH if they could reformat the links, that'd be much easier.

@camallen
Copy link
Contributor

@simoneduca, is a url like with ~ the problem, e.g. http://example.com/~/images/test.png the problem?

@simoneduca
Copy link
Contributor

yep.

@camallen
Copy link
Contributor

right - that seems like something we should support by default.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants