Bug 546803 - [UI] Category Manifest editor should have source tab
Summary: [UI] Category Manifest editor should have source tab
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.11   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.12 M3   Edit
Assignee: Alexander Fedorov CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2019-04-29 02:43 EDT by Alexander Fedorov CLA
Modified: 2019-05-21 03:45 EDT (History)
2 users (show)

See Also:


Attachments
gif showing the image (766.63 KB, image/gif)
2019-05-09 04:08 EDT, Vikas Chandra CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Fedorov CLA 2019-04-29 02:43:13 EDT
Currently it is required to open category.xml with some general editor to control all the changes explicitly.

Category Manifest editor should have "Source" tab like Target editor.
Comment 1 Eclipse Genie CLA 2019-05-07 09:13:09 EDT
New Gerrit change created: https://git.eclipse.org/r/141715
Comment 3 Julian Honnen CLA 2019-05-08 05:20:04 EDT
Thanks, Alexander.
Comment 4 Alexander Fedorov CLA 2019-05-08 05:25:02 EDT
@Julian N&N?
Comment 5 Julian Honnen CLA 2019-05-08 05:28:05 EDT
(In reply to Alexander Fedorov from comment #4)
> @Julian N&N?

Yes, please ;)
Comment 6 Vikas Chandra CLA 2019-05-08 05:48:35 EDT
Whenever I click on Source tab and then click on the other 2 tabs, the editor shows as dirty.
Comment 7 Julian Honnen CLA 2019-05-08 06:30:01 EDT
(In reply to Vikas Chandra from comment #6)
> Whenever I click on Source tab and then click on the other 2 tabs, the
> editor shows as dirty.

I've had that issue as well when I used a 4.11 target platform. I'll re-check that tomorrow with an I-Build.
Comment 8 Alexander Fedorov CLA 2019-05-08 06:39:18 EDT
(In reply to Julian Honnen from comment #7)
> (In reply to Vikas Chandra from comment #6)
> > Whenever I click on Source tab and then click on the other 2 tabs, the
> > editor shows as dirty.
> 
> I've had that issue as well when I used a 4.11 target platform. I'll
> re-check that tomorrow with an I-Build.

I think it is not related with the patch, one need to click the second tab "Repository Properties"
Comment 9 Vikas Chandra CLA 2019-05-08 06:42:34 EDT
>> one need to click the second tab "Repository Properties"

Yes but once you save it, that problem doesn't happen again. 

But in this case, dirty mark comes every single time when you click on source.
Comment 10 Alexander Fedorov CLA 2019-05-08 07:42:19 EDT
(In reply to Vikas Chandra from comment #9)
> >> one need to click the second tab "Repository Properties"
> 
> Yes but once you save it, that problem doesn't happen again. 
> 
> But in this case, dirty mark comes every single time when you click on
> source.

I still think that added "Source" tab just highlighted the existing problem: added Bug 547088 to track
Comment 11 Vikas Chandra CLA 2019-05-09 00:41:30 EDT
>>I still think that added "Source" tab just highlighted the existing problem

I have fixed the scenario where the dirty mark issue was very visible post this change. The old problem still exists but it is only a 1 time event when stats are being set on that particular tab. It can also be seen as a 1 time initialization for that tab. I still think that it is a bug but it is a pre-existing bug

Can you test in tomorrow's I build if everything is working fine.
Comment 12 Julian Honnen CLA 2019-05-09 02:14:13 EDT
(In reply to Vikas Chandra from comment #11)
> >>I still think that added "Source" tab just highlighted the existing problem
> 
> I have fixed the scenario where the dirty mark issue was very visible post
> this change. The old problem still exists but it is only a 1 time event when
> stats are being set on that particular tab. It can also be seen as a 1 time
> initialization for that tab. I still think that it is a bug but it is a
> pre-existing bug
> 
> Can you test in tomorrow's I build if everything is working fine.

I20190508-1800 no longer has a dirty marker when switching between source and category definition.
Comment 13 Vikas Chandra CLA 2019-05-09 02:24:17 EDT
(In reply to Julian Honnen from comment #12)
> (In reply to Vikas Chandra from comment #11)
> > >>I still think that added "Source" tab just highlighted the existing problem
> > 
> > I have fixed the scenario where the dirty mark issue was very visible post
> > this change. The old problem still exists but it is only a 1 time event when
> > stats are being set on that particular tab. It can also be seen as a 1 time
> > initialization for that tab. I still think that it is a bug but it is a
> > pre-existing bug
> > 
> > Can you test in tomorrow's I build if everything is working fine.
> 
> I20190508-1800 no longer has a dirty marker when switching between source
> and category definition.

The problem exists in I20190508-1800. Do you have latest PDE code by any chance and then testing on the launched version. I have put a fix which fixes "dirty marker when switching between source and category definition"
Comment 14 Julian Honnen CLA 2019-05-09 02:53:13 EDT
Sorry, I thought your fix was already in I20190508-1800.

I still don't get the dirty marker from source tab (only from repository properties), not on I20190508-1800 nor with latest sources. Also no marker when I comment out your changes.
Comment 15 Vikas Chandra CLA 2019-05-09 04:08:47 EDT
Created attachment 278551 [details]
gif showing the image

Note that I save the file after seeing dirty mark using control S to show further problems exists whether or not you go to 1st tab or 2nd tab after clicking source tab
Comment 16 Julian Honnen CLA 2019-05-09 04:21:00 EDT
Ah, you have to activate repository properties once to trigger the bug. After that switching between source and category def will make the editor dirty (without your fix). With the fix it works.
Comment 17 Eclipse Genie CLA 2019-05-13 12:56:25 EDT
New Gerrit change created: https://git.eclipse.org/r/142091
Comment 19 Vikas Chandra CLA 2019-05-21 02:54:59 EDT
Can the bug submitter/bug assignee please verify this defect for 4.12 M3?
Comment 20 Alexander Fedorov CLA 2019-05-21 03:45:28 EDT
Eclipse SDK
Version: 2019-06 (4.12)
Build id: I20190520-1805
OS: Windows 10, v.10.0, x86_64 / win32
Java version: 1.8.0_211