##// END OF EJS Templates
Removing extra refresh that is no longer needed because of CM fix.
Brian Granger -
Show More
@@ -530,11 +530,11 b' var IPython = (function (IPython) {'
530 530
531 531 CodeCell.prototype.select = function () {
532 532 IPython.Cell.prototype.select.apply(this);
533 // In some cases (inserting a new cell) we need a refresh before and
534 // after the focus. Not sure why this is the case.
535 533 this.code_mirror.refresh();
536 534 this.code_mirror.focus();
537 this.code_mirror.refresh();
535 // We used to need an additional refresh() after the focus, but
536 // it appears that this has been fixed in CM. This bug would show
537 // up on FF when a newly loaded markdown cell was edited.
538 538 };
539 539
540 540
@@ -96,14 +96,11 b' var IPython = (function (IPython) {'
96 96 var output = text_cell.find("div.text_cell_render");
97 97 output.hide();
98 98 text_cell.find('div.text_cell_input').show();
99 // I don't know why I need to do this, but if I don't do
100 // refresh/focus/refresh, the to_markdown method won't work.
101 99 this.code_mirror.refresh();
102 100 this.code_mirror.focus();
103 // This final refresh is needed on Firefox to trigger the editor
104 // to be auto-sized. This glitch only happens on cell that are
105 // loaded initially and haven't had their editor focused before.
106 this.code_mirror.refresh();
101 // We used to need an additional refresh() after the focus, but
102 // it appears that this has been fixed in CM. This bug would show
103 // up on FF when a newly loaded markdown cell was edited.
107 104 this.rendered = false;
108 105 if (this.get_text() === this.placeholder) {
109 106 this.set_text('');
General Comments 0
You need to be logged in to leave comments. Login now