Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 132208

Summary: keep repository task editor open after submission, refreshing contents
Product: z_Archived Reporter: Eugene Kuleshov <ekuleshov>
Component: MylynAssignee: Robert Elves <robert.elves>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: mik.kersten, simcoen, wmitsuda
Version: unspecified   
Target Milestone: 2.0   
Hardware: PC   
OS: Windows All   
Whiteboard:
Attachments:
Description Flags
mylar/context/zip none

Description Eugene Kuleshov CLA 2006-03-16 12:50:40 EST
When you add new comment to some bugzilla issue and hit submit, the editor jumps to next issue like in web UI. I don't think it is necessary in embedder issue editor
Comment 1 Mik Kersten CLA 2006-03-17 10:23:11 EST
I'm not sure I understand.  Do you mean in the "Browser" tab?  We don't have a way of controlling that--that's just a bridge to the Web UI. 
Comment 2 Eugene Kuleshov CLA 2006-03-17 10:26:44 EST
No. This is happening in form editor
Comment 3 Mik Kersten CLA 2006-03-17 11:28:20 EST
How does it do that?  The current behavior is that after submission the editor closes itselfand I've never seen anything to the contrary.  Perhaps you had another editor underneath?
Comment 4 Eugene Kuleshov CLA 2006-03-17 12:01:36 EST
Maybe you right. I didn't expect it to close... Will keep an eye on this.
Comment 5 Mik Kersten CLA 2006-03-17 12:06:32 EST
I sometimes wonder if it really should close, but it's a bit of work to reset the editor input.  Might be less just to reopen after submit, but that will be too wierd.  We should reconsider this behavior after 0.5.
Comment 6 Eugene Kuleshov CLA 2006-03-17 12:12:20 EST
I think that closing it is really weird. So, as action buttons inside editor. There is already save on toolbar, so you may as well put compare there.
Comment 7 Mik Kersten CLA 2006-03-17 12:16:15 EST
Compare could even move to a task list popup menu, and yes, that is a wierd spot for it so we should resolve that as part of this report.

Save is tricker--if you hit save it means that you're saving it locally, and that's important.  "Submit" is analogous to "Commit" for resources...
Comment 8 Eugene Kuleshov CLA 2006-03-17 12:25:04 EST
(In reply to comment #7)
> Save is tricker--if you hit save it means that you're saving it locally, and
> that's important.  "Submit" is analogous to "Commit" for resources...

Hmm. It use to be an offline preference that would of cover that if you don't want to publish it to server on save.

BTW, there is still weird "mid air change" conflict that is not handled nicely...
Comment 9 Mik Kersten CLA 2006-03-17 12:58:52 EST
All of the submission errors (e.g. mid-air collisions) will most likely continue to be handled via the HTML Bugzilla response error dialog until we get the web service UI.  It would be too hard and brittle to parse every error bugzilla can return, so we only parse for successful submission.
Comment 10 Eugene Kuleshov CLA 2006-03-17 13:04:52 EST
(In reply to comment #9)
> All of the submission errors (e.g. mid-air collisions) will most likely
> continue to be handled via the HTML Bugzilla response error dialog until we get
> the web service UI.  It would be too hard and brittle to parse every error
> bugzilla can return, so we only parse for successful submission.

Can you at least make that dialog resizeable?

Also it is unclear what to do. If it is just a new comment then error can be simply ignored or what?
Comment 11 Stefan Langer CLA 2006-10-26 03:13:44 EDT
Is there any plan on resolving this? I find it very annyoing that for each change done in the editor and submitted to bugzilla the editor closes. 
Comment 12 Ismael Juma CLA 2006-10-26 04:13:46 EDT
I would have to say the same. I often have to perform two actions on a bug (Mark as RESOLVED and then close the bug) and having the editor close after the first one is not the ideal scenario. At least ctrl-shift-F11 makes it easy to reopen the bug. ;)
Comment 13 Robert Elves CLA 2006-10-26 12:41:54 EDT
I don't see this getting attention until 1.0. Short term, what if we force the editor to reopen as suggested in comment#5?
Comment 14 Ismael Juma CLA 2006-10-26 12:52:26 EDT
Personally I would see it as an improvement.
Comment 15 Mik Kersten CLA 2006-10-26 17:51:16 EDT
Me too, although I must admit I've become so accustomed to this I hardly notice.

Rob: to be sure, all that this takes is the ability to reset the input on an editor, right?  Hopefull that's straightforward now that you've improved the editor architecture?  If so please consider this for 0.9.
Comment 16 Robert Elves CLA 2006-10-26 18:08:01 EDT
I'm not sure of the extent of the necessary changes for this one. If editor can't be updated easily I'll go with the re-open option for 0.9 then complete for 1.0.
Comment 17 Robert Elves CLA 2006-11-14 14:34:46 EST
*** Bug 164220 has been marked as a duplicate of this bug. ***
Comment 18 Willian Mitsuda CLA 2006-11-14 15:54:03 EST
+1 to simply reopen the editor and make 90% of people happy :)

