Berker Peksag added the comment: > Do you mean in addition to the "Quick search" text that is already on the > page or do you want to remove that?
The latter. All of the popular browsers (Chrome, Firefox, Edge and even Safari) already support the placeholder attribute so I think we can safely ignore IE 8 :) > What problems are you thinking of? Honestly, I can't remember now, but I was talking about old Chrome and Firefox versions. Let's ignore my comment for now. I have two more minor comments: 1. + <div class="inline-search" style="display: none" role="search"> I know this is basically a copy of upstream searchbox.html, but we can probably remove ``style="display: none"`` and ``$('.inline-search').show(0);``. 2. + <li class="right inline-search"> inline-search doesn't seem to be necessary here. ``display: inline`` is already applied by ``div.related li``. I also found a regression in search.html. It doesn't show or highlight the search words provided in the URL. Try the following URL to reproduce the problem: https://docs.python.org/dev/search.html?q=append Screenshot from docs.python.org: https://dl.dropboxusercontent.com/u/166024/highlight-docs.pyo.png Screenshor from my local copy with searchbar_in_header.diff3 applied: https://dl.dropboxusercontent.com/u/166024/highlight-after.png ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue19489> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com