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

Modernized help UI prototype: Refactor JavaScript code #738

Merged
merged 1 commit into from
May 8, 2024

Conversation

howlger
Copy link
Contributor

@howlger howlger commented Oct 10, 2023

Refactor JavaScript code for better readability. No functional changes.

See also #735 (review)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

Test Results

 1 731 files  ±0   1 731 suites  ±0   1h 23m 39s ⏱️ - 1m 11s
 4 005 tests ±0   3 983 ✅ ±0   22 💤 ±0  0 ❌ ±0 
12 612 runs  ±0  12 451 ✅ ±0  161 💤 ±0  0 ❌ ±0 

Results for commit f17c6bc. ± Comparison against base commit a1466b8.

♻️ This comment has been updated with latest results.

@howlger
Copy link
Contributor Author

howlger commented Oct 11, 2023

The test failures do not seem to me to be related to this change.

@howlger howlger force-pushed the patch-before-1 branch 2 times, most recently from 54713fd to b2647dc Compare October 13, 2023 09:06
@howlger
Copy link
Contributor Author

howlger commented Oct 26, 2023

To review this change, the modernized help UI has to be enabled via the Java property -Dorg.eclipse.help.webapp.experimental.ui=true

Refactor JavaScript code for better readability. No functional changes.

See also eclipse-platform#735 (review)
@iloveeclipse
Copy link
Member

@howlger : is this still relevant?

@howlger
Copy link
Contributor Author

howlger commented May 8, 2024

@howlger : is this still relevant?

Yes, as it blocks #735. We at Advantest are using this change for quite some time now without any problems.

@iloveeclipse
Copy link
Member

We at Advantest are using this change for quite some time now without any problems.

:-)

If so, let's merge

@iloveeclipse iloveeclipse merged commit 6a0853d into eclipse-platform:master May 8, 2024
16 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