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

Unified Diff: pkg/dartdoc/lib/src/client/client-shared.dart

Issue 11028138: Fields, getters and setters documented as properties. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 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: pkg/dartdoc/lib/src/client/client-shared.dart
diff --git a/pkg/dartdoc/lib/src/client/client-shared.dart b/pkg/dartdoc/lib/src/client/client-shared.dart
index bddaa1226c46821ec37b15b312270bae84989c9c..a448ae2765bb89daba8549f701996f4beda7c268 100644
--- a/pkg/dartdoc/lib/src/client/client-shared.dart
+++ b/pkg/dartdoc/lib/src/client/client-shared.dart
@@ -38,18 +38,20 @@ enableCodeBlocks() {
// disabled.
if (showCode == null) continue;
- var pre = elem.query('pre.source');
+ var preList = elem.queryAll('pre.source');
Bob Nystrom 2012/10/11 17:04:21 Nit, I would prefer "pres" or "sources" over "preL
Lasse Reichstein Nielsen 2012/10/12 09:47:26 Agree, and the HTML tag seems less interesting tha
showCode.on.click.add((e) {
- if (pre.classes.contains('expanded')) {
- pre.classes.remove('expanded');
- } else {
- // Syntax highlight.
- if (!pre.classes.contains('formatted')) {
- pre.innerHTML = classifySource(pre.text);
- pre.classes.add('formatted');
- };
- pre.classes.add('expanded');
+ for (final pre in preList) {
+ if (pre.classes.contains('expanded')) {
+ pre.classes.remove('expanded');
+ } else {
+ // Syntax highlight.
+ if (!pre.classes.contains('formatted')) {
+ pre.innerHTML = classifySource(pre.text);
+ pre.classes.add('formatted');
+ };
+ pre.classes.add('expanded');
+ }
}
});
}

Powered by Google App Engine
This is Rietveld 408576698