296989 2025-08-06 01:51:46 -0700 madisoncity.k12.al.us: Search button is not working (Adds Quirks needsFormControlToBeMouseFocusable) 2025-12-09 14:01:03 -0800 1 1 1 Unclassified WebKit Forms WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://www.madisoncity.k12.al.us/Page/11 BrowserCompat, InRadar P2 Normal --- 22261 193478 1 karlcow karlcow cdumez webkit-bug-importer wenson_hsieh oldest_to_newest 2134649 0 karlcow 2025-08-06 01:51:46 -0700 This was initially reported on https://github.com/webcompat/web-bugs/issues/151961 1. Go to https://www.madisoncity.k12.al.us/Page/11 2. Click on the magnifier at the top right of the menu bar. Results: nothing Expected: a search form appears like in Firefox and Chrome. 2134650 1 webkit-bug-importer 2025-08-06 01:54:22 -0700 <rdar://problem/157639717> 2134651 2 karlcow 2025-08-06 01:59:17 -0700 ``` <div id="gb-search" class="ui-clear" data-search-icon-color="White"> <button id="gb-search-btn" class="cs-btn-reset" aria-controls="gb-search-form" aria-label="Search" aria-expanded="false"><span class="cs-ally-hidden">Search</span></button> <div id="gb-search-form-menu" aria-hidden="true"> <form id="gb-search-form"> <div class="search-input-icon drt-aspire-icon-font"></div> <label for="gb-search-input" class="cs-ally-hidden">Search...</label> <input tabindex="-1" type="text" id="gb-search-input" class="gb-search-input" value="Search..."> <button tabindex="-1" type="submit" id="gb-submit-search" class="cs-btn-reset" aria-label="submit search"><span>Go</span></button> </form> </div> </div> ``` There are two events defined: focus and hover and they both call: ``` $("#gb-search #gb-search-btn").on("hover focus", function(e) { //if(_this.AllyClick(e)) { e.preventDefault(); if(!$("#gb-search-form-menu").hasClass("open")){ $("#gb-search-input, #gb-submit-search").attr("tabindex","0"); $("#gb-search-form-menu").addClass("open").attr("aria-hidden","false"); $(this).addClass("open").attr("aria-expanded","true"); $("#gb-search-form #gb-search-input").focus(); } else { $("#gb-search-input, #gb-submit-search").attr("tabindex","-1"); $("#gb-search-form-menu").removeClass("open").attr("aria-hidden","true"); $(this).removeClass("open").focus().attr("aria-expanded","false"); } //} }); }, ``` not sure why AllyClick has been put in comment. ``` "AllyClick": function(event) { if(event.type == "click") { return true; } else if(event.type == "keydown") { if(event.keyCode == this.KeyCodes.space || event.keyCode == this.KeyCodes.enter) { return true; } } else { return false; } }, ``` 2136132 3 karlcow 2025-08-13 00:07:43 -0700 The website expects button to be focusable. While this is expected by the web platform, it is for now a choice for macOS platform convention to not follow this behavior. See previously: Bug 22261 Bug 193478 2136148 4 karlcow 2025-08-13 01:19:07 -0700 To note that if the site was just doing that ``` $("#gb-search #gb-search-btn").on("focus hover click", function(e) { //if(_this.AllyClick(e)) { e.preventDefault(); ``` it would work. 2136151 5 karlcow 2025-08-13 01:31:03 -0700 So it's not enough to add https://searchfox.org/wubkat/rev/1880bd04e75e45ec48e76e3c61706f9514ab3d99/Source/WebCore/page/Quirks.cpp#174-183 ```cpp // ceac.state.gov https://bugs.webkit.org/show_bug.cgi?id=193478 // weather.com rdar://139689157 bool Quirks::needsFormControlToBeMouseFocusable() const { #if PLATFORM(MAC) return needsQuirks() && m_quirksData.needsFormControlToBeMouseFocusableQuirk; #else return false; #endif // PLATFORM(MAC) } ``` to solve the issue 2136154 6 karlcow 2025-08-13 02:00:49 -0700 I was wondering if it was because the code has $("#gb-search-input, #gb-submit-search").attr("tabindex","0"); and this part https://searchfox.org/wubkat/rev/1880bd04e75e45ec48e76e3c61706f9514ab3d99/Source/WebCore/html/HTMLFormControlElement.cpp#236-246 ```cpp bool HTMLFormControlElement::isMouseFocusable() const { #if (PLATFORM(GTK) || PLATFORM(WPE)) return HTMLElement::isMouseFocusable(); #else // FIXME: We should remove the quirk once <rdar://problem/47334655> is fixed. if (!!tabIndexSetExplicitly() || document().quirks().needsFormControlToBeMouseFocusable()) return HTMLElement::isMouseFocusable(); return false; #endif } ``` but we are not reaching the code inside the if, so maybe there is something broken in the code. 2136155 7 karlcow 2025-08-13 02:10:25 -0700 Oh… is it because https://searchfox.org/wubkat/source/Source/WebCore/html/HTMLButtonElement.cpp has isMouseFocusable at all. 2136156 8 karlcow 2025-08-13 02:39:02 -0700 Ah no we correctly enter bool HTMLFormControlElement::isMouseFocusable() const {} we do not pass the `if` for https://www.madisoncity.k12.al.us/Page/11 maybe my domain check is not correct. 2136159 9 karlcow 2025-08-13 03:04:23 -0700 Pull request: https://github.com/WebKit/WebKit/pull/49323 2164586 10 ews-feeder 2025-12-09 14:01:01 -0800 Committed 304177@main (92a0e1480875): <https://commits.webkit.org/304177@main> Reviewed commits have been landed. Closing PR #49323 and removing active labels.