SeasideSt / Seaside

The framework for developing sophisticated web applications in Smalltalk.
MIT License
519 stars 71 forks source link

JQDraggableFunctionalTest >> renderContentOn: assumes Symbol equals String #1260

Closed marianopeck closed 3 years ago

marianopeck commented 3 years ago

As we all know, the equality between Symbol and String is one of the big difference between Smalltalk dialects. In Pharo it is true while in other dialects like GemStone or VAST, it is false. Therefore, I propose to change the line

callbackDraggable: [ :e | top := (e at: #position) at:#top. left := (e at: #position) at:#left ];

to:

callbackDraggable: [ :e | top := (e at: #position) at: 'top'. left := (e at: #position) at: 'left' ];

As the keys are originally as String, that would work in all dialects.