Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 334192

Summary: [client] nav-table places search results underneath breadcrumb, fix DOM structure funkiness
Product: [ECD] Orion Reporter: Susan McCourt <susan>
Component: ClientAssignee: Susan McCourt <susan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 0.2   
Target Milestone: 0.2   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Susan McCourt CLA 2011-01-12 21:57:13 EST
I'm not sure when this problem crept up, it may have been some time and I just noticed it.  You can see it in M4.

On the tree view, when you do a search in the search bar, or click on a saved favorite, the bread crumb bar and tree both get replaced with search results.  This is the intention. 

However, on the table view, the search results simply replace the tree table.  The breadcrumb bar remains.  This is not right.

While looking at this problem, there is funkiness that should be cleaned up in both navs:

- we reuse the inner tree id when placing progress or search results.  We probably need some kind of content id that remains constant and the tree would go inside that. 
- because we often blow away/replace the content, we can't rely on storing dom nodes in the explorer.  As such, I changed the glue code to pass in ids rather than dom nodes.  But we really shouldn't be replacing a *parent* node that gets passed in.  It's okay to muck with the internal guts underneath, but it seemed to me we might be blowing away or replacing a parent node which is not cool.  check into this.
Comment 1 Susan McCourt CLA 2011-02-08 19:23:36 EST
fixed.