Hi,

On 28 Nov 2016, at 21:32, stepharo <stepharo@free.fr> wrote:

Hi

- Could we rename FastTable into Table?

not for Pharo 6 but Pharo 7 yes�� in fact, I think it should be called TableMorph.

(Also we need to refactor a lot, to eliminate the FT prefix, etc.)

- Then I miss an important design point. Why datasource returns Morph?

I do not get get why a data source should return UI element. To me it violates layers.

no, because that��s its purpose: to provide the table with the cell elements (which are by definition Morphs�� any kind of morphs).

a TableDataSource is not a spec, is the provider of cells.

I got inspired for Cocoa NSTableView design (and NSTableViewDataSource), and that��s what it does.

See for example here: https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/TableView/PopulatingView-TablesProgrammatically/PopulatingView-TablesProgrammatically.html

This function:

- (NSView *)tableView:(NSTableView *)tableView

   viewForTableColumn:(NSTableColumn *)tableColumn

                  row:(NSInteger)row

is the equivalent of our FTDataSource>>cellColumn:row:

Esteban

Stef

--
Using Opera's mail client: http://www.opera.com/mail/