Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CertXplorer - Normalize the View model #14

Open
odalet opened this issue Jan 2, 2014 · 0 comments
Open

CertXplorer - Normalize the View model #14

odalet opened this issue Jan 2, 2014 · 0 comments

Comments

@odalet
Copy link
Owner

odalet commented Jan 2, 2014

All tool windows should implement IView (or IDocumentView) and provide an activation/deactivation capability. Also the same model for embedding classes should be applied.

For instance, we have:

  • Asn1DocumentView (a DockContent Form) contains Asn1DocumentControl which contains Asn1Viewer (which is an ISelectionSource), but
  • CertificateListWindow (which is a ToolWindow which is a DockContent) contains CertificateListControl (which is a ServicedUserControl and an ISelectionSource)...

Some refactoring may also be useful: the CertManager and Asn1Decoder namespaces contain non related classes (GlobalSelectionService for instance) !

Achieving this would help resolve issues #7 and #13 in a non hacky way!

This issue was migrated from issue #14 in former Delta repository
Originally created by @odalet and assigned to @odalet
Originally created at 2013-10-13T16:06:48+00:00
Originally updated at 2013-10-13T16:07:13+00:00
Original url: https://api.github.com/repos/odalet/Delta/issues/14

@ghost ghost assigned odalet Jan 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant