Bug 508903 - Rewrite QuickAccess using a Virtual Table
Summary: Rewrite QuickAccess using a Virtual Table
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.7   Edit
Hardware: PC Windows NT
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-08 08:28 EST by Patrik Suzzi CLA
Modified: 2016-12-21 12:40 EST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Patrik Suzzi CLA 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 CLA 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