on master help sign in
Stop creating dummy divs around every Element in a collage, fixes ordering issue in elm-at-pldi-2013
commitb4440ad6078b8f0406b7cbf92b8979c575d1cb5f authorEvan Czaplicki <info@elm-lang.org> dateSun 3 Nov 2013, 02:29 (13 years ago) parent1e1b0c1
1 file changed+39−52
runtime/Render/Collage.js+39−52
runtime/Render/Collage.js+39−52
@@ -173,16 +173,16 @@ function str(n) {
173173
174174function makeTransform(w, h, form, matrices) {
175175 var props = form.form._0.props;
176− var m = A6( Transform.matrix, 1, 0, 0, 1,
177− (w - props.width)/2,
178− (h - props.height)/2 );
176+ var m = A6( Transform.matrix, 1, 0, 0, -1,
177+ (w - props.width ) / 2,
178+ (h - props.height) / 2 );
179179 var len = matrices.length;
180180 for (var i = 0; i < len; ++i) { m = A2( Transform.multiply, m, matrices[i] ); }
181181 m = A2( Transform.multiply, m, formToMatrix(form) );
182182
183− return 'matrix(' + str( m[0]) + ',' + str( m[3]) + ',' +
184− str(-m[1]) + ',' + str(-m[4]) + ',' +
185− str( m[2]) + ',' + str( m[5]) + ')';
183+ return 'matrix(' + str( m[0]) + ', ' + str( m[3]) + ', ' +
184+ str(-m[1]) + ', ' + str(-m[4]) + ', ' +
185+ str( m[2]) + ', ' + str( m[5]) + ')';
186186}
187187
188188function stepperHelp(list) {
@@ -200,7 +200,7 @@ function stepperHelp(list) {
200200 return { peekNext:peekNext, next:next };
201201}
202202
203−function stepper(forms) {
203+function formStepper(forms) {
204204 var ps = [stepperHelp(forms)];
205205 var matrices = [];
206206 var alphas = [];
@@ -253,11 +253,12 @@ function makeCanvas(w,h) {
253253
254254function render(model) {
255255 var div = newElement('div');
256+ div.style.overflow = 'hidden';
256257 update(div, model, model);
257258 return div;
258259}
259260
260−function updateTracker(w,h,div) {
261+function nodeStepper(w,h,div) {
261262 var kids = div.childNodes;
262263 var i = 0;
263264 function transform(transforms, ctx) {
@@ -271,7 +272,7 @@ function updateTracker(w,h,div) {
271272 }
272273 return ctx;
273274 }
274− function getContext(transforms) {
275+ function nextContext(transforms) {
275276 while (i < kids.length) {
276277 var node = kids[i];
277278 if (node.getContext) {
@@ -290,75 +291,61 @@ function updateTracker(w,h,div) {
290291 ++i;
291292 return transform(transforms, canvas.getContext('2d'));
292293 }
293− function element(matrices, alpha, form) {
294− var container = kids[i];
295− if (!container || container.getContext) {
296− container = newElement('div');
297− container.style.overflow = 'hidden';
298− container.style.position = 'absolute';
299− addTransform(container.style, 'scaleY(-1)');
300−
301− var kid = kids[i];
302− kid ? div.insertBefore(container, kid)
303− : div.appendChild(container);
304− }
305− // we have added a new node, so we must step our position
306− ++i;
294+ function addElement(matrices, alpha, form) {
295+ var kid = kids[i];
296+ var elem = form.form._0;
307297
308− container.style.width = w + 'px';
309− container.style.height = h + 'px';
310− container.style.opacity = alpha * form.alpha;
298+ var node = (!kid || kid.getContext)
299+ ? Render.render(elem)
300+ : (Render.update(kid, kid.oldElement, elem), kids[i]);
311301
312− var elem = form.form._0;
313− var node = container.firstChild;
314− if (node) {
315− Render.update(node, node.oldElement, elem);
316− node = container.firstChild;
317− } else {
318− node = Render.render(elem);
319− container.appendChild(node);
320− }
321− node.oldElement = elem;
302+ node.style.position = 'absolute';
303+ node.style.opacity = alpha * form.alpha;
322304 addTransform(node.style, makeTransform(w, h, form, matrices));
305+ node.oldElement = elem;
306+ ++i;
307+ if (!kid) {
308+ div.appendChild(node);
309+ } else if (kid.getContext) {
310+ div.insertBefore(node, kid);
311+ }
323312 }
324313 function clearRest() {
325314 while (i < kids.length) {
326315 div.removeChild(kids[i]);
327316 }
328317 }
329− return { getContext:getContext, element:element, clearRest:clearRest };
318+ return { nextContext:nextContext, addElement:addElement, clearRest:clearRest };
330319}
331320
332321
333322function update(div, _, model) {
334323 var w = model.w;
335324 var h = model.h;
336− div.style.width = w + 'px';
337− div.style.height = h + 'px';
338− if (model.forms.ctor === 'Nil') {
339− while (div.hasChildNodes()) {
340− div.removeChild(div.lastChild);
341− }
342− }
343− var stpr = stepper(model.forms);
344− var tracker = updateTracker(w,h,div);
325+
326+ var forms = formStepper(model.forms);
327+ var nodes = nodeStepper(w,h,div);
345328 var ctx = null;
346329 var formType = '';
347330
348− while (formType = stpr.peekNext()) {
331+ while (formType = forms.peekNext()) {
332+ // make sure we have context if we need it
349333 if (ctx === null && formType !== 'FElement') {
350− ctx = tracker.getContext(stpr.transforms());
351− ctx.globalAlpha = stpr.alpha();
334+ ctx = nodes.nextContext(forms.transforms());
335+ ctx.globalAlpha = forms.alpha();
352336 }
353− var form = stpr.next(ctx);
337+
338+ var form = forms.next(ctx);
339+ // if it is FGroup, all updates are made within formStepper when next is called.
354340 if (formType === 'FElement') {
355− tracker.element(stpr.transforms(), stpr.alpha(), form);
341+ // update or insert an element, get a new context
342+ nodes.addElement(forms.transforms(), forms.alpha(), form);
356343 ctx = null;
357344 } else if (formType !== 'FGroup') {
358345 renderForm(function() { update(div, model, model); }, ctx, form);
359346 }
360347 }
361− tracker.clearRest();
348+ nodes.clearRest();
362349 return false;
363350}
364351