search

dsifford / academic-bloggers-toolkit

WordPress plugin providing an all-in-one solution for effective academic blogging.
https://wordpress.org/plugins/academic-bloggers-toolkit/
GNU General Public License v3.0
120 stars 38 forks source link

Link shows target = "blank" in html text #549

Closed JanaJarecki closed 5 years ago

JanaJarecki commented 5 years ago

ABT Version: 4.13.3 PHP Version: PHP 7.0 Theme: Photoreactive Plugin List: [note: after disabling all plugins, the issue remained]

Browser: Google Chrome | 72.0.3626.121 Expected behavior: the html tags to be included in the <a href ...> issue Actual behavior: see here: http://janajarecki.com/research/ The reference shows the html tags in the text, as shown here image

Console messages: no errors in console, this is the html image

dsifford commented 5 years ago

Sorry for the delay in getting back with you @JanaJarecki.

This is indeed strange. Can you share with me the following info...

  1. What is the citation style you are using?
  2. How did you enter the reference in question that caused this problem?

Thanks! Sorry for the trouble!

glorious1 commented 5 years ago

Looks like she solved the problem, as the link shows correctly now in the page she links to.

JanaJarecki commented 5 years ago

@glorius1: The bug remains. Why you don't see it is because the site now uses "APA with doi", which you can see from the doi prefix in front of the link. See my last comment on the style.

Let me know if you have a fix and I'll switch the style back to test. I can switch back to the bug temporarily in coordination if it helps debugging.

glorious1 notifications@github.com schrieb am Sa., 16. März 2019, 19:26:

Looks like she solved the problem, as the link shows correctly now in the page she links to.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dsifford/academic-bloggers-toolkit/issues/549#issuecomment-473572547, or mute the thread https://github.com/notifications/unsubscribe-auth/AJWtn-W8yQnef1e6RnR_y88eMbXKpd86ks5vXTdtgaJpZM4bi4o6 .

dsifford commented 5 years ago

@JanaJarecki If you could, please address the questions I asked and I'll look into this further.

JanaJarecki commented 5 years ago

Hi, I did answer you via the email answering, the message seems to have gotten lost, I'll copy-paste my answer from 14. March:

HI! thank you for the reply and taking the time. Sure:

I entered the reference via doi. The style is "American Psychological Association 6th edition". Note, the error disappears when the style is switched to "American Psychological Association 6th edition ("doi:" DOI prefix)".

Best Jana

dsifford commented 5 years ago

Thanks for clarifying. I'll check into this further soon.

dsifford commented 5 years ago

@JanaJarecki Sorry for the delay.

Version 5.0.4 of the plugin that I'm about to push out should fix this issue.