Single component redraw fix

[dhinesh03]

Applied single component redraw patch on the current master.

[Naddiseo]

https://jslinterrors.com/dont-make-functions-within-a-loop

[dhinesh03]

@Naddiseo Thanks for pointing it out.

[Naddiseo]

trailing whitespace

[Naddiseo]

How does redrawSelf get set initially?

Is there a way to merge the for-loop with another for-loop called indirectly earlier on in the function?

[dhinesh03]

@Naddiseo

1. We should set redrawSelf as empty function where ever needed.

redrawSelf = function () {
        };

Refer @jonahx 's codepen for more details.
2) I've tried, but the cached object will be overwritten on the following line

cached = reconstructCached(data, attrs, children, node, namespace, views, controllers); 

[avesus]

@dhinesh03 please, consider adding a documentation for controller.redrawSelf(). Also, could we write a test for this?

[dhinesh03]

@avesus Busy with my work, Will add doc soon.

[sebastiansandqvist]

@dhinesh03 I'd definitely like to see something like this in a future release. What is the status of this pr?

[dead-claudia]

Closing for now (cleaning up old PRs). Please open a new issue or PR if you want this to be fixed/added/implemented.

[jonahx]

Closing it and reopening it seems like it makes things messier, to me. I'm still interested in this, and based on thread comments no reason to think other people's interest has changed.

[dead-claudia]

You want to run a diff for yourself? It has tons of merge conflicts, and is best just redone.

[jonahx]

I’m happy to redo it if there are any plans to merge it. If not,
nothing has changed from the original PR, so it’s probably more
accurate to close it as WON’T FIX.

On 10 Dec 2015, at 19:18, Isiah Meadows wrote:

You want to run a diff for yourself? It has tons of merge conflicts,
and is best just redone.

---

Reply to this email directly or view it on GitHub:
#743 (comment)

[dead-claudia]

If you clean this up (trust me in that you'd have to deal with a massive
merge conflict), I can reopen it once it's updated.

On Thu, Dec 10, 2015, 21:07 Jonah [email protected] wrote:

I’m happy to redo it if there are any plans to merge it. If not,
nothing has changed from the original PR, so it’s probably more
accurate to close it as WON’T FIX.

On 10 Dec 2015, at 19:18, Isiah Meadows wrote:

You want to run a diff for yourself? It has tons of merge conflicts,
and is best just redone.

---

Reply to this email directly or view it on GitHub:
#743 (comment)

—
Reply to this email directly or view it on GitHub
#743 (comment).

[willycamargo]

Any plans to implement and merge it? It is a great feature for some applications.

My application has many components working at the same time, and sometimes this feature would be good.