Bug 457002 - [About] Make AboutDialog available for Eclipse 4 RCP application
Summary: [About] Make AboutDialog available for Eclipse 4 RCP application
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.5   Edit
Hardware: PC Windows 7
: P3 normal with 2 votes (vote)
Target Milestone: ---   Edit
Assignee: Alexander Fedorov CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 445600 548516
Blocks:
  Show dependency tree
 
Reported: 2015-01-08 05:01 EST by Fabian Miehe CLA
Modified: 2020-06-16 09:59 EDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Fabian Miehe CLA 2015-01-08 05:01:07 EST
In a pure e4 application the e3 AboutDialog is not available/useable.

A default AboutDialog would be very useful for us.
Comment 1 Lars Vogel CLA 2015-01-08 05:03:15 EST
Fabian, I know you are working on a patch for the existing about dialog. Looking forward to your Gerrit upload..
Comment 2 Fabian Miehe CLA 2015-01-08 05:07:24 EST
https://git.eclipse.org/r/39173

We have converted the e3 dialog to e4.

The help function is still missing.
I found Bug 445600 - [Help] Add setHelp method to EHelpService.

if the EHelpService work as required :-), i would use it in the AboutDialog plugin
Comment 3 Lars Vogel CLA 2015-01-08 05:15:34 EST
(In reply to Fabian Miehe from comment #2)
> The help function is still missing.
> I found Bug 445600 - [Help] Add setHelp method to EHelpService.
> 
> if the EHelpService work as required :-), i would use it in the AboutDialog
> plugin

I try to implement it latest next week now that I have a potential consumer. :-)
Comment 4 Fabian Miehe CLA 2015-01-08 07:19:21 EST
Hmmm my eclipse/gerrit config seems to be wrong. 
I updated the formater settings and now i have a new gerrit code review 
https://git.eclipse.org/r/#/c/39179/

