Crash/"bug" in docking pane keyboard navigation

Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic

The docking pane keyboard navigation does not work if a document can have more than 1 view.

In my case there may be 0 or up to 70 different views of the same document, as well as multiple documents. I use the tabbed MDI to switch between the different workspaces.

The document is created using CMyDocument::CreateObject because there is no initial view, so the m_pDocTemplate member is NULL.

In CXTPDockingPaneWindowSelect::OnPaint you are making the wrong assumption that every document was created from a template (pDocument->GetDocTemplate()->GetDocString), which result in a crash because GetDocTemplate returns NULL.

Also the keyboard navigation is quite useless if there are more than one view per document because you can't distinguish between the different views. The caption "Active Files" is also wrong in that case.

It would be great if you could check wheter a document has multiple views and then add the view caption to the items, i.e. "My file: My view". Or a flag that sets wheter to display the file name or view caption.

And last but not least the navigation popup doesn't look good with the SkinFramework.

Author	Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic
ABuenger Members Profile Send Private Message Find Members Posts Add to Buddy List Newbie Joined: 02 February 2006 Status: Offline Points: 1075	Post Options Post Reply Quote ABuenger Report Post Thanks(0) Quote Reply Topic: Crash/"bug" in docking pane keyboard navigation Posted: 23 February 2007 at 1:06pm
	The docking pane keyboard navigation does not work if a document can have more than 1 view. In my case there may be 0 or up to 70 different views of the same document, as well as multiple documents. I use the tabbed MDI to switch between the different workspaces. The document is created using CMyDocument::CreateObject because there is no initial view, so the m_pDocTemplate member is NULL. In CXTPDockingPaneWindowSelect::OnPaint you are making the wrong assumption that every document was created from a template (pDocument->GetDocTemplate()->GetDocString), which result in a crash because GetDocTemplate returns NULL. Also the keyboard navigation is quite useless if there are more than one view per document because you can't distinguish between the different views. The caption "Active Files" is also wrong in that case. It would be great if you could check wheter a document has multiple views and then add the view caption to the items, i.e. "My file: My view". Or a flag that sets wheter to display the file name or view caption. And last but not least the navigation popup doesn't look good with the SkinFramework.
	Codejock support

ABuenger Members Profile Send Private Message Find Members Posts Add to Buddy List Newbie Joined: 02 February 2006 Status: Offline Points: 1075	Post Options Post Reply Quote ABuenger Report Post Thanks(0) Quote Reply Posted: 23 February 2007 at 5:01pm
	CDocument::GetDocTemplate Return Value: A pointer to the document template for this document type, or NULL* if the document is not managed by a document template.*
	Codejock support

Oleg Members Profile Send Private Message Find Members Posts Add to Buddy List Senior Member Joined: 21 May 2003 Location: United States Status: Offline Points: 11234	Post Options Post Reply Quote Oleg Report Post Thanks(0) Quote Reply Posted: 25 February 2007 at 12:25am
	Hello, Thanks, yes you right. I thought CDocument can be used only with DocTemplate.
	Oleg, Support Team CODEJOCK SOFTWARE SOLUTIONS