Eclipse Community Forums: Remote Application Platform (RAP)

Help

Home

Home » Eclipse Projects » Remote Application Platform (RAP) » [GEF] "Drag-Job" on progress view

Show: Today's Messages :: Show Polls :: Message Navigator

[GEF] "Drag-Job" on progress view [message #728545]

Fri, 23 September 2011 13:10

Arnaud MERGEY

Messages: 243
Registered: March 2010
Location: France

Senior Member

Hello,

Using GEF in RAP, "Drag-Job" jobs are displayed on progress view. It would probably be cleaner to hide these jobs by having them as system jobs.

I attach a patch, if it makes sense to hide them.

Regards,
Arnaud

Attachment: patch.txt
(Size: 1.31KB, Downloaded 200 times)

Report message to a moderator

Re: [GEF] "Drag-Job" on progress view [message #728595 is a reply to message #728545]

Fri, 23 September 2011 14:07

Rüdiger Herrmann

Messages: 335
Registered: July 2009

Senior Member

Hi Arnaud,

thanks for the patch. I would also agree to hide this kind of Jobs. Even
though the change is small, I suggest to first submit the patch to GEF.
If it gets accepted there, it can be integrated into RAP and we avoid
that both projects diverge unnecessarily.

More generally, to contribute a patch to RAP (or any other Eclipse
project), it should be attached to a bug. This makes it easy to keep a
clean IP log [1].

Cheers,
Rüdiger

[1] http://wiki.eclipse.org/Development_Resources/IP_Log#Contributions

On 23.09.2011 15:10, Arnaud MERGEY wrote:
> Hello,
>
> Using GEF in RAP, "Drag-Job" jobs are displayed on progress view. It would probably be cleaner to hide these jobs by having them as system jobs.
>
> I attach a patch, if it makes sense to hide them.
>
> Regards,
> Arnaud
--
Rüdiger Herrmann

Twitter: @EclipseRAP
Blog: http://eclipsesource.com/blogs/

Professional services for RAP and RCP?
http://eclipsesource.com/services/rap/

Report message to a moderator

Re: [GEF] "Drag-Job" on progress view [message #728612 is a reply to message #728595]

Fri, 23 September 2011 14:39

Arnaud MERGEY

Messages: 243
Registered: March 2010
Location: France

Senior Member

Hello Rudiger
I think the Drag-Job thing is specific to RAP as it is created with a class called RAPDragTracker, and also with comment // HACK for DND in RAP
, so I think my patch is relevant in RAP port for GEF.
But before creating a bug to attach my patch, I wanted to be sure, it is why I had posted it here first. Could you confirm me I can assign the bug to rap incubator component ?
Thanks and best regards,
Arnaud

Report message to a moderator

Re: [GEF] &quot;Drag-Job&quot; on progress view [message #728619 is a reply to message #728612]

Fri, 23 September 2011 14:44

Rüdiger Herrmann

Messages: 335
Registered: July 2009

Senior Member

On 23.09.2011 16:39, Arnaud MERGEY wrote:
> Hello Rudiger
> I think the Drag-Job thing is specific to RAP as it is created with a
> class called RAPDragTracker, and also with comment // HACK for DND in RAP
> , so I think my patch is relevant in RAP port for GEF.
I see, sorry that I didn't read the patch properly.

> But before creating a bug to attach my patch, I wanted to be sure, it is
> why I had posted it here first. Could you confirm me I can assign the
> bug to rap incubator component ?
Yes, even if it is a hack that gets removed somewhen I agree that the
Job should be marked as 'system'.

> Thanks and best regards,
> Arnaud
>
>
>
--
Rüdiger Herrmann

Twitter: @EclipseRAP
Blog: http://eclipsesource.com/blogs/

Professional services for RAP and RCP?
http://eclipsesource.com/services/rap/

Report message to a moderator