Diff
Modified: branches/safari-537-branch/Source/WebInspectorUI/ChangeLog (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/ChangeLog 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/ChangeLog 2013-07-02 19:27:12 UTC (rev 152310)
@@ -1,5 +1,33 @@
2013-07-02 Lucas Forschler <[email protected]>
+ Merge r152199
+
+ 2013-06-28 James Craig <[email protected]>
+
+ Web Inspector: AX: VoiceOver speaks all inspector search fields as 'required'
+ https://bugs.webkit.org/show_bug.cgi?id=118122
+
+ Reviewed by Timothy Hatcher.
+
+ Removed bogus "required" state, and switched :valid selector to use :not(:placeholder-shown)
+ so that screen readers will not indicate a bogus :required/:invalid state to the user.
+ Minor cosmetic difference blocked by http://webkit.org/b/118162
+
+ * UserInterface/FilterBar.css:
+ (.filter-bar > input[type="search"]:focus):
+ (.filter-bar > input[type="search"]:not(:placeholder-shown)):
+ * UserInterface/FilterBar.js:
+ (WebInspector.FilterBar):
+ * UserInterface/ResourceSidebarPanel.js:
+ (WebInspector.ResourceSidebarPanel):
+ * UserInterface/SearchBar.css:
+ (.search-bar > input[type="search"]:focus):
+ (.search-bar > input[type="search"]:not(:placeholder-shown)):
+ * UserInterface/SearchBar.js:
+ (WebInspector.SearchBar):
+
+2013-07-02 Lucas Forschler <[email protected]>
+
Merge r152141
2013-06-27 James Craig <[email protected]>
Modified: branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.css (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.css 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.css 2013-07-02 19:27:12 UTC (rev 152310)
@@ -57,10 +57,8 @@
color: rgba(0, 0, 0, 0.35);
}
-.filter-bar > input[type="search"]:focus,
-.filter-bar > input[type="search"]:valid {
- background-color: white;
-}
+.filter-bar > input[type="search"]:focus { background-color: white; }
+.filter-bar > input[type="search"]:not(:placeholder-shown) { background-color: white; } /* Needs to be a separate rule. See http://webkit.org/b/118162 */
.filter-bar > input[type="search"]::-webkit-search-decoration {
width: 13px;
Modified: branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.js (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.js 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/UserInterface/FilterBar.js 2013-07-02 19:27:12 UTC (rev 152310)
@@ -31,7 +31,6 @@
this._inputField = document.createElement("input");
this._inputField.type = "search";
- this._inputField.required = true;
this._inputField.spellcheck = false;
this._inputField.incremental = true;
this._inputField.addEventListener("search", this._inputFieldChanged.bind(this), false);
Modified: branches/safari-537-branch/Source/WebInspectorUI/UserInterface/ResourceSidebarPanel.js (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/UserInterface/ResourceSidebarPanel.js 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/UserInterface/ResourceSidebarPanel.js 2013-07-02 19:27:12 UTC (rev 152310)
@@ -32,7 +32,6 @@
this._inputElement = document.createElement("input");
this._inputElement.type = "search";
- this._inputElement.required = true;
this._inputElement.spellcheck = false;
this._inputElement.addEventListener("search", this._searchFieldChanged.bind(this));
this._inputElement.addEventListener("input", this._searchFieldInput.bind(this));
Modified: branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.css (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.css 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.css 2013-07-02 19:27:12 UTC (rev 152310)
@@ -45,7 +45,5 @@
color: rgba(0, 0, 0, 0.35);
}
-.search-bar > input[type="search"]:focus,
-.search-bar > input[type="search"]:valid {
- background-color: white;
-}
+.search-bar > input[type="search"]:focus { background-color: white; }
+.search-bar > input[type="search"]:not(:placeholder-shown) { background-color: white; } /* Needs to be a separate rule. See http://webkit.org/b/118162 */
Modified: branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.js (152309 => 152310)
--- branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.js 2013-07-02 19:24:35 UTC (rev 152309)
+++ branches/safari-537-branch/Source/WebInspectorUI/UserInterface/SearchBar.js 2013-07-02 19:27:12 UTC (rev 152310)
@@ -35,7 +35,6 @@
this._searchInput = this._element.appendChild(document.createElement("input"));
this._searchInput.type = "search";
- this._searchInput.required = true;
this._searchInput.spellcheck = false;
this._searchInput.incremental = true;
this._searchInput.setAttribute("results", 5);
