I have set the Help Id of a Property Sheet, ie:
Sheet.Set s = new Sheet.Set ();
s.setValue ("helpID", "my_help_tag");
Using FFJ 3.0 (based on NB3.2), if I displayed
this tab on the property sheet, hit F1, the help
was displayed in the help viewer. Using new
builds of FFJ based on NB3.3, this no longer works.
I found out you are right and ment that always the default help page
was showed, instead of the one you specified by help ID. The regressio
was caused by refactoring of prop sheets about 2 months ago.
Fixed in [main-trunk].
verified in [nb_dev](20011112)
Now if you have selected some property which has setvalue, after
hitting F1 Help window with right help arise.
But there is another issue with additional problem issue 17424.
Sorry, it is not working for me. Maybe it was fixed, but it is not working now, in the
trunk at least (early March sources). Lower priority since no one really used
property-level help so I do not think it is critical even though it is a regression,
but it would be nice to fix for 3.4.
See unit tests in issue #14701. (Not committed to CVS yet but you can run them, see
attachments there.) Also see interactive test class in #20794 which makes the problem
easy to see.
Node.PropertySet help works fine. But Node.Property help does not work at all. Setting
an ID on the N.P has no effect; the property sheet continues to look for an ID on the
active N.PS and then on the containing Node. Examining the
org.openide.explorer.propertysheet.* code, I can see nothing which even looks for an ID
on a N.P (the code which finds it on a N.PS looks straightforward).
Please reexamine this. If fixed properly, the config-file exclude in #14701 should not
be necessary - all the help tests ought to pass. The unit tests should make it easier
for propsheet developers to make sure it does not get rebroken.
Jirka I found three problems with your patch to cfg-unit.xml:
1. The sequence
is redundant, the second line does nothing.
2. FindHelpTest is supposed to be run with the code executor, not the
ide executor. Please change
to indicate that *both* tests in this class should be run.
3. You did not remove this test from the failing config:
<!-- Implementation was apparently killed during a
Are you sure this problem is now fixed (you verified it manually using
the supplied demo class from issue #20794, as well as running the unit
test)? I did not see any code changes which were supposed to fix the
bug, and a search of org.openide.explorer.propertysheet.* sources
still reveals only one use of "helpID" in PropertySheetTab, getting
help on the tab but not an individual property.
Jesse, you are right, I overlooked there were two FindHelpTest in
explorer and else in explorer/propertysheet directory at first and I
included the explorer/propertysheet/FindHelpTest twice at second and
FindHelpTest/testFindHelpOnPropertySheetRow() didn't work at last. I'm
going to set the unit tests back for now. Sorry for annoyance.