From a186df3ac4736b29565acd4b0629f78dfb39f6a9 Mon Sep 17 00:00:00 2001 From: Abhishek S Lal Date: Sat, 8 Nov 2025 01:54:50 +0530 Subject: [PATCH] refactor: update UI interactions and improve test stability (#6042) --- .../src/components/SingleLineEditor/index.js | 6 +- .../src/components/TagList/StyledWrapper.js | 7 +- .../bruno-app/src/components/TagList/index.js | 6 +- .../create-requests/graphql-requests.spec.ts | 10 +- .../create-requests/grpc-requests.spec.ts | 10 +- .../create-requests/http-requests.spec.ts | 10 +- .../create-requests/ws-requests.spec.ts | 10 +- .../cross-collection-drag-drop-folder.spec.ts | 72 +++------- ...cross-collection-drag-drop-request.spec.ts | 52 +------ .../moving-requests/tag-persistence.spec.ts | 130 ++++++++++-------- .../multiple-persist-vars.spec.ts | 10 +- .../global-env-export.spec.ts | 6 +- .../collection-env-import.spec.ts | 12 -- tests/onboarding/sample-collection.spec.ts | 8 +- .../default-collection-location.spec.js | 4 +- tests/protobuf/manage-protofile.spec.ts | 4 + .../request/copy-request/copy-request.spec.ts | 9 +- .../response-examples/create-example.spec.ts | 4 +- tests/response-examples/edit-example.spec.ts | 2 +- .../response-examples/menu-operations.spec.ts | 3 +- tests/utils/page/locators.ts | 7 +- 21 files changed, 171 insertions(+), 211 deletions(-) diff --git a/packages/bruno-app/src/components/SingleLineEditor/index.js b/packages/bruno-app/src/components/SingleLineEditor/index.js index 0bbb0bbde..87c47f3b0 100644 --- a/packages/bruno-app/src/components/SingleLineEditor/index.js +++ b/packages/bruno-app/src/components/SingleLineEditor/index.js @@ -205,7 +205,11 @@ class SingleLineEditor extends Component { render() { return (
- + {this.secretEye(this.props.isSecret)}
); diff --git a/packages/bruno-app/src/components/TagList/StyledWrapper.js b/packages/bruno-app/src/components/TagList/StyledWrapper.js index c81897a27..97fbdf920 100644 --- a/packages/bruno-app/src/components/TagList/StyledWrapper.js +++ b/packages/bruno-app/src/components/TagList/StyledWrapper.js @@ -23,13 +23,18 @@ const StyledWrapper = styled.div` max-width: 200px; transition: all 0.2s ease; box-shadow: 0 1px 2px rgba(0, 0, 0, 0.05); + cursor: default; - &:hover { + &:has(.tag-remove:hover) { background-color: ${(props) => props.theme.requestTabs.active.bg}; border-color: ${(props) => props.theme.requestTabs.active.border || props.theme.requestTabs.bottomBorder}; box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1); transform: translateY(-1px); } + + .tag-remove { + cursor: pointer; + } } .tag-icon { diff --git a/packages/bruno-app/src/components/TagList/index.js b/packages/bruno-app/src/components/TagList/index.js index e683a0a41..1b800a6a0 100644 --- a/packages/bruno-app/src/components/TagList/index.js +++ b/packages/bruno-app/src/components/TagList/index.js @@ -49,6 +49,7 @@ const TagList = ({ tagsHintList = [], handleAddTag, tags, handleRemoveTag, onSav onChange={handleInputChange} onRun={handleKeyDown} onSave={onSave} + data-testid="tag-input" /> {error && {error}}