DuneSt / MaterialDesignLite

This project prime goal is to bind the google's Material Design Lite project to Seaside and as second goal to build widgets on top of Material Design to help Seaside developers to create web applications with material design faster.
https://mdl.ferlicot.fr/
MIT License
40 stars 20 forks source link

Remove dependency to Morphic (#detectIndex:) #184

Closed pdebruic closed 6 years ago

pdebruic commented 6 years ago

https://github.com/DuneSt/MaterialDesignLite/blob/f3eda5e69e539ffcad80e905b114d50a88a37c5e/src/Material-Design-Lite-Widgets.package/MDLNestedList.class/instance/elementsDisplayList.st#L10

Is there something else tht would work here?

jecisc commented 6 years ago

I'll take a look tomorrow.

jecisc commented 6 years ago

This is weird, I do not see it in Morphic but in Collection-Abstract.

jecisc commented 6 years ago

pharo_2018-06-07_01-05-41

jecisc commented 6 years ago

Apparently, it changed in Pharo 7. Now it is not part of Morphic anymore.

Does that raise a problem for the Gemstone compatibility?

pdebruic commented 6 years ago

No I just have to add it to the list of extensions but I don't understand what using it gives you vs #indexOf: except that you don't have to test for =0

jecisc commented 6 years ago

@pdebruic #indexOf: takes an element. #detectIndex: takes a block.