Bug 299697 - [editor] implement preference to enable current line highlight in task editor
Summary: [editor] implement preference to enable current line highlight in task editor
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: 3.3.2   Edit
Assignee: David Green CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks: 257001
  Show dependency tree
 
Reported: 2010-01-14 15:26 EST by David Green CLA
Modified: 2010-02-22 22:56 EST (History)
2 users (show)

See Also:


Attachments
indication of where the new preference should appear. (73.91 KB, image/png)
2010-01-14 15:29 EST, David Green CLA
no flags Details
patch that implements the enhancement (13.22 KB, patch)
2010-01-14 21:00 EST, David Green CLA
no flags Details | Diff
mylyn/context/zip (74.07 KB, application/octet-stream)
2010-01-14 21:00 EST, David Green CLA
no flags Details
backported patch (2.10 KB, patch)
2010-01-26 10:50 EST, Steffen Pingel CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Green CLA 2010-01-14 15:26:31 EST
A preference is to be added to enable current line highlight, default value false.  The preference should appear under *Tasks -> Task Editing* in the Eclipse preferences.
WikiText task editor extension to be modified to use this preference for enabling current line highlight.

After this enhancement current line highlight will be disabled by default in the task editor when WikiText is used.
 
Enhancement based on conference call discussion.
Comment 1 David Green CLA 2010-01-14 15:29:11 EST
Created attachment 156155 [details]
indication of where the new preference should appear.
Comment 2 David Green CLA 2010-01-14 21:00:02 EST
Created attachment 156191 [details]
patch that implements the enhancement

Attached a working patch.  The only thing I don't like about this is that it introduces the use of tasks.ui internals into wikitext.tasks.ui.
Comment 3 David Green CLA 2010-01-14 21:00:14 EST
Created attachment 156192 [details]
mylyn/context/zip
Comment 4 David Green CLA 2010-01-14 21:04:07 EST
Fixed in CVS
Comment 5 Steffen Pingel CLA 2010-01-19 11:00:43 EST
David, would you be okay with back-porting the WikiText change to 3.3.2? Otherwise I would have to revert the changes to tasks ui for the maintenance release. 

For 3.4 we could also consider making this API, e.g. adding a method to TasksUiUtil that returns whether highlighting is enabled.
Comment 6 David Green CLA 2010-01-19 11:26:35 EST
(In reply to comment #5)
> David, would you be okay with back-porting the WikiText change to 3.3.2?
> Otherwise I would have to revert the changes to tasks ui for the maintenance
> release.

The ugly head of CVS rises once again!  Can't the maintenance release be built off a tag?

BTW, I don't mind backporting the change.  What tag should I be using on org.eclipse.mylyn.wikitext.tasks.ui ?


> For 3.4 we could also consider making this API, e.g. adding a method to
> TasksUiUtil that returns whether highlighting is enabled.

Great
Comment 7 Steffen Pingel CLA 2010-01-19 11:40:33 EST
(In reply to comment #6)
> (In reply to comment #5)
> > David, would you be okay with back-porting the WikiText change to 3.3.2?
> > Otherwise I would have to revert the changes to tasks ui for the maintenance
> > release.
> 
> The ugly head of CVS rises once again!  Can't the maintenance release be built
> off a tag?

Yes, I am planning to tag the release and to then build from that tag. It's not so much the fault of CVS that we don't branch but rather laziness of the release engineer who wants to keep the number of parallel branches that need to be maintained to a minimum...

> BTW, I don't mind backporting the change.  What tag should I be using on
> org.eclipse.mylyn.wikitext.tasks.ui ?

I'll have to setup a workspace and create a branch anyways so I can take care of that. I'll post the patch here so you can review.

> > For 3.4 we could also consider making this API, e.g. adding a method to
> > TasksUiUtil that returns whether highlighting is enabled.
> 
> Great

I have opened bug 300065 to track that.
Comment 8 David Green CLA 2010-01-19 12:08:32 EST
(In reply to comment #7)
> I'll have to setup a workspace and create a branch anyways so I can take care of
> that. I'll post the patch here so you can review.

Great, thanks for taking care of that.
Comment 9 Steffen Pingel CLA 2010-01-26 10:50:47 EST
Created attachment 157273 [details]
backported patch
Comment 10 Steffen Pingel CLA 2010-01-26 10:53:18 EST
I have committed the attached patch to the e_3_6_m_3_3_x branch and did a quick smoke test.