Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(426)

Unified Diff: ui/webui/resources/cr_elements/network/cr_network_list_item.js

Issue 2755643006: MD Settings: Network a11y fixes (Closed)
Patch Set: Use item.Name for known networks Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/webui/resources/cr_elements/network/cr_network_list_item.js
diff --git a/ui/webui/resources/cr_elements/network/cr_network_list_item.js b/ui/webui/resources/cr_elements/network/cr_network_list_item.js
index e483e417b8a5db3aa0270243488bd8018ac1f175..f1a49c6c49b6433ab38cffa86f521d691548edef 100644
--- a/ui/webui/resources/cr_elements/network/cr_network_list_item.js
+++ b/ui/webui/resources/cr_elements/network/cr_network_list_item.js
@@ -195,4 +195,13 @@ Polymer({
this.fire('show-detail', this.networkState);
event.stopPropagation();
},
+
+ /**
+ * @param {!CrOnc.NetworkStateProperties}} state
+ * @return {string}
+ * @private
+ */
+ getDetailsA11yString_: function(state) {
hcarmona 2017/03/17 21:58:22 Will the network name from the state be different
stevenjb 2017/03/17 22:54:08 No, and this will cover OOBE usage which I didn't
+ return CrOnc.getNetworkName(state);
+ },
});

Powered by Google App Engine
This is Rietveld 408576698