Skip to content

feat: add stop request button in api url bar#5993

Merged
helloanoop merged 2 commits intousebruno:mainfrom
adarshajit:feature/stop-request-button-in-api-url-bar
Nov 13, 2025
Merged

feat: add stop request button in api url bar#5993
helloanoop merged 2 commits intousebruno:mainfrom
adarshajit:feature/stop-request-button-in-api-url-bar

Conversation

@adarshajit
Copy link
Contributor

Description

This PR introduces the following changes to fix issue (#5971):

  • Adds a Stop Request button in the URL bar

Refer to the attached recording below on how this new feature behaves.

Screen.Recording.2025-11-05.at.5.22.55.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.

@Pragadesh44-Bruno
Copy link
Collaborator

@adarshajit Thanks for the contribution! Will something like this work?
image

@adarshajit
Copy link
Contributor Author

Hi @Pragadesh44-Bruno, I've added a red color to the stop button. I went with this style since it will stay uniform with how the other buttons are presented in bruno say for example the save button which has a yellow outline to the icon.

Since we are using tabler icons thorughout the codebase, these are the only available styles: https://tabler.io/icons/icon/player-stop

Would love to know what you think. Here is a screenshot of how it looks like at the moment.

image

@Pragadesh44-Bruno
Copy link
Collaborator

@adarshajit I think keeping it in plain colors, like the other icons, looks nice since the red color doesn’t sit right in the dark theme
image
image

@adarshajit
Copy link
Contributor Author

Hi @Pragadesh44-Bruno, I agree. The code changes in this PR should be sufficient, as I haven’t pushed the changes related to the stop icon color. The new stop icon already uses the plain color. Please refer to the video recording once more and let me know if we’re good to proceed. Thanks.

@Pragadesh44-Bruno
Copy link
Collaborator

This PR looks good to me, and I have approved it. I’ll share it with the team for further review and merging

@helloanoop helloanoop merged commit fec37f4 into usebruno:main Nov 13, 2025
7 checks passed
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