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

Implement Highlight and Hover Support for Environment Variables in JSON Request Body #643

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

BalaSubramaniam12007
Copy link

@BalaSubramaniam12007 BalaSubramaniam12007 commented Mar 8, 2025

PR Description

  1. This PR adds env variable highlighting and hover values to the JSON request body editor .It reuses EnvRegExpSpanBuilder, EnvVarSpan, and EnvVarPopover, updates TextFieldEditor to ExtendedTextField with specialTextSpanBuilder, sets it in EditRequestBody,

  2. And fixes the import path. Improves API editing without changing style or behavior.

  3. Support both light and dark mode

image

Related Issues

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • I am using the latest Flutter stable branch (run flutter upgrade and verify)
  • I have run the tests (flutter test) and all tests are passing

Added/updated tests?

We encourage you to add relevant test cases.

  • Yes
  • No, and this is why: Existing tests already cover the reused components (EnvRegExpSpanBuilder, EnvVarSpan), and this change focuses on integrating them into the JSON editor.So additional testing is not required.

OS on which you have developed and tested the feature?

  • Windows
  • macOS
  • Linux

…Editor

- Enabled environment variable highlighting in  via .
- Switched  to  for special text span support
- Updated  to use  for JSON editor.
- Maintained UI/UX consistency without new themes or colors.
- Ensured regex accuracy and hover popovers work correctly.

Signed-off-by: Balasubramaniam12007 <[email protected]>
@BalaSubramaniam12007 BalaSubramaniam12007 marked this pull request as ready for review March 8, 2025 19:32
@BalaSubramaniam12007
Copy link
Author

@ashitaprasad Could you please review this PR when you get a chance? Your feedback would be really valuable in improving the implementation. Thanks in advance! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Env. Variable Support for JSON request body
2 participants