Bug 375099 - Automatically refresh the Builds view upon user initiated activation or bringing to foreground
Summary: Automatically refresh the Builds view upon user initiated activation or bring...
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 1.0   Edit
Assignee: Lucas Panjer CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, noteworthy
Depends on:
Blocks:
 
Reported: 2012-03-22 13:55 EDT by Lucas Panjer CLA
Modified: 2012-04-07 00:02 EDT (History)
0 users

See Also:

Attachments
patch (1.37 KB, patch)
2012-03-27 18:57 EDT, Lucas Panjer CLA 		no flags Details | Diff
patch (1.37 KB, patch)
2012-03-27 18:57 EDT, Lucas Panjer CLA 		no flags Details | Diff
mylyn/context/zip (3.12 KB, application/octet-stream)
2012-03-27 18:57 EDT, Lucas Panjer CLA 		no flags Details
updated to respect the refresh auto pref (1.49 KB, patch)
2012-03-28 12:48 EDT, Lucas Panjer CLA 		no flags Details | Diff
mylyn/context/zip (72.21 KB, application/octet-stream)
2012-04-07 00:02 EDT, Steffen Pingel CLA 		no flags Details
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Panjer CLA 2012-03-22 13:55:57 EDT 
A common sequence for me is to:
# click on the Builds view tab, bringing the view to foreground (or otherwise open the view)
# click on refresh toolbar action

I'd like #2 to disappear and be automatic.

This would be an extra refresh in addition to the background timer refreshing.
Comment 1 Steffen Pingel CLA 2012-03-22 14:02:22 EDT 
+1 I always manually refresh when making the view visible.
Comment 2 Lucas Panjer CLA 2012-03-27 18:57:20 EDT 
Created attachment 213264 [details]
patch

minimal change to fire a background refresh operation each time the builds view gets focus
Comment 3 Lucas Panjer CLA 2012-03-27 18:57:28 EDT 
Created attachment 213265 [details]
patch

minimal change to fire a background refresh operation each time the builds view gets focus
Comment 4 Lucas Panjer CLA 2012-03-27 18:57:30 EDT 
Created attachment 213266 [details]
mylyn/context/zip
Comment 5 Lucas Panjer CLA 2012-03-27 23:29:34 EDT 
This should probably be tied to the "Refresh Automatically" preference.
Comment 6 Lucas Panjer CLA 2012-03-28 12:48:06 EDT 
Created attachment 213301 [details]
updated to respect the refresh auto pref
Comment 7 Steffen Pingel CLA 2012-04-07 00:02:51 EDT 
Thanks for the contribution, Lucas! I have applied the change to master. 

I ended up introducing another preference to allow enablement of the refresh on focus independently of the auto refresh setting. The new setting is enabled by default. We may end up simplifying this but I have a workspace where I would like to control one independently of the other.
Comment 8 Steffen Pingel CLA 2012-04-07 00:02:53 EDT 
Created attachment 213723 [details]
mylyn/context/zip