Skip to content

Commit b5690f9

Browse files
committed
Auto merge of #2103 - Turbo87:flash-message, r=locks
Simplify and reuse `FlashMessage` component r? @locks
2 parents 6acb22a + ebd9610 commit b5690f9

File tree

4 files changed

+3
-6
lines changed

4 files changed

+3
-6
lines changed

app/components/flash-message.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
import Component from '@ember/component';
2-
import { readOnly } from '@ember/object/computed';
3-
import { inject as service } from '@ember/service';
42

53
export default Component.extend({
6-
flashMessages: service(),
7-
message: readOnly('flashMessages.message'),
84
tagName: '',
95
});

app/controllers/application.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { EKMixin, keyDown, keyPress } from 'ember-keyboard';
55
import { on } from '@ember/object/evented';
66

77
export default Controller.extend(EKMixin, {
8+
flashMessages: service(),
89
search: service(),
910
searchQuery: oneWay('search.q'),
1011
session: service(),

app/templates/application.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@
127127
</form>
128128

129129
<main id="main" class='inner-content'>
130-
<FlashMessage />
130+
<FlashMessage @message={{this.flashMessages.message}} />
131131

132132
{{outlet}}
133133
</main>

app/templates/catch-all.hbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<p id='flash' class='shown'>Oops, that route doesn't exist!</p>
1+
<FlashMessage @message="Oops, that route doesn't exist!" />
22

33
<p id='p404'>
44
Perhaps a search of the site may help?

0 commit comments

Comments
 (0)