CalvinBackup/iD
1
0
Fork 0
mirror of https://github.com/FoggedLens/iD.git synced 2026-02-13 01:02:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

Wrap selection list geometry icon in a span

Browse Source 
This is important so d3 update selection can select the geometry icon but not
the close icon.
This commit is contained in:
Bryan Housel
2016-07-26 15:51:09 -04:00
parent 77fa3c9d8b
commit 018e7f207e
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
modules/ui/selection_list.js
View File

@@ -51,7 +51,10 @@ export function SelectionList(context, selectedIDs) {
// Enter
var label = enter.append('div')
.attr('class', 'label')
.attr('class', 'label');
label.append('span')
.attr('class', 'entity-geom-icon')
.call(Icon('', 'pre-text'));
label.append('span')
@@ -60,8 +63,13 @@ export function SelectionList(context, selectedIDs) {
label.append('span')
.attr('class', 'entity-name');
label.append('span')
.attr('class', 'close')
.on('click', deselectEntity)
.call(Icon('#icon-close'));
// Update
items.selectAll('use')
items.selectAll('.entity-geom-icon use')
.attr('href', function() {
var entity = this.parentNode.parentNode.__data__;
return '#icon-' + context.geometry(entity.id);
@@ -73,11 +81,6 @@ export function SelectionList(context, selectedIDs) {
items.selectAll('.entity-name')
.text(function(entity) { return displayName(entity); });
label.append('span')
.attr('class', 'close')
.on('click', deselectEntity)
.call(Icon('#icon-close'));
// Exit
items.exit()
.remove();