Revert "Work around OS X issue with dropped #fragment in launched URLs"

This reverts commit 80b658e247.
This commit is contained in:
Thom Chiovoloni 2018-01-12 17:58:28 -05:00 committed by Fabian Homborg
parent 56604f598e
commit 7dae40a240
2 changed files with 14 additions and 17 deletions

View file

@ -1,12 +1,2 @@
</body>
<script type='text/javascript'>
// Some platforms don't handle launching a browser to navigate a page with
// a #fragment specified, so we use this workaround combined with a query string.
// See https://github.com/fish-shell/fish-shell/issues/4480
var params = new URLSearchParams(window.location.search); //?section=foo
var section = params.get("section");
if (section != null && location.hash == "") {
location.hash = "#" + section;
}
</script>
</html>

View file

@ -86,18 +86,16 @@ function help --description 'Show help for the fish shell'
end
switch "$fish_help_item"
# Issue #4480: OS X strips the #fragment when opening URLs, so we also pass it in as
# a query string and jump to the appropriate section with some very basic javascript
case "."
set fish_help_page "commands.html\?section=source\#source"
set fish_help_page "commands.html\#source"
case globbing
set fish_help_page "index.html\?section=expand\#expand"
set fish_help_page "index.html\#expand"
case (__fish_print_commands)
set fish_help_page "commands.html\?section=$fish_help_item\#$fish_help_item"
set fish_help_page "commands.html\#$fish_help_item"
case $help_topics
set fish_help_page "index.html\?section=$fish_help_item\#$fish_help_item"
set fish_help_page "index.html\#$fish_help_item"
case 'tut_*'
set fish_help_page "tutorial.html\?section=$fish_help_item\#$fish_help_item"
set fish_help_page "tutorial.html\#$fish_help_item"
case tutorial
set fish_help_page "tutorial.html"
case "*"
@ -131,6 +129,15 @@ function help --description 'Show help for the fish shell'
set page_url https://fishshell.com/docs/$version_string/$fish_help_page
end
# OS X /usr/bin/open swallows fragments (anchors), so use osascript
# Eval is just a cheesy way of removing the hash escaping
if test "$fish_browser" = osascript
set -l opencmd 'open location "'(eval echo $page_url)'"'
osascript -e 'try' -e $opencmd -e 'on error' -e $opencmd -e 'end try'
return
end
# If browser is known to be graphical, put into background
if contains -- $fish_browser[1] $graphical_browsers
switch $fish_browser[1]