Updated Seaside Control Panel for Pharo 6 / Seaside 3.2

> Gesendet: Mittwoch, 24. Mai 2017 um 16:27 Uhr
> Von: "Sven Van Caekenberghe" <[hidden email]>
> An: "Pharo Development List" <[hidden email]>, "Seaside - developer list" <[hidden email]>
> Betreff: [Pharo-dev] Updated Seaside Control Panel for Pharo 6 / Seaside 3.2
>
> Hi,
>
> I updated the Seaside Control Panel for Pharo 6 / Seaside 3.2. As far as I can see, it works fine now.
>
> I don't know the exact procedure to get this into Seaside proper, hence I include the MC package.
>
> It is obviously important to get this in before we release Pharo 6, but it is probably also a good idea that somebody else has a look at what I did.
>
> ===
> Name: Seaside-Pharo-Tools-Spec-SvenVanCaekenberghe.20
> Author: SvenVanCaekenberghe
> Time: 24 May 2017, 4:17:49.490067 pm
> UUID: b1168881-a70c-0d00-9de6-8d500cd45859
> Ancestors: Seaside-Pharo-Tools-Spec-JohanBrichau.19
>
> Updated the Seaside Control Panel for Pharo 6 / Seaside 3.2 using latest Spec conventions
>
> - Add a refresh button to the main toolbar to sync with changes done outside the UI (add a #refreshAdaptors action and use it for add/remove updating as well)
> - Inform the user when remove fails due to a still running adaptor
> - Use IconListModel instead of NewListModel
> - Fix selection maintenance on the list of adaptors
> - Fix the context menu on the list of adaptors
> - Use new recommended icon access method
> - Added help strings to the buttons in the main toolbar
> - Added a proper About text
> - Add a window icon
> - Remove unused state/action stuff from the toolbar implementation
> ===
>
> Sven
>
>
>
> PS: The 'Spec UI Framework' book was helpful as was Nicolai's help on the ML - Thank you.
>
>

> On 24 May 2017, at 22:34, Torsten Bergmann <[hidden email]> wrote:
>
> Hi Sven,
>
> I uploaded the MCZ to the Seaside repo htto://smalltalkhub.com/mc/Seaside/Seaside32/main
> and tested it. Looks fine and all is working for me.
>
> The only thing I'm not sure about is that Philippe (PMM) in Seaside-Pharo-Tools-Spec-pmm.21
> made a distinction to either use NewListModel or FastTableModel.
> The issue was also noted in https://github.com/SeasideSt/Seaside/issues/869
>
> I guess this not necessary anymore as Pharo 6 has FastTableModel only.
>
> Additionally:
> ============
> I extended your version so that "Browse" opens a real webbrowser when two conditions are
> met:  - the Pharo image includes the WebBrowser package (which one can load from catalog)
>      - teh adaptor is running
>
> In any other case it has the same behavior as before by just inspecting the adaptor.
>
> This extension can be found in "Seaside-Pharo-Tools-Spec-TorstenBergmann.28" in the
> same repository.
>
> Thanks
> Torsten
>
>> Gesendet: Mittwoch, 24. Mai 2017 um 16:27 Uhr
>> Von: "Sven Van Caekenberghe" <[hidden email]>
>> An: "Pharo Development List" <[hidden email]>, "Seaside - developer list" <[hidden email]>
>> Betreff: [Pharo-dev] Updated Seaside Control Panel for Pharo 6 / Seaside 3.2
>>
>> Hi,
>>
>> I updated the Seaside Control Panel for Pharo 6 / Seaside 3.2. As far as I can see, it works fine now.
>>
>> I don't know the exact procedure to get this into Seaside proper, hence I include the MC package.
>>
>> It is obviously important to get this in before we release Pharo 6, but it is probably also a good idea that somebody else has a look at what I did.
>>
>> ===
>> Name: Seaside-Pharo-Tools-Spec-SvenVanCaekenberghe.20
>> Author: SvenVanCaekenberghe
>> Time: 24 May 2017, 4:17:49.490067 pm
>> UUID: b1168881-a70c-0d00-9de6-8d500cd45859
>> Ancestors: Seaside-Pharo-Tools-Spec-JohanBrichau.19
>>
>> Updated the Seaside Control Panel for Pharo 6 / Seaside 3.2 using latest Spec conventions
>>
>> - Add a refresh button to the main toolbar to sync with changes done outside the UI (add a #refreshAdaptors action and use it for add/remove updating as well)
>> - Inform the user when remove fails due to a still running adaptor
>> - Use IconListModel instead of NewListModel
>> - Fix selection maintenance on the list of adaptors
>> - Fix the context menu on the list of adaptors
>> - Use new recommended icon access method
>> - Added help strings to the buttons in the main toolbar
>> - Added a proper About text
>> - Add a window icon
>> - Remove unused state/action stuff from the toolbar implementation
>> ===
>>
>> Sven
>>
>>
>>
>> PS: The 'Spec UI Framework' book was helpful as was Nicolai's help on the ML - Thank you.
>>
>>
> _______________________________________________
> seaside-dev mailing list
> [hidden email]
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

> On 25 May 2017, at 00:38, Sven Van Caekenberghe <[hidden email]> wrote:
>
> Thx Torsten.
>
> Is that enough to get it included in the main development line, what else has to happen ?
>
>> On 24 May 2017, at 22:34, Torsten Bergmann <[hidden email]> wrote:
>>
>> Hi Sven,
>>
>> I uploaded the MCZ to the Seaside repo htto://smalltalkhub.com/mc/Seaside/Seaside32/main
>> and tested it. Looks fine and all is working for me.
>>
>> The only thing I'm not sure about is that Philippe (PMM) in Seaside-Pharo-Tools-Spec-pmm.21
>> made a distinction to either use NewListModel or FastTableModel.
>> The issue was also noted in https://github.com/SeasideSt/Seaside/issues/869
>>
>> I guess this not necessary anymore as Pharo 6 has FastTableModel only.
>>
>> Additionally:
>> ============
>> I extended your version so that "Browse" opens a real webbrowser when two conditions are
>> met:  - the Pharo image includes the WebBrowser package (which one can load from catalog)
>>     - teh adaptor is running
>>
>> In any other case it has the same behavior as before by just inspecting the adaptor.
>>
>> This extension can be found in "Seaside-Pharo-Tools-Spec-TorstenBergmann.28" in the
>> same repository.
>>
>> Thanks
>> Torsten
>>
>>> Gesendet: Mittwoch, 24. Mai 2017 um 16:27 Uhr
>>> Von: "Sven Van Caekenberghe" <[hidden email]>
>>> An: "Pharo Development List" <[hidden email]>, "Seaside - developer list" <[hidden email]>
>>> Betreff: [Pharo-dev] Updated Seaside Control Panel for Pharo 6 / Seaside 3.2
>>>
>>> Hi,
>>>
>>> I updated the Seaside Control Panel for Pharo 6 / Seaside 3.2. As far as I can see, it works fine now.
>>>
>>> I don't know the exact procedure to get this into Seaside proper, hence I include the MC package.
>>>
>>> It is obviously important to get this in before we release Pharo 6, but it is probably also a good idea that somebody else has a look at what I did.
>>>
>>> ===
>>> Name: Seaside-Pharo-Tools-Spec-SvenVanCaekenberghe.20
>>> Author: SvenVanCaekenberghe
>>> Time: 24 May 2017, 4:17:49.490067 pm
>>> UUID: b1168881-a70c-0d00-9de6-8d500cd45859
>>> Ancestors: Seaside-Pharo-Tools-Spec-JohanBrichau.19
>>>
>>> Updated the Seaside Control Panel for Pharo 6 / Seaside 3.2 using latest Spec conventions
>>>
>>> - Add a refresh button to the main toolbar to sync with changes done outside the UI (add a #refreshAdaptors action and use it for add/remove updating as well)
>>> - Inform the user when remove fails due to a still running adaptor
>>> - Use IconListModel instead of NewListModel
>>> - Fix selection maintenance on the list of adaptors
>>> - Fix the context menu on the list of adaptors
>>> - Use new recommended icon access method
>>> - Added help strings to the buttons in the main toolbar
>>> - Added a proper About text
>>> - Add a window icon
>>> - Remove unused state/action stuff from the toolbar implementation
>>> ===
>>>
>>> Sven
>>>
>>>
>>>
>>> PS: The 'Spec UI Framework' book was helpful as was Nicolai's help on the ML - Thank you.
>>>
>>>
>> _______________________________________________
>> 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

> I’ll ping back here later today..
>
> johan
>
>> On 25 May 2017, at 00:38, Sven Van Caekenberghe <[hidden email]> wrote:
>>
>> Thx Torsten.
>>
>> Is that enough to get it included in the main development line, what else has to happen ?
>>
>>> On 24 May 2017, at 22:34, Torsten Bergmann <[hidden email]> wrote:
>>>
>>> Hi Sven,
>>>
>>> I uploaded the MCZ to the Seaside repo htto://smalltalkhub.com/mc/Seaside/Seaside32/main
>>> and tested it. Looks fine and all is working for me.
>>>
>>> The only thing I'm not sure about is that Philippe (PMM) in Seaside-Pharo-Tools-Spec-pmm.21
>>> made a distinction to either use NewListModel or FastTableModel.
>>> The issue was also noted in https://github.com/SeasideSt/Seaside/issues/869
>>>
>>> I guess this not necessary anymore as Pharo 6 has FastTableModel only.
>>>
>>> Additionally:
>>> ============
>>> I extended your version so that "Browse" opens a real webbrowser when two conditions are
>>> met:  - the Pharo image includes the WebBrowser package (which one can load from catalog)
>>>    - teh adaptor is running
>>>
>>> In any other case it has the same behavior as before by just inspecting the adaptor.
>>>
>>> This extension can be found in "Seaside-Pharo-Tools-Spec-TorstenBergmann.28" in the
>>> same repository.
>>>
>>> Thanks
>>> Torsten
>>>
>>>> Gesendet: Mittwoch, 24. Mai 2017 um 16:27 Uhr
>>>> Von: "Sven Van Caekenberghe" <[hidden email]>
>>>> An: "Pharo Development List" <[hidden email]>, "Seaside - developer list" <[hidden email]>
>>>> Betreff: [Pharo-dev] Updated Seaside Control Panel for Pharo 6 / Seaside 3.2
>>>>
>>>> Hi,
>>>>
>>>> I updated the Seaside Control Panel for Pharo 6 / Seaside 3.2. As far as I can see, it works fine now.
>>>>
>>>> I don't know the exact procedure to get this into Seaside proper, hence I include the MC package.
>>>>
>>>> It is obviously important to get this in before we release Pharo 6, but it is probably also a good idea that somebody else has a look at what I did.
>>>>
>>>> ===
>>>> Name: Seaside-Pharo-Tools-Spec-SvenVanCaekenberghe.20
>>>> Author: SvenVanCaekenberghe
>>>> Time: 24 May 2017, 4:17:49.490067 pm
>>>> UUID: b1168881-a70c-0d00-9de6-8d500cd45859
>>>> Ancestors: Seaside-Pharo-Tools-Spec-JohanBrichau.19
>>>>
>>>> Updated the Seaside Control Panel for Pharo 6 / Seaside 3.2 using latest Spec conventions
>>>>
>>>> - Add a refresh button to the main toolbar to sync with changes done outside the UI (add a #refreshAdaptors action and use it for add/remove updating as well)
>>>> - Inform the user when remove fails due to a still running adaptor
>>>> - Use IconListModel instead of NewListModel
>>>> - Fix selection maintenance on the list of adaptors
>>>> - Fix the context menu on the list of adaptors
>>>> - Use new recommended icon access method
>>>> - Added help strings to the buttons in the main toolbar
>>>> - Added a proper About text
>>>> - Add a window icon
>>>> - Remove unused state/action stuff from the toolbar implementation
>>>> ===
>>>>
>>>> Sven
>>>>
>>>>
>>>>
>>>> PS: The 'Spec UI Framework' book was helpful as was Nicolai's help on the ML - Thank you.
>>>>
>>>>
>>> _______________________________________________
>>> 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
>
> _______________________________________________
> seaside-dev mailing list
> [hidden email]
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

> Hi Sven,
>
> I uploaded the MCZ to the Seaside repo htto://smalltalkhub.com/mc/Seaside/Seaside32/main
> and tested it. Looks fine and all is working for me.
>
> The only thing I'm not sure about is that Philippe (PMM) in Seaside-Pharo-Tools-Spec-pmm.21
> made a distinction to either use NewListModel or FastTableModel.
> The issue was also noted in https://github.com/SeasideSt/Seaside/issues/869
>
> I guess this not necessary anymore as Pharo 6 has FastTableModel only.