Bug #74493 [Com]: Summary: On page links jump to incorrect anchors if the name contains the target

From: Date: Wed, 26 Apr 2017 10:21:44 +0000
Subject: Bug #74493 [Com]: Summary: On page links jump to incorrect anchors if the name contains the target
References: 1  Groups: php.webmaster 
Request: Send a blank email to php-webmaster+get-25768@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=74493&edit=1 ID: 74493 Comment by: jay at room11 dot org Reported by: php-bugs-2014-08 at dotancohen dot com Summary: Summary: On page links jump to incorrect anchors if the name contains the target Status: Verified Type: Bug Package: Website problem PHP Version: Irrelevant Assigned To: peehaa Block user comment: N Private report: N New Comment: @levi - All done Previous Comments: ------------------------------------------------------------------------ [2017-04-25 14:30:54] levim@php.net I'm not entirely sure why someone did "contains" in the first place but I'm inclined to believe there was actually a reason. I suppose we can commit this with an added comment that mentions this bug so when we uncover the reason it is like this we can do a proper fix. ------------------------------------------------------------------------ [2017-04-25 12:26:19] peehaa@php.net Jay created a PR for to fix this https://github.com/php/web-php/pull/154 If somebody with powers could review and merge that please. ------------------------------------------------------------------------ [2017-04-24 08:15:15] requinix@php.net Ah, sorry @peehaa. ------------------------------------------------------------------------ [2017-04-24 08:14:26] requinix@php.net Because it's de-limit-er. https://github.com/php/web-php/blob/master/js/common.js#L411 '.parameters .parameter:contains("' + id + '"), .options .parameter:contains("' + id + '")' ------------------------------------------------------------------------ [2017-04-24 08:04:32] peehaa@php.net The links jumps to de*limit*er. Basically the first *limit* param the JS sess. Should probably be made more strict to only jump to exact matches. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=74493 -- Edit this bug report at https://bugs.php.net/bug.php?id=74493&edit=1

« previous php.webmaster (#25768) next »