Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix buggy Javadoc styling enhancements menu +hard to read colored type parameters (#1442) #1482

Merged
merged 4 commits into from
Jul 9, 2024

Conversation

RedeemerSK
Copy link
Contributor

@RedeemerSK RedeemerSK commented Jun 29, 2024

What it does

Fixes buggy styling enhancements menu (see #1442 (comment)):

  • menu content not properly reflecting enable/disable state of styling enhancements
  • menu visually corrupted on Linux / GTK in certain situations (eg. after hover's shell is activated)

Also makes Javadoc to render (generic) type parameters with same font-weight (was lighter) like the rest of the method signature - see #1442 (comment)

How to test

Toggle / change all styling enhancements settings via styling enhancements menu on JavadocHover's and JavadocView's tollbar and confirm styling enhancements menu reflect current settings + javadoc content being displayed and available actions have expected results (eg. toggling / changing / resetting colors).
Scenarios of using styling enhancements menu to consider:

  • in JavadocHover that was activated = by pressing F2, clicking inside content (including on toolbar after entering into with mouse, or on any tollbar item button (but not button's sub-menu arrow))
  • in JavadocHover that was not activated = just hovered into by mouse, then opening enhancements menu by clicking button's sub-menu arrow
  • in JavadocView that is displaying javadoc for some source code element
  • in JavadocView that is not displaying any javadoc = view was just opened and no source code element was focused on yet
  • current settings presented in enhancements menus in JavadocHover and JavadocView should be in sync

Author checklist

- incorrect text was searched for inside HTML content to see if
enhancements are present
- menu actions for enhancements enabled state were not set / left only
when enhancements were present in content

Fixes eclipse-jdt#1442
Re-create them every time they're being requested.

Fixes eclipse-jdt#1442
@iloveeclipse
Copy link
Member

I've just rebased on master to run tests / checks on latest head before merge.
The patch looks good for me, many thanks!
Here screenshot:
image

@iloveeclipse iloveeclipse merged commit 4a918e9 into eclipse-jdt:master Jul 9, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants