WAScreenshot>>uiProcess

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

WAScreenshot>>uiProcess

timrowledge
In the seaside I have loaded right now - a very recent install - the WAScreenshot>>uiProcess method tries to run

^ UIManager default uiProcess

... which fails because UIManager does not implement this. The current Squeak images both 5.1 & 5.2 would do this with ^*Project current* uiProcess.

tim
--
tim Rowledge; [hidden email]; http://www.rowledge.org/tim
Strange OpCodes: XYZZY: Branch and Play Adventure


_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAScreenshot>>uiProcess

Johan Brichau-2
Dear Tim,

Thank you.
Can you please submit this issue via https://github.com/SeasideSt/Seaside/issues ?

I notice this code is in a Pharo-specific package, which is included when loaded in Squeak but was never correctly adapted to work on it.

I see three options:
- Extend Grease with the necessary abstractions for both platforms and the code in the package should be rewritten in terms of those abstractions
- Create a Squeak-specific package for it and include that in the load for Squeak
- Leave out this functionality when loading in Squeak

Cheers
Johan

On 12 Oct 2018, at 02:21, tim Rowledge <[hidden email]> wrote:

In the seaside I have loaded right now - a very recent install - the WAScreenshot>>uiProcess method tries to run

^ UIManager default uiProcess

... which fails because UIManager does not implement this. The current Squeak images both 5.1 & 5.2 would do this with ^*Project current* uiProcess.

tim
--
tim Rowledge; [hidden email]; http://www.rowledge.org/tim
Strange OpCodes: XYZZY: Branch and Play Adventure


_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev


_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAScreenshot>>uiProcess

timrowledge


> On 2018-10-12, at 12:12 AM, Johan Brichau <[hidden email]> wrote:
>
> Dear Tim,
>
> Thank you.
> Can you please submit this issue via https://github.com/SeasideSt/Seaside/issues ?

No problem; done.


tim

Give a man a compliment, he’ll feel good for a day.
Teach a man to fish for compliments and he’ll never feel good enough for the rest of his life.

_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev