mattcasters opened a new pull request, #7937:
URL: https://github.com/apache/hop/pull/7937

   ## Summary
   
   - Extends `TextCompositeToolbarJsonFormatButton` so the existing 
pretty-print JSON action also appears on the shared Content Editor toolbar 
(`IContentEditorWidget.GUI_PLUGIN_TOOLBAR_PARENT_ID`).
   - Filters the button to editors with language `json` (e.g. explorer 
`JsonExplorerFileType`), matching the Markdown preview pattern.
   - Shares the Jackson pretty-print helper between TextComposite and Content 
Editor entry points.
   
   Fixes #7935
   
   ## Test plan
   
   - [x] Compile `plugins/transforms/json`
   - [x] Spotless apply on changed module
   - [x] Open `.json` in Explorer: Pretty-print JSON button present on content 
editor toolbar
   - [x] Non-JSON text files: button not shown
   - [x] Format valid JSON: reformats and marks dirty
   - [x] Invalid JSON: existing error dialog
   - [ ] Transform dialog JSON `TextComposite` still has the original format 
button


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to