Hi.  I'm submitting the patch for this work in the proper place.  Sorry for spamming the other forum - still learning the ropes.

• Original Post

I don't know if there is a ticket for this yet.

Thanks

Dave

I messed up w.r.t submitting patches.  I had read about signing the CLA a while ago, but in my enthusiam, I forgot to get it taken care of officially here at work.  I've set the wheels in motion, but it might take awhile.  I'm really sorry I messed up the process.  Please handle my posts as you see fit.  Unofficially, I have not given away any proprietary stuff, but I still need to ask for forgiveness since I forgot to ask for permission.

Thanks

Dave

Hi Dave-

Just letting you know that we reviewed and integrated portions of the patch you submitted, under the following ticket:

https://trac.clearcanvas.ca/source/ticket/4413

Ultimately we came to the conclusion that having the CommandHistory property delegate up to the parent host was not an appropriate solution across the board, because the user experience of sharing a command history between pages of a container that supports pages (tab, navigator, etc) may not be acceptable.  Therefore this solution was adopted only for the SplitComponentContainer and SimpleComponentContainer.

Otherwise, the patch looked good from a quality/coding style point of view.  The only significant change I made was to rename the SubHost class to ContainedComponentHost, and make it protected rather than public.

Thanks again for the contribution.

j