Merge pull request #41 from yzen/bug-1162040

Bug 1162040 - added some helpful diagnostics/warning when gaia-icon e…
fxos-components · Jun 3, 2015 · de969d9 · de969d9
2 parents aac506e + 5aee941
commit de969d9
Show file tree

Hide file tree

Showing 5 changed files with 256 additions and 206 deletions.
diff --git a/gaia-icons-embedded.css b/gaia-icons-embedded.css
diff --git a/gaia-icons.css b/gaia-icons.css
@@ -23,3 +23,20 @@
 	font-size: 30px;
 	-webkit-font-smoothing: antialiased;
 }
+
+[data-icon]:not([data-l10n-id]):not([aria-hidden="true"]):before {
+	/* Highlight cases where icons are not localized and still visible to screen
+	   reader */
+	background: rgba(255, 0, 0, 0.4);
+}
+
+[data-icon]:not([data-l10n-id]):not([aria-hidden="true"]):after {
+	background: #333;
+	color: #fff;
+	position: absolute;
+	width: 20rem;
+	border-radius: 0.5rem;
+	padding: 0.5rem;
+	content: "Icon is not localized and is still visible to screen reader. Please add data-l10n-id or aria-label for localization or use aria-hidden to hide the icon from screen reader.";
+	z-index: 9999;
+}