Details
Bug
Minor
Description
One of my site members noticed that the keyboard focus is not set to the keywords field when search.php is selected, requiring a second click. I confirmed this using IE and Firefox on Vista.
Adding a little bit of JavaScript to the search_body.html worked for Firefox:
<script type="text/javascript" language="javascript">document.getElementById("keywords").focus();</script>
Unfortunately it did not work for IE. The login also has the same problem, i.e., a second click is required.
Activity
Patch for prosilver for standards-compliant browsers.
It handles login and search pages.
Login page should work on ie too.
Patch for subsilver: http://github.com/p/phpbb3/compare/feature/7417-subsilver
I can't get it to actually work. This code if executed after page is loaded focuses the text fields successfully in fx but it does nothing when it runs during page load.
Due to subsilver remaining blissfully unaware of element id's and onload functions, quite a bit more contraption is required to make it work.
Why don't you try this instead:
if (elements[i].id == 'keywords')
|
Also for prosilver you need to push that function to the onload_functions array. For subsilver2 you'd have to assign your function to window.onload.
What's the status here? Should I be waiting for an update to the subsilver fix?
I am happy with subsilver fix in http://github.com/p/phpbb3/compare/feature/7417-subsilver. True, it is verbose but it does seem to function and considering that subsilver is being eliminated anyway I don't see much point in spending more time on it.
I'll merge both branches.
The reason why focus does not work in ie on search page is due to ie bug where getElementById returns elements whose name matches the id given. In particular, ie finds and attempts to focus this element:
<meta name="keywords" content="" />