[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
|
+1, sounds good..
Eric will you do the changes to the spec?
--
Gorkem
On Wed, Apr 30, 2008 at 9:34 PM, Mark Rogalski <rogalski@xxxxxxxxxx> wrote:
>
> Then your suggested changes sound good to me.
>
>
>
>
>
> Eric MF Hsu <mfhsu@xxxxxxxxxx>
> Sent by: dsdp-ercp-dev-bounces@xxxxxxxxxxx
>
> 04/30/2008 09:16 AM
>
>
>
> Please respond to
> DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
> To DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
> cc
>
> Subject Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
>
>
>
>
>
>
> No, it wouldn't break the compatibility with SWT.
> SWT introduces index field of Event class from eclipse 3.2, which is used
> for virtual style of Table/Tree support.
> You could refer to SWT javadoc (
> http://help.eclipse.org/help33/topic/org.eclipse.rap.help/help/html/reference/api/org/eclipse/swt/widgets/Event.html#index
> ).
> This article
> (http://www.eclipse.org/articles/Article-SWT-Virtual/Virtual-in-SWT.html )
> demostrates how to use Event.index in virtual Table and Tree widget.
>
> Sincerely,
>
> Eric MF Hsu (徐銘法)
> eRCP/Lotus Expeditor device development
> China Software Development Lab, IBM Taiwan
> E-Mail : mfhsu@xxxxxxxxxx
> Mark Rogalski <rogalski@xxxxxxxxxx>
>
>
> Mark Rogalski <rogalski@xxxxxxxxxx>
> Sent by: dsdp-ercp-dev-bounces@xxxxxxxxxxx
>
> 2008/04/30 下午 09:03
>
>
> Please respond to
> DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
>
>
> To
> DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
> cc
>
>
> Subject
> Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
>
>
>
>
>
> Wouldn't changing the Event class create an incompatibility with SWT?
>
>
> Eric MF Hsu <mfhsu@xxxxxxxxxx>
> Sent by: dsdp-ercp-dev-bounces@xxxxxxxxxxx
>
> 04/30/2008 04:01 AM
>
>
>
> Please respond to
> DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
>
> To "DSDP ercp list" <dsdp-ercp-dev@xxxxxxxxxxx>
>
> cc
>
> Subject Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
>
>
>
>
>
>
>
> Hi....All
>
> Since MultiPageDialog extends Dialog which doesn't have addListener() and
> removeListener() methods to support SWT.Virtual functionality, I suggest to
> add new three methods in eSWT MPD and Event javadoc in below.
>
> 1. public void addListener(int eventType, Listener listener)
> 2. public void removeListener(int eventType, Listener listener)
> 3."int index" field in Event class indicates the index number for items
> transferred into SetData block.
>
> Any suggestion? Thanks
>
> Sincerely,
>
> Eric MF Hsu (徐銘法)
> eRCP/Lotus Expeditor device development
> China Software Development Lab, IBM Taiwan
> E-Mail : mfhsu@xxxxxxxxxx
> Eric MF Hsu/Taiwan/IBM
> Eric MF Hsu/Taiwan/IBM
>
> 2008/04/16 上午 12:14
>
>
>
>
> To
> "DSDP ercp list" <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
>
> cc
>
>
>
> Subject
> Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
>
>
>
>
>
> From the flexibility perspective, I am in favor of option 1.
> I'd suggest to add one more field(index) in Event class.
> Since SWT ultilizes it to pass the item index in handleEvent() block,
> developer can take advantage of event.index to get corresponding index of
> transferred item.
> Please refer to SWT javadoc and the article
> (http://www.eclipse.org/articles/Article-SWT-Virtual/Virtual-in-SWT.html)
> for more details.
> Thank you all.
>
> Sincerely,
>
> Eric MF Hsu (徐銘法)
> eRCP/Lotus Expeditor device development
> China Software Development Lab, IBM Taiwan
> E-Mail : mfhsu@xxxxxxxxxx
> "Gorkem Ercan" <gercan@xxxxxxx>
> "Gorkem Ercan" <gercan@xxxxxxx>
> Sent by: dsdp-ercp-dev-bounces@xxxxxxxxxxx
>
> 2008/04/14 下午 10:59
>
>
>
> Please respond to
> DSDP ercp list <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
>
>
>
> To
> "DSDP ercp list" <dsdp-ercp-dev@xxxxxxxxxxx>
>
>
>
> cc
>
>
>
> Subject
> Re: [dsdp-ercp-dev] RE: eRCP MultiPageDialog Virtual support issue
>
>
>
>
>
>
> I am in favor of option 1. The composite should be returned from the
> createPage() call, it enables more flexible solutions to the use of
> virtual. Comparing the composites is one option but we can also pass
> the page number using one of the fields on Event class.
> --
> Gorkem
>
> 2008/4/14 Mark Rogalski <rogalski@xxxxxxxxxx>:
> >
> > Using the widgetSelected event for MPD virtual support is dangerous. We
> > don't know that filling in the page composite at that point will
> necessarily
> > get that content rendered. It may in WM, but not other implementations.
> > Therefore, i suggest sticking with setData().
> > A setPageCount(int) method does not work because page titles are not
> > supplied and the MPD tabs would be unnamed until the page is actually
> > created.
> >
> > Here are two alternatives:
> >
> > 1) As normally done now, program creates pages before dialog is
> > displayed but does not populate them until handleEvent is called. MPD
> code
> > must create the composite, but that is only one widget.. When handleEvent
> is
> > called program can match provided composite to the originally created
> one.
> >
> > 2) Instead of getting a composite back from createPage, that
> method
> > could return null when SWT.VIRTUAL is used. Then no widgets are created
> > until needed. However, for that to work, the handleEvent must pass both
> the
> > Composite as well as a page number, so the program knows what page to
> > populate. Can this be done?
> >
> >
> >
> >
> > Eric MF Hsu <mfhsu@xxxxxxxxxx>
> >
> > 04/13/2008 11:57 AM
> >
> >
> > To <Gorkem.Ercan@xxxxxxxxx>, Mark Rogalski/Austin/IBM@IBMUS, Uriel KL Liu
> > <liukl@xxxxxxxxxx>
> >
> > cc Daniel DL Su <dlsu@xxxxxxxxxx>
> >
> > Subject eRCP MultiPageDialog Virtual support issue
> >
> >
> >
> >
> >
> >
> >
> > Hi....Gorkem et al
> >
> > I have some issues regarding the new function of virtual support of
> > MultipageDialog.
> >
> > The sample code in mobile spec is as below
> > final MultiPageDialog dialog = new MultiPageDialog(shell,
> SWT.PRIMARY_MODAL
> > | SWT.VIRTUAL);
> > dialog.createPage("Page 1", null);
> > dialog.createPage("Page 2", null);
> > dialog.createPage("Page 3", null);
> > dialog.addListener(SWT.SetData, new Listener() {
> > public void handleEvent(Event event) {
> > Composite page = (Composite)event.item;
> > //Populate rest of the UI
> > }
> >
> > Since dialog.createPage(...) will return a instantiated Composite to
> > developer, they can just use it without writing any codes in
> handleEvent()
> > block. This will break the spirit of virtual style and make virtual
> useless.
> > I'd suggest to add a new method ( setPageCount(int PageCount)). Then the
> > sample code will be :
> >
> > final MultiPageDialog dialog = new MultiPageDialog(shell,
> SWT.PRIMARY_MODAL
> > | SWT.VIRTUAL);
> > dialog.addListener(SWT.SetData, new Listener() {
> > public void handleEvent(Event event) {
> > Composite page = (Composite)event.item;
> > //Populate rest of the UI
> > }
> > dialog.setPageCount(3);
> >
> >
> > However, we noticed that MPD has the method
> > addSelectionListener(SelectionListener listener) {} already. This will be
> > notified while one page is selected.
> > The method can be used to simulate similar functionality of virtual MPD.
> > The sample code can be as below.
> > final MultiPageDialog dialog = new MultiPageDialog(shell,
> > SWT.PRIMARY_MODAL);
> > dialog.createPage("Page 1", null);
> > dialog.createPage("Page 2", null);
> > dialog.createPage("Page 3", null);
> > dialog.addSelectionListener(new SelectionListener() {
> > public void widgetDefaultSelected(SelectionEvent event) {
> > }
> >
> > public void widgetSelected(SelectionEvent event) {
> > //Populate rest of the UI
> > }
> > });
> > However, developer needs to get desired page to complete work, It's not
> > convenient as virtual MPD.
> >
> > Do you have any suggestion? Thank you.
> >
> >
> > Sincerely,
> >
> > Eric MF Hsu (徐銘法)
> > eRCP/Lotus Expeditor device development
> > China Software Development Lab, IBM Taiwan
> > E-Mail : mfhsu@xxxxxxxxxx Tel : 886-2-8170-6571
> >
> >
> > _______________________________________________
> > dsdp-ercp-dev mailing list
> > dsdp-ercp-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
> >
> >
> _______________________________________________
> dsdp-ercp-dev mailing list
> dsdp-ercp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
>
> _______________________________________________
> dsdp-ercp-dev mailing list
> dsdp-ercp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
> (See attached file: pic22833.gif)(See attached file:
> pic05239.gif)_______________________________________________
> dsdp-ercp-dev mailing list
> dsdp-ercp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
> _______________________________________________
> dsdp-ercp-dev mailing list
> dsdp-ercp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
>
>
> _______________________________________________
> dsdp-ercp-dev mailing list
> dsdp-ercp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-ercp-dev
>
>