Bug #74493 [Ver->Csd]: Summary: On page links jump to incorrect anchors if the name contains the target
| From: | peehaa@php.net | Date: | Fri, 21 Jul 2017 13:28:01 +0000 |
| Subject: | Bug #74493 [Ver->Csd]: 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-26175@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
Updated by: peehaa@php.net
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
+Status: Closed
Type: Bug
Package: Website problem
PHP Version: Irrelevant
Assigned To: peehaa
Block user comment: N
Private report: N
New Comment:
The fix for this bug has been committed. Since the websites are not directly
updated from the repository, the fix might need some time to spread
across the globe to all mirror sites, including PHP.net itself.
Thank you for the report, and for helping us make PHP.net better.
Previous Comments:
------------------------------------------------------------------------
[2017-04-26 10:21:43] jay at room11 dot org
@levi - All done
------------------------------------------------------------------------
[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 + '")'
------------------------------------------------------------------------
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