Bug 405354 - Swing: hard-wired SwingScoutTable instantiation
Summary: Swing: hard-wired SwingScoutTable instantiation
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Scout (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Judith Gull CLA
QA Contact:
URL:
Whiteboard:
Keywords: ui
Depends on:
Blocks:
 
Reported: 2013-04-10 09:49 EDT by Andre Wegmueller CLA
Modified: 2021-08-19 11:13 EDT (History)
2 users (show)

See Also:
stephan.leichtvogt: kepler+


Attachments
Adds createTable(ITable) method to ISwingEnvironment (9.39 KB, patch)
2013-04-10 10:23 EDT, Andre Wegmueller CLA
judith.gull: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andre Wegmueller CLA 2013-04-10 09:49:47 EDT
Typically in a Swing Scout application instantiation of SwingScout* classes is done through the application specific AbstractSwingEnvironment factory methods. This is not the case for SwingScoutTable. The instantiation of this class is hard-wired in SwingScoutTableField, SwingScoutListBox and SwingScoutPlannerField. Today one must copy & paste the source-code of these three classes in order to replace SwingScoutTable with another implementation.

With this enhancement table instantiation is delegated to the Swing environment. A new method 'ISwingScoutTable createTable(ITable table)' will be added to the ISwingEnvironment. SwingScoutTableField, SwingScoutListBox and SwingScoutPlannerField will use this new method instead of the old call to 'new SwingScoutTable()'.
Comment 1 Andre Wegmueller CLA 2013-04-10 10:23:21 EDT
Created attachment 229559 [details]
Adds createTable(ITable) method to ISwingEnvironment
Comment 2 Jeremie Bresson CLA 2013-04-22 09:27:26 EDT
Patch pushed as Gerrit change:
https://git.eclipse.org/r/12095

This is a patch for L-Release (API Change).
Comment 3 Judith Gull CLA 2013-05-06 07:39:20 EDT
Currently on hold until we create a new branch.
Comment 5 Matthias Zimmermann CLA 2013-07-08 17:25:45 EDT
Bug closed, shipped with 3.9.0 (Kepler)