Skip to content

Commit

Permalink
cmd/compile: tweak ssa.html colors
Browse files Browse the repository at this point in the history
Make yellow the last highlight color rather than the first.
Yellow is also the color that Chrome uses to highlight
search results, which can be confusing.
Also, when Night Shift is on on macOS,
yellow highlighting is completely invisible.
I suppose should be sleeping instead.

Also, remove a completed TODO.

Change-Id: I0eb4439272fad9ccb5fe8e2cf409fdd5dc15b26e
Reviewed-on: https://go-review.googlesource.com/43463
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
  • Loading branch information
josharian committed May 14, 2017
1 parent 61336b7 commit cfae61b
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/cmd/compile/internal/ssa/html.go
Expand Up @@ -142,13 +142,13 @@ dd.ssa-prog {
<script type="text/javascript">
// ordered list of all available highlight colors
var highlights = [
"highlight-yellow",
"highlight-aquamarine",
"highlight-coral",
"highlight-lightpink",
"highlight-lightsteelblue",
"highlight-palegreen",
"highlight-lightgray"
"highlight-lightgray",
"highlight-yellow"
];
// state: which value is highlighted this color?
Expand Down Expand Up @@ -264,8 +264,6 @@ function toggle_visibility(id) {
</script>
</head>`)
// TODO: Add javascript click handlers for blocks
// to outline that block across all phases
w.WriteString("<body>")
w.WriteString("<h1>")
w.WriteString(html.EscapeString(name))
Expand Down

0 comments on commit cfae61b

Please sign in to comment.