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

Feat/highlight path params #2415

Merged
merged 8 commits into from
Jun 21, 2024

Conversation

sanjai0py
Copy link
Member

@sanjai0py sanjai0py commented Jun 5, 2024

Description

Screen.Recording.2024-06-05.at.5.33.19.PM.mov

Contribution Checklist:

  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you want to make multiple changes, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

sanjai0py and others added 6 commits June 5, 2024 12:34
… This will highlight the pathparams in the QueryUrl.
…ombinedCodeMirrorMode for highlighting Bruno variables instead of defineCodeMirrorBrunoVariablesMode
…binedVariablesMode. Now the pathparams at the end of the URL is also highlighted.
@sanjai0py sanjai0py marked this pull request as ready for review June 7, 2024 14:59
…sMode instead of defineCodeMirrorCombinedVariablesMode. Now the path params in the URL will be highlighted on application open.
@lohxt1 lohxt1 added the pr-qol label Jun 19, 2024
@helloanoop helloanoop merged commit 01e93b5 into usebruno:main Jun 21, 2024
2 checks passed
@helloanoop
Copy link
Contributor

Great work @sanjai0py !

@lohxt1 As discussed, please raise an extension PR for the comments we discussed during the call.

jwetzell pushed a commit to jwetzell/bruno that referenced this pull request Aug 2, 2024
* fix: Update CodeMirror mode to use combinedmode for SingleLineEditor. This will highlight the pathparams in the QueryUrl.

* Refactor: Updated url highlighting.

* refactor: Improved the hinting part.

* refactor: CodeEditor, MultiLineEditor, and QueryEditor to use defineCombinedCodeMirrorMode for highlighting Bruno variables instead of defineCodeMirrorBrunoVariablesMode

* refactor: Updated  defineCombinedCodeMirrorMode to defineCodeMirrorCombinedVariablesMode. Now the pathparams at the end of the URL is also highlighted.

* refactor: Update CodeMirror mode to use defineCodeMirrorBrunoVariablesMode instead of defineCodeMirrorCombinedVariablesMode. Now the path params in the URL will be highlighted on application open.

---------

Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants