From c30342dfc7f0aba034d5beeda72c690a57293050 Mon Sep 17 00:00:00 2001 From: Misha Koryak Date: Wed, 20 Oct 2021 09:17:08 -0400 Subject: [PATCH] completely remove floatthead Removed all code that initializes the plugin (doing this under the assumption that https://github.com/timbunce/devel-nytprof/pull/194 removes the js file from this project). Also removed `show_fragment_target()` because it would no longer work without floatthead present. This fixes #192 , or at least fixes the fix that was made in it. The proper way to fix #192 would be to take a look at the `show_fragment_target()` function and figure out why its setting the incorrect scrollTop there when you click on links. I cannot do this without seeing the generated code. Either way, someone needs to test this change before merging it, because I have not done so. --- bin/nytprofhtml | 31 +------------------------------ 1 file changed, 1 insertion(+), 30 deletions(-) diff --git a/bin/nytprofhtml b/bin/nytprofhtml index 0c02e142..323d5b62 100755 --- a/bin/nytprofhtml +++ b/bin/nytprofhtml @@ -419,12 +419,8 @@ sub subroutine_table { 4: { sorter: 'fmt_time' } } }); - $(".floatHeaders").each( function(){ $(this).floatThead(); } ); - show_fragment_target(); - $(window).on('hashchange', function(e){ - show_fragment_target(); - }); + > if @subs_to_show == @subs; @@ -1507,13 +1503,6 @@ sub output_file_table { 2: { sorter: false } } }); - - $(".floatHeaders").each( function(){ $(this).floatThead(); } ); - - show_fragment_target(); - $(window).on('hashchange', function(e){ - show_fragment_target(); - }); }; return ""; @@ -1659,7 +1648,6 @@ EOD $html .= <<'EOD' unless $opts->{skip_jquery}; - EOD $html .= $opts->{head_epilogue} if $opts->{head_epilogue};