Bug 508903

Summary:	Rewrite QuickAccess using a Virtual Table
Product:	[Eclipse Project] Platform	Reporter:	Patrik Suzzi <psuzzi>
Component:	UI	Assignee:	Platform-UI-Inbox <Platform-UI-Inbox>
Status:	NEW ---	QA Contact:
Severity:	enhancement
Priority:	P3	CC:	daniel_megert, gautier.desaintmartinlacaze, simon.scholz
Version:	4.7
Target Milestone:	---
Hardware:	PC
OS:	Windows NT
See Also:	https://bugs.eclipse.org/bugs/show_bug.cgi?id=495016
Whiteboard:

Description Patrik Suzzi

2016-12-08 08:28:40 EST

Quick Access uses a Table with "ugly hacks" to implement a Virtual Table. This is a source of errors. Indeed, as seen in bug 495016, when we edit the QuickAccess table behavior, we face unpredictable timing exceptions.

Please, read this comment to fully understand the underlying problem:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=495016#c6

Scope of this bug is to fully rewrite the QuickAccess Table, using a Virtual Table instead of a simple Table.

Comment 1 Patrik Suzzi

2016-12-21 12:40:14 EST

During the Eclipse Platform Hackathon in Hamburg, Simon and I we spent some time analysing the code, and we agree on the followings:

- We should create a Table with SWT.VIRTUAL
- We should use a subclass of LazyContentProvider (see #1)
- The actual code is complex, and we should isolate the data displayed w.r.t. the table which is not the case.

#1: http://www.vogella.com/tutorials/EclipseJFaceTableAdvanced/article.html#virtual-tables-with-lazycontentprovider