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.

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

In-page JavaScript links are treated as external links #915

Closed
toupeira opened this issue Mar 23, 2018 · 2 comments
Closed

In-page JavaScript links are treated as external links #915

toupeira opened this issue Mar 23, 2018 · 2 comments

Comments

@toupeira
Copy link

I recently upgraded Zeal from 0.4 to 0.6 (using the Debian package) and noticed in the Rails docset that the "show" links to toggle the source for a method stopped working and now trigger the new dialog for opening external links:

selection_001

The href of these look like javascript:toggleSource('method-i-change_column_source'). Other docsets using onclick might have the same issue.

It looks like this bug was introduced with the changes in https://github.com/peterhoeg/zeal/commit/c2ca28fbde0cd42f97204e712c519917cd05419b.

I also tried changing the link behaviour to "Open in Zeal", but then nothing happens if I click JS links.

@toupeira
Copy link
Author

@trollixx thanks for the fix :)

@lock
Copy link

lock bot commented Oct 14, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for a related request.

@lock lock bot locked and limited conversation to collaborators Oct 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

2 participants