RFR: JDK-8241292: Interactive Search results are not highlighted as they used to be

Jonathan Gibbons jonathan.gibbons at oracle.com
Fri Mar 20 16:20:47 UTC 2020


Updated webrev, with updated test.

The update to the test is intentionally simple, with repeated scans with 
different regex, instead of trying to write a single big pattern, which 
would then result in issues trying to get the right group match.

I note the test confirms that it has detected the new pattern, as seen 
in this fragment of the .jtr file.  i.e. note that result-item is now in 
the list.

> Starting subtest 1.36: Checking CSS classes found
> Passed: 5 found: [.result-highlight, .result-item, 
> .search-tag-desc-result, .search-tag-holder-result, 
> .ui-autocomplete-category]
-- Jon

Webrev: http://cr.openjdk.java.net/~jjg/8241292/webrev.01/index.html


On 3/20/20 8:04 AM, Jonathan Gibbons wrote:
> Thanks for the response; I'll check out "resultItem".
>
> -- Jon
>
> On 3/20/20 3:56 AM, Pavel Rappo wrote:
>> Hi Jon,
>>
>> The proposed patch does fix the issue in question. That said, I 
>> noticed one more
>> visual difference between the current L&F and that of before 
>> JDK-8240916:
>> the font size of search result items.
>>
>> Long story short, there's one more class name we forgot to change, 
>> "resultItem".
>> I'm not sure though how to update the proposed test to cover for that 
>> case too.
>>
>> Otherwise, looks good.
>>
>> -Pavel
>>
>>> On 19 Mar 2020, at 21:08, Jonathan Gibbons 
>>> <jonathan.gibbons at oracle.com> wrote:
>>>
>>> Please review a very simple fix for the reported regression. The fix 
>>> is just to update the search.js file with the hyphenated class names.
>>>
>>> Since there is no compile-time check between the contents of a 
>>> JavaScript file and the contents of a stylesheet file, I've updated 
>>> TestSearch.java to check that any CSS class names referenced in the 
>>> JavaScript file also appear in the stylesheet file.
>>>
>>> Note: there is another old-style stylesheet files with camelCase 
>>> names in the test directory, that could be updated at some point.
>>> open/test/langtools/jdk/javadoc/doclet/testOptions/custom-stylesheet.css 
>>>
>>> This file exists to help test options, and the content does not 
>>> appear to be important.  Updating it can be done separately.
>>>
>>> -- Jon
>>>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8241292
>>> Webrev: http://cr.openjdk.java.net/~jjg/8241292/webrev.00/index.html
>>>
>>>


More information about the javadoc-dev mailing list