More complex and elegant solutions can be analyzed for post 1.0.
Comment 19 Robert Elves CLA 2006-11-14 16:16:17 EST
I agree Willian.
Comment 20 Eugene Kuleshov CLA 2006-11-14 16:40:48 EST
BTW, can editor close (and then reopen) after submit and synchronization completed? So, there won't be nasty delay between close and reopen...
Comment 21 Robert Elves CLA 2006-11-14 16:46:41 EST
I'll look into this now...
Comment 22 Robert Elves CLA 2006-11-14 18:03:43 EST
Update: Bugzilla editor closes/reopens after submit synchronization.
Comment 23 Robert Elves CLA 2006-11-14 18:03:46 EST
Created attachment 53867 [details]
mylar/context/zip
Comment 24 Robert Elves CLA 2006-11-20 15:00:55 EST
Any further improvement will happen post 1.0.
Comment 25 Mik Kersten CLA 2006-11-20 18:52:04 EST
*** Bug 164985 has been marked as a duplicate of this bug. ***
Comment 26 Robert Elves CLA 2007-01-23 16:14:16 EST
I think we've discussed adding a check box indicating if the editor should be reopened but my current thinking is that this should just be another button something like:

[ Submit ]   [ Submit and reopen]

Just to limit the number of clicks involved in submitting a report.
Comment 27 Eugene Kuleshov CLA 2007-01-23 16:20:57 EST
(In reply to comment #26)
> I think we've discussed adding a check box indicating if the editor should be
> reopened but my current thinking is that this should just be another button
> something like:
> 
> [ Submit ]   [ Submit and reopen]
> 
> Just to limit the number of clicks involved in submitting a report.

I thought it is planned for 2.0M1 to refresh editors without closing them...
Comment 28 Robert Elves CLA 2007-01-23 16:36:14 EST
Would be nice to have that done... not scheduled yet though that I'm aware of.
Comment 29 Mik Kersten CLA 2007-01-23 17:02:06 EST
Rob: forget my suggestion of the check box.  For 2.0 we really should stop this editor closing, and just reload the editor input.  If that ends up being a network operation,the editor can go into a "busy" state like the compare editor currently does.
Comment 30 Robert Elves CLA 2007-01-23 18:04:38 EST
Sounds good I'll try to get this in to 2.0.
Comment 31 Robert Elves CLA 2007-01-27 20:14:15 EST
Committed first pass at this. Give it a try and see what you think.
Comment 32 Eugene Kuleshov CLA 2007-01-27 20:59:03 EST
Test
Comment 33 Eugene Kuleshov CLA 2007-01-29 13:02:54 EST
It seems like there is much less jumping now, which is really great.

Few comments:

-- Need to restore focus and scroller states and state of the foldable sections after refreshing form

-- It would be neat to use new fancy progress indicator like in Eclipse 3.3M4.

-- Opening tasks probably should open stub editor and show progress indicator while fetching/synchronizing task data from the repository

-- Reopen report on changes still closing editor
Comment 34 Robert Elves CLA 2007-01-29 13:37:05 EST
 (In reply to comment #33)
> It seems like there is much less jumping now, which is really great.
> Few comments:
> -- Need to restore focus and scroller states and state of the foldable sections
> after refreshing form

Agreed. Created bug#172033.

> -- It would be neat to use new fancy progress indicator like in Eclipse 3.3M4.
> -- Opening tasks probably should open stub editor and show progress indicator
> while fetching/synchronizing task data from the repository

Just renamed bug#154396 to make these two explicit and scheduled for 2.0

> -- Reopen report on changes still closing editor

Sorry, are you talking about new bug editor here? 
Comment 35 Eugene Kuleshov CLA 2007-01-29 13:44:06 EST
(In reply to comment #34)
> > -- Reopen report on changes still closing editor
> Sorry, are you talking about new bug editor here? 

No, when we have opened editor and task synchronization discover new changes in that task popup dialog brought up and then editor is reopened. Unless I am missing something.
Comment 36 Robert Elves CLA 2007-01-29 14:14:51 EST
Oh good catch. I'll look into that as part of this task.
Comment 37 Robert Elves CLA 2007-01-29 14:43:23 EST
It's a little more involved than I had expected. I've create a new report handling stale editor correctly: bug#172042.
Comment 38 Willian Mitsuda CLA 2007-02-20 20:56:02 EST
Although it is working well on 2.0 M1, it does not work for remote tasks, i.e. which are not on task list.