Message 272279 - Python tracker (original) (raw)

This looks pretty good to me, thanks! I have two minor suggestions:

1. It would be better to make "Quick search" a placeholder:

3. will add an unnecessary border in search page. I'd change it to

   <li{%- if pagename != "search" and builder != "singlehtml" %} class="right"{% endif %}>

Also,

+form.inline-search input {
+    display: inline;
+}

may cause weird compatibility problems in some (old?) browsers. We can change it to use display: inline-block instead.

Is there a way to send this to upstream Sphinx?