Skip to content

Commit 2cc8e17

Browse files
rueckstiesskangas
authored andcommitted
INT-401 saved connections can be deleted
1 parent 5d821af commit 2cc8e17

File tree

3 files changed

+51
-12
lines changed

3 files changed

+51
-12
lines changed

src/connect/connection.jade

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
li.list-group-item
2+
.close-icon(data-hook='close')
3+
i.fa.fa-close
4+
a(data-hook='name')

src/connect/index.js

Lines changed: 28 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,21 +4,33 @@ var Connection = require('../models/connection');
44
var format = require('util').format;
55
var $ = require('jquery');
66
var app = require('ampersand-app');
7+
var debug = require('debug')('scout:connect:index');
78

89
var ConnectionView = View.extend({
910
props: {
10-
model: Connection
11-
},
11+
model: Connection,
12+
hover: {
13+
type: 'boolean',
14+
default: false
15+
}
16+
},
1217
events: {
13-
click: 'onClick',
14-
dblclick: 'onDoubleClick'
18+
'click': 'onClick',
19+
'dblclick': 'onDoubleClick',
20+
'mouseover': 'onMouseOver',
21+
'mouseout': 'onMouseOut',
22+
'click [data-hook=close]': 'onCloseClick'
1523
},
1624
bindings: {
1725
'model.name': {
1826
hook: 'name'
27+
},
28+
'hover': {
29+
type: 'toggle',
30+
hook: 'close'
1931
}
2032
},
21-
template: '<li class="list-group-item"><a data-hook="name"></a></li>',
33+
template: require('./connection.jade'),
2234
onClick: function(event) {
2335
event.stopPropagation();
2436
event.preventDefault();
@@ -30,6 +42,17 @@ var ConnectionView = View.extend({
3042
onDoubleClick: function(event) {
3143
this.onClick(event);
3244
this.parent.parent.onSubmit(event);
45+
},
46+
onCloseClick: function(event) {
47+
event.stopPropagation();
48+
event.preventDefault();
49+
this.model.destroy();
50+
},
51+
onMouseOver: function(event) {
52+
this.hover = true;
53+
},
54+
onMouseOut: function(event) {
55+
this.hover = false;
3356
}
3457
});
3558

styles/sidebar.less

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,15 @@
5959
background: none;
6060
color: @gray5;
6161

62+
&:hover {
63+
background: lighten(@slate0, 5%);
64+
a {
65+
color: @pw;
66+
border-left: 4px solid lighten(@slate0, 5%);
67+
text-decoration: none;
68+
}
69+
}
70+
6271
a {
6372
display: block;
6473
color: @gray5;
@@ -68,14 +77,8 @@
6877
overflow: hidden;
6978
text-overflow: ellipsis;
7079
border-left: 4px solid transparent;
71-
72-
&:hover {
73-
color: @pw;
74-
text-decoration: none;
75-
background: lighten(@slate0, 5%);
76-
border-left: 4px solid lighten(@slate0, 5%);
77-
}
7880
}
81+
7982
i {
8083
color: lighten(@text-color, 40%);
8184
line-height: 24px;
@@ -97,6 +100,15 @@
97100

98101
}
99102
}
103+
104+
.close-icon {
105+
position: absolute;
106+
right: 0px;
107+
108+
&:hover i {
109+
color: @pw;
110+
}
111+
}
100112
}
101113
}
102114
.sidebar-controls {

0 commit comments

Comments
 (0)