errorrik/mdv

Automatically exported from code.google.com/p/mdv

mdv's Issues

Design the use_case where a widget wants to bind to a model, but also to custom transient state

e.g. selectedRow for a masterView

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:41

FAQ Answer: Leave it to libraries?

http://mdv.googlecode.com/svn/trunk/docs/faq.html#leaveItToLibraries

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 6:49

Array index mutations should also notify that 'length' property changed

UI may want to display or limit the number of items.

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:24

template: removeBindings can find its bindings from the boundSources_ array of its modelOwner

This avoids having to walk its entire sub-dom

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:37

Implement async callbacks

We can take the same approach here to async template operation: implement it 
such that it happens synchronously to a top-level change, but sub-changes are 
delayed/queued.

For callbacks, this will at least mean we can create tests that assert that 
only one *callback* is on the stack at one time.

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:34

Design and implement a creation event for template instances

The idea should be to let authors hook into instance creation so they can do 
some post-processing on the DOM created by MDV.

Original issue reported on code.google.com by [email protected] on 2 May 2011 at 7:13

rename instancePrototype to templateDom in docs to match prototype

Or the other way around. They should just match

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:29

FAQ Answer: Relation to XBL?

http://mdv.googlecode.com/svn/trunk/docs/faq.html#relationToXBL

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 6:49

Create a transform that removes the attribute

We have sample code like this which is pretty ugly

<span hidden="{{ expr(bid, reserve) bid >= reserve ? '' : null }}">

Original issue reported on code.google.com by [email protected] on 11 Jan 2013 at 9:54

Add extension docs use_case

Needs to be pulled over from old repo

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:17

Make 1-pass operation work with IE9

It should be close

Original issue reported on code.google.com by [email protected] on 19 Apr 2011 at 10:35

Make 1-pass operation work in Opera

nuff said

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:23

Design how bindings work within an XBL2 component

Does the anonymous DOM inherit the model implicitly?

How are attributes made available to elements?

Can the anonymous DOM consume model scope? (e.g. what happens to <content> 
output nodes in terms of their scope)?

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:40

Patch for /trunk/capabilities/transform.html

NaN used to show both messages.
Can we have special handling for numbers? It seems odd to have to write "var 
!== NaN" every time.

Original issue reported on code.google.com by [email protected] on 12 May 2011 at 10:54

Attachments:

• transform.html.patch

sandbox needs work

Not working in FF (need to check Safari/IE9)
Needs more UI (so you know what the boxes are for)
Needs a JS command line (for updating model properties)

Create links to sandbox from examples in docs/capabilities

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:26

Wrong link to use cases from summary page

What steps will reproduce the problem?
1. Go to MDV project home page.
2. Click 'Use cases' link

https://code.google.com/p/mdv/source/browse/#git%2Ftrunk%2Fuse_cases

Should be:

https://code.google.com/p/mdv/source/browse/use_cases

Original issue reported on code.google.com by [email protected] on 24 Oct 2012 at 6:19

rename templateScope_ to instancePath_ in prototype

to match docs

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:28

Main page capitalizes Firefox incorrectly

navigate to: code.google.com/p/mdv

The text talks about requiring FireFox 4, should be Firefox.

/pedantic
Thanks!

Original issue reported on code.google.com by robmcampbell on 28 Apr 2011 at 4:08

Implement lastObservedValue for observeObject observers

Should match the semantics of PathValue observers.

Update the docs

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:35

Design and implement bind attribute

this includes URL-context aware behavior for inline and OOB bindings

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:31

modelScope appears to be broken

the following doesn't seem to work:

<template instantiate>
<div modelScope="foo">
{{ bar }}
</div>
</template>

I debugged a bit, and I think the problem has to do with phantomInstances

Original issue reported on code.google.com by [email protected] on 18 Apr 2011 at 11:55

Calling Model.dirtyCheck within a Model.observe callback creates an iloop.

What steps will reproduce the problem?
1. Create an observer for some data.
2. In that observer call Model.dirtyCheck
3. Make a data change that will cause the observer to be notified and call 
Model.dirtyCheck

What is the expected output? What do you see instead?

Expect this not to cause an iloop but it does.

What version of the product are you using? On what operating system?
r 9a1b79b34515

Please provide any additional information below.

Here's a simple example:

var m = {};
Model.observe(m, 'foo', function() {
  Model.dirtyCheck();
});
m.foo = 'bar';
Model.dirtyCheck();

I think we can fix this by adding a simple re-entrancy check to dirtyCheck, 
something like: 

if (notificationQueueIsRunning)
      return;

before this loop in Model.dirtyCheck

do {
  observedList.forEach(addNotification);
  startNotifications();
} while (notificationsMade)

As it currently is, notificationsMade is managed in startNotifications; 
however, startNotifications itself has a re-entrancy check. Therefore, if 
notificationsMade is true and Model.dirtyCheck is called, the loop in 
dirtyCheck never exists.

Original issue reported on code.google.com by [email protected] on 11 Jan 2013 at 6:25

Binding input.value to a missing path puts "undefined" in the input field

For example: use_cases/simple.html.

It makes it look like something is broken. We should at least fix the examples 
to create empty string default values.

More importantly, we should think about what the right behavior is here.

Original issue reported on code.google.com by [email protected] on 19 Apr 2011 at 12:00

Provide a way to destroy (possibly update) views

Currently, there's no way to destroy a view: its observers live forever, which 
keeps them and the view from being GCed.

The main reason we need view destruction is because assigning a new view is the 
only way to get the framework to re-sort (say, after the user clicks on a title 
header). So it might be that providing a way to update an existing view would 
be sufficient.

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 11:39

Model.setValueAtPath does not update the tracker value

The following is a bit strange:

var m = {a: {b: 1}};
var changed = false;
function observer() {
  changed = true;
}
Model.observe(m, 'a.b', observer);
Model.dirtyCheck();
Model.setValueAtPath(m, 'a.b', 2);
changed = false;
m.a.b = 1;
Model.dirtyCheck();
assertTrue(changed);

This does not notify the observer since the value has not change since last 
time we dirty checked. However, this causes problem because the ValueBinding 
sets the value of the input and we need to synchronize the tracker.

Original issue reported on code.google.com by [email protected] on 11 Jan 2013 at 9:28

capabilities/model_perf.html degrades over time

Initially, it held steady at around 5-10x. Now it gets work with every refresh. 
We must have a leak somewhere.

Original issue reported on code.google.com by [email protected] on 18 Apr 2011 at 11:56

Patch for /trunk/capabilities/model_perf.html

Not using style anyway.

Original issue reported on code.google.com by [email protected] on 12 May 2011 at 10:51

Attachments:

• model_perf.html.patch

Add mdv-phonecat example to use_cases

Needs to be pulled over from old repo

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:17

Make 1-pass operation work with Safari

Looks like it's almost there.

Original issue reported on code.google.com by [email protected] on 19 Apr 2011 at 12:04

Finish implementing fully-async operation for templates.

Almost there. The tricky part is now allowing phantomInstances to contain 
template iterators

Original issue reported on code.google.com by [email protected] on 19 Apr 2011 at 12:01

Implement comment node terminators for template usage/instances

And update the docs

Original issue reported on code.google.com by [email protected] on 27 Apr 2011 at 9:18