Skip to content

Conversation

@danthe1st
Copy link
Contributor

This PR contains the help page update for eclipse-jdt/eclipse.jdt.ui#1825.

@danthe1st danthe1st force-pushed the custom-folding-regions branch from 2f4537b to cce50a6 Compare December 27, 2024 14:38
@mickaelistria
Copy link
Contributor

@danthe1st this is still marked as draft. Is there anything pending on your end preventing a merge?

@danthe1st
Copy link
Contributor Author

danthe1st commented Jan 6, 2025

this is still marked as draft. Is there anything pending on your end preventing a merge?

The PR introducing this feature is still open: eclipse-jdt/eclipse.jdt.ui#1825

Until the feature is actually merged, I don't think there's a point in merging the PR to the help.

@danthe1st danthe1st force-pushed the custom-folding-regions branch from cce50a6 to de93443 Compare January 31, 2025 21:12
@akurtakov
Copy link
Member

@danthe1st Now that it has been merged in jdt, please fix the merge conflict .
@jjohnstn Would you please review that the content matches what was merged in jdt?

@danthe1st danthe1st force-pushed the custom-folding-regions branch 3 times, most recently from 0b93fd9 to 3cbfbcb Compare March 7, 2025 15:28
@danthe1st
Copy link
Contributor Author

danthe1st commented Mar 7, 2025

For now, I updated this PR following the format introduced in #2826:
image

Alternatively, I could do it in a similar way to the content assist preferences with headings:
image

Another option is using indentation for the Region start/end text entries but I am unsure about using indentation for text inputs.

@jjohnstn
Copy link
Contributor

jjohnstn commented Mar 7, 2025

@akurtakov Will do

@danthe1st danthe1st marked this pull request as ready for review March 7, 2025 21:34
@jjohnstn
Copy link
Contributor

Hi @akurtakov The content being added is correct and matches the new options/behavior.

@akurtakov akurtakov force-pushed the custom-folding-regions branch from 3cbfbcb to 9cc3e48 Compare March 11, 2025 05:45
@danthe1st danthe1st force-pushed the custom-folding-regions branch from 9cc3e48 to af977ce Compare March 11, 2025 16:00
@danthe1st
Copy link
Contributor Author

^ bumped the version to make the build pass

@danthe1st danthe1st force-pushed the custom-folding-regions branch from af977ce to 16fc39e Compare March 11, 2025 17:55
@akurtakov akurtakov dismissed BeckerWdf’s stale review March 12, 2025 11:41

I believe your request have been fullfilled and you haven't acted on the PR for long period thus dismissing your review.

@akurtakov akurtakov merged commit 64ddb85 into eclipse-platform:master Mar 12, 2025
4 checks passed
@danthe1st danthe1st deleted the custom-folding-regions branch March 12, 2025 14:32
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.

5 participants