Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 65660 - [Markers] markerHelp does not seem to be working - probable regression
Summary: [Markers] markerHelp does not seem to be working - probable regression
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P2 normal with 1 vote (vote)
Target Milestone: 3.0 RC4   Edit
Assignee: Stefan Xenos CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-03 21:50 EDT by Pat McCarthy CLA
Modified: 2004-06-24 12:18 EDT (History)
3 users (show)

See Also:


Attachments
Fixes marker help (2.40 KB, patch)
2004-06-22 15:30 EDT, Stefan Xenos CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pat McCarthy CLA 2004-06-03 21:50:55 EDT
markerHelp does not seem to be working - probable regression
I'm trying out my own markerHelp implementation and not getting anywhere, so I 
dropped back to try and see if the example in the readmetool was working; it 
is not.

I can take a 2.1.1 system and just add the examples, create a readme file, use 
the add> context menu and add the 1234 marker. Select the marker in tasks view 
and press F1 - I get a marker help hover.

Same process on RC1/0529 code fails.  I get the workbench popup help with 
hyperlinks for Workbench, Perspectives, and Workbench management.
Comment 1 Stefan Xenos CLA 2004-06-04 11:43:56 EDT
We should probably investigate this regression for 3.0
Comment 2 Pat McCarthy CLA 2004-06-17 23:34:04 EDT
Status request.
Comment 3 Michael Van Meekeren CLA 2004-06-21 16:39:12 EDT
could you attach an example that does/does not work in 2.1 and 3.0
Comment 4 Pat McCarthy CLA 2004-06-21 17:12:00 EDT
No need.  Use the org.eclipse.ui.examples.readmetool in the examples zip.

Works in 2.1, does not in 3.0.

Test path once installed (unzip on top of a base install)

-Create simple project.
-Use wizard to create readme file in project (in examples cat)
-Swipe text in readme file editor - use context menu Add > marker example 6
-Marker in tasks view (you may have to adjust filters to let the readme marker 
type be visible).
-Select marker in view.  Two tests (Quick fix is not working either)
1> Context menu - quick fix - should open dialog with Add Sentence proposals
2> F1 while marker with 1234, code=7 red... is selected should open a popup 
with this text content: Readme marker example 6 Help id = 1234, level = 7, 
department = infra, code = red, language = english 

Above paste from the 2.1.2 test I just ran.

I've recreated same myself - but the readme example is more convenient and 
fails just the same.
Comment 5 Michael Van Meekeren CLA 2004-06-22 14:41:44 EDT
stefan could you investigate whether this is this a regression?
Comment 6 Stefan Xenos CLA 2004-06-22 15:30:04 EDT
Created attachment 12685 [details]
Fixes marker help

This is indeed a regression. This patch fixes the F1 help for markers.

Note: I believe that the quickfix thing is an intentional behavior: you can fix
a problem, but not a task or bookmark. Quickfix still works for the problems
view. Please file another bug for the missing quickfix if you know of a good
use case for it.
Comment 7 Pat McCarthy CLA 2004-06-22 15:55:01 EDT
Quick fix is fine.  I was in 2.1 mode in my head looking in tasks view.

I'll see if I can test the patch.
Comment 8 Pat McCarthy CLA 2004-06-22 16:09:12 EDT
Patch file is bad - but got it to work. (Patch content in there twice?).

Tests fine for me on top of RC3 using readme example.
Comment 9 Stefan Xenos CLA 2004-06-22 16:13:17 EDT
Ironically, the repeated patch thing is a bugzilla bug. Whenever you attach a
patch to a bugzilla PR and select the "patch" button, it copies everything in
the file twice. :-)
Comment 10 Michael Van Meekeren CLA 2004-06-22 16:13:31 EDT
needs to be reviewed by Arch. Group.  marking as RC4
Comment 11 Tod Creasey CLA 2004-06-23 11:55:08 EDT
Verified patch. +1
Comment 12 Tod Creasey CLA 2004-06-23 12:55:58 EDT
Released and verified patch with SX. Released for build 20040623-1600
Comment 13 Debbie Wilson CLA 2004-06-24 10:54:31 EDT
*** Bug 68441 has been marked as a duplicate of this bug. ***
Comment 14 Debbie Wilson CLA 2004-06-24 12:18:11 EDT
Verified by Debbie and Stefan on I20040624 (0800).