Skip to content

Commit d004527

Browse files
committed
scroll elements into view when editing/selecting
fixes #2313
1 parent 11ffa07 commit d004527

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

indigo_app/static/javascript/indigo/views/document_editor.js

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,8 @@
4949
this.onXmlElementParsed.bind(this),
5050
) : null;
5151

52+
this.contentPane = document.querySelector('.document-primary-pane-content-pane');
53+
5254
// setup renderer
5355
this.editorReady = $.Deferred();
5456
this.listenTo(this.document, 'change', this.onDocumentChanged);
@@ -86,16 +88,13 @@
8688
},
8789

8890
quickEdit: function(e) {
89-
var elemId = e.currentTarget.parentElement.parentElement.id,
90-
element = this.parent.documentContent.xmlDocument;
91-
92-
// the id might be scoped
93-
elemId.split("/").forEach(function(id) {
94-
element = element.querySelector('[eId="' + id + '"]');
95-
});
91+
const htmlElement = e.currentTarget.parentElement.parentElement;
92+
const elemId = htmlElement.id;
93+
const element = this.parent.documentContent.xmlDocument.querySelector('[eId="' + elemId + '"]');
9694

9795
if (element && this.confirmAndDiscardChanges()) {
9896
this.editXmlElement(element);
97+
htmlElement.scrollIntoView({behavior: "smooth"});
9998
}
10099
},
101100

@@ -191,7 +190,7 @@
191190
}
192191

193192
this.render();
194-
this.$('.document-sheet-container').scrollTop(0);
193+
this.contentPane.scrollTo(0, 0);
195194
}
196195
},
197196

0 commit comments

Comments
 (0)