mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-05-05 18:52:56 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibHTML: Clicking on a fragment link should prefer <a id> over <a name>

Browse source 
It turns out that other engines prefer <a id> over <a name> when
deciding which anchor element to jump to.

This patch aligns LibHTML's behavior with WebKit and Gecko.

Thanks to "/cam 2" for bringing this up. :^)
This commit is contained in:
Andreas Kling 2019-10-21 12:04:17 +02:00
parent 4d9740ecef
commit 8e710b16de
Notes: sideshowbarker 2024-07-19 11:36:03 +09:00
1 changed files with 12 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

21
Libraries/LibHTML/HtmlView.cpp
View file

@ -319,17 +319,20 @@ void HtmlView::scroll_to_anchor(const StringView& name)
if (!document())
return;
HTMLAnchorElement* element = nullptr;
document()->for_each_in_subtree([&](auto& node) {
if (is<HTMLAnchorElement>(node)) {
auto& anchor_element = to<HTMLAnchorElement>(node);
if (anchor_element.name() == name) {
element = &anchor_element;
return IterationDecision::Break;
const HTMLAnchorElement* element = nullptr;
if (auto* candidate = document()->get_element_by_id(name)) {
if (is<HTMLAnchorElement>(*candidate))
element = to<HTMLAnchorElement>(candidate);
}
if (!element) {
auto candidates = document()->get_elements_by_name(name);
for (auto* candidate : candidates) {
if (is<HTMLAnchorElement>(*candidate)) {
element = to<HTMLAnchorElement>(candidate);
break;
}
}
return IterationDecision::Continue;
});
}
if (!element) {
dbg() << "HtmlView::scroll_to_anchor(): Anchor not found: '" << name << "'";