Is it possible to merge https://git.eclipse.org/r/#/c/39179/ and 
https://git.eclipse.org/r/#/c/39173/ ?
Comment 5 Lars Vogel CLA 2015-01-08 08:13:42 EST
(In reply to Fabian Miehe from comment #4)
> Is it possible to merge https://git.eclipse.org/r/#/c/39179/ and 
> https://git.eclipse.org/r/#/c/39173/ ?

You can use interactive rebase to combine both and abondan the second one.
Comment 6 Lars Vogel CLA 2015-02-17 10:49:00 EST
No feedback from Fabian, getting late for 4.5 moving to 4.6.
Comment 7 Eclipse Genie CLA 2015-03-09 09:33:38 EDT
WARNING: this patchset contains 10423 new lines of code and may require a Contribution Questionnaire (CQ) if the author is not a committer on the project. Please see:https://wiki.eclipse.org/Project_Management_Infrastructure/Creating_A_Contribution_Questionnaire
Comment 8 Eclipse Genie CLA 2015-03-09 09:35:43 EDT
WARNING: this patchset contains 10423 new lines of code and may require a Contribution Questionnaire (CQ) if the author is not a committer on the project. Please see:https://wiki.eclipse.org/Project_Management_Infrastructure/Creating_A_Contribution_Questionnaire
Comment 9 Eclipse Genie CLA 2015-03-09 09:39:54 EDT
WARNING: this patchset contains 10424 new lines of code and may require a Contribution Questionnaire (CQ) if the author is not a committer on the project. Please see:https://wiki.eclipse.org/Project_Management_Infrastructure/Creating_A_Contribution_Questionnaire
Comment 10 Pawel Pogorzelski CLA 2016-02-18 12:11:13 EST
Guys, what's the status? Is there any chance you'll make it for 4.6?
Comment 11 Lars Vogel CLA 2016-04-20 12:10:42 EDT
Mass move to 4.7 as M7 is approaching. Please move back in case you are planning to fix it for Neon.
Comment 12 Lars Vogel CLA 2019-07-08 15:53:12 EDT
Alexander indicated that he is interested in working on this one.
Comment 13 Lars Vogel CLA 2019-08-13 12:13:59 EDT
Alexander, any update on this one? Is this still planned for M3? I'm asking because I see UI freezes in the old FilteredTree (see Bug 550009) and can easily fix it for the new FilteredTree (see Bug 550023).
Comment 14 Alexander Fedorov CLA 2019-08-14 03:31:52 EDT
(In reply to Lars Vogel from comment #13)
> Alexander, any update on this one? Is this still planned for M3? I'm asking
> because I see UI freezes in the old FilteredTree (see Bug 550009) and can
> easily fix it for the new FilteredTree (see Bug 550023).

@Lars, I don't think it is realistic to fit in 4.13 as we need to redesign the implementation to fit e4. It is done for System Info collectors, but contributable About dialog pages are still under development. 
I have an idea to extend E4 Workbench model to cover the custom About dialog page functionality. Not sure if it is still considered as recommended way to go.
Comment 15 Ralf Heydenreich CLA 2020-01-21 09:05:07 EST
Based on Fabian Miehe's proposal I've just created my own plugin for the About dialog. It's a pure E4 plugin without dependencies to org.eclipse.ui. But before I'm contributing it to the Eclipse project I want to discuss some points, although I'm unsure if this is the right place for it (or if it is better to discuss via mailing list).

Now some facts about the About Dialog :-)

- does not depend on org.eclipse.ui (if someone wants to use the compat layer one can use the old AboutDialog class)
- the dialog behaves mostly like the old one and shows about information from product file (of course!), installation details about features and plugins and has a system information view
- it can show the log file from instance area
- internally it uses the injection framework of Eclipse (only for a few classes)
- currently it uses some deprecated classes from org.eclipse.update package (didn't found a viable alternative for this)
- the messages are handled with NLS (but I wonder if we could use the TranslationService)
- where possible the old classes were used

A first impression can be seen in https://bitbucket.org/fakturamadev/fakturama-2/src/develop/Fakturama-Parent/org.fakturama.e4.ui.dialogs.ext.

If it's ok I'll create a Gerrit commit for further discussion and reviewing. If there are other hints please provide me with some information.
Comment 16 Alexander Fedorov CLA 2020-01-22 03:29:59 EST
(In reply to Ralf Heydenreich from comment #15)
> 
> A first impression can be seen in
> https://bitbucket.org/fakturamadev/fakturama-2/src/develop/Fakturama-Parent/
> org.fakturama.e4.ui.dialogs.ext.
> 
> If it's ok I'll create a Gerrit commit for further discussion and reviewing.
> If there are other hints please provide me with some information.

Hi Ralph!

Thank you for your effort!

As I can see the trick was to copy all the needed E3 classes inside the new e4 bundle. Well, it is possible in theory as a temporary solution, but we will need to have corresponding tasks to rework this. I recommend you to put all these types to some ".internal.e3." subpackage tree to highlight this technical debt and also to remove all the E3 code you do not need for E4. 

In general, the contribution is huge and will require CQ procedure to follow, see https://wiki.eclipse.org/Development_Resources/Contribution_Questionnaire
If you can split the contribution by self-sufficient parts that doesn't exceed 1k lines - it will be much eaisier to discuss and contribute it one by one.

Thanks again and please keep us posted,
AF
Comment 17 Alexander Fedorov CLA 2020-01-22 03:55:37 EST
(In reply to Alexander Fedorov from comment #16)

My apologies for the typo in your name, Ralf!
Comment 18 Ralf Heydenreich CLA 2020-01-30 04:18:53 EST
no problem :-)

I'll create some follow-up tickets for the new implementation.
Comment 19 Christoph Laeubrich CLA 2020-06-16 03:02:18 EDT
I think this would be really useful. I'm just wondering if it won't be possible to extract some of the classes into a public package that can be share between e4+e3?
Comment 20 Ralf Heydenreich CLA 2020-06-16 03:53:35 EDT
Hi,
I've created some follow-up tickets and implemented some functionalities. There are only two or three issues left which I want to implement asap. But for the moment I'm stuck with my own project. I think I can go further in a few days.
Comment 21 Christoph Laeubrich CLA 2020-06-16 03:58:10 EDT
That sounds good, do you mind reference the corresponding ticket here?
Comment 22 Ralf Heydenreich CLA 2020-06-16 04:10:49 EDT
* Bug #559693
* Bug #559694
* Bug #559695
* Bug #559696
* Bug #559697
* Bug #559698
Comment 23 Christoph Laeubrich CLA 2020-06-16 09:59:50 EDT
Thanks Ralf let me know if you need any help in accomplish this.