Skip to content

Commit 1850e30

Browse files
committed
Toggle the StacktraceView class on activation.
1 parent 28b8eae commit 1850e30

File tree

3 files changed

+19
-4
lines changed

3 files changed

+19
-4
lines changed

lib/stacktrace-view.coffee

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,14 @@
11
{View, EditorView} = require 'atom'
2+
{Subscriber} = require 'emissary'
3+
24
{Stacktrace, PREFIX} = require './stacktrace'
35

46
class StacktraceView extends View
57

8+
Subscriber.includeInto this
9+
610
@content: (trace) ->
7-
tclass = if trace.isActive() then 'activated' else 'deactivated'
11+
tclass = if trace.isActive() then 'activated' else ''
812
@div class: "stacktrace tool-panel padded #{tclass}", =>
913
@div class: 'panel padded', =>
1014
@h2 class: 'error-message', trace.message
@@ -19,6 +23,11 @@ class StacktraceView extends View
1923
@subview 'frame', new FrameView(frame)
2024

2125
initialize: (@trace) ->
26+
@subscribe Stacktrace, 'active-changed', (e) =>
27+
if e.newTrace is @trace
28+
@addClass 'activated'
29+
else
30+
@removeClass 'activated'
2231

2332
# Internal: Return the window title.
2433
#

spec/stacktrace-view-spec.coffee

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,12 @@ describe 'StacktraceView', ->
3737
vs = view.find('.frame')
3838
expect(vs.length).toBe(3)
3939

40+
it 'changes its class when its trace is activated or deactivated', ->
41+
Stacktrace.getActivated()?.deactivate()
42+
expect(view.hasClass 'activated').toBe(false)
43+
trace.activate()
44+
expect(view.hasClass 'activated').toBe(true)
45+
4046
describe 'FrameView', ->
4147
[view] = []
4248

stylesheets/stacktrace.less

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,11 @@
3737
}
3838
}
3939

40+
.deactivate-control { display: none; }
41+
4042
&.activated {
43+
.deactivate-control { display: block; }
4144
.activate-control { display: none; }
4245
}
4346

44-
&.deactivated {
45-
.deactivate-control { display: none; }
46-
}
4747
}

0 commit comments

Comments
 (0)