[PR #3874] [MERGED] fix: context menu bug and incorrect position #4561

Closed
opened 2026-03-17 02:05:18 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3874
Author: @nivedin
Created: 3/1/2024
Status: Merged
Merged: 3/7/2024
Merged by: @AndrewBastin

Base: release/2024.3.0Head: fix/response-context-menu-bug


📝 Commits (4)

  • 55ec90e fix: use debounce to fix performance and scroll bug
  • 4b8a4fd chore: fix type
  • 44fba56 fix: context menu position bug due to pane z posiiton change
  • 9dd6373 chore: add debounce in env input scroll text selection

📊 Changes

3 files changed (+27 additions, -19 deletions)

View changed files

📝 packages/hoppscotch-common/src/components/app/ContextMenu.vue (+2 -2)
📝 packages/hoppscotch-common/src/components/smart/EnvInput.vue (+12 -8)
📝 packages/hoppscotch-common/src/composables/codemirror.ts (+13 -9)

📄 Description

Closes HFE-403

Description

This PR fixes the issue where sometimes when scrolling the response section while the context menu is opened will break the syntax highlighting in codemirror.
This PR also fixes a slight change in context menu position due the the z position of the splitpanes changed.

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/3874 **Author:** [@nivedin](https://github.com/nivedin) **Created:** 3/1/2024 **Status:** ✅ Merged **Merged:** 3/7/2024 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2024.3.0` ← **Head:** `fix/response-context-menu-bug` --- ### 📝 Commits (4) - [`55ec90e`](https://github.com/hoppscotch/hoppscotch/commit/55ec90e36066057e006f57a363457a881179e00b) fix: use debounce to fix performance and scroll bug - [`4b8a4fd`](https://github.com/hoppscotch/hoppscotch/commit/4b8a4fddcd74cc63b7e9c1311c18734c052c477c) chore: fix type - [`44fba56`](https://github.com/hoppscotch/hoppscotch/commit/44fba567df9035a08506d17b4c88b1673cb88d6b) fix: context menu position bug due to pane z posiiton change - [`9dd6373`](https://github.com/hoppscotch/hoppscotch/commit/9dd6373cfb3e029da43734d4f691defe890f9f27) chore: add debounce in env input scroll text selection ### 📊 Changes **3 files changed** (+27 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-common/src/components/app/ContextMenu.vue` (+2 -2) 📝 `packages/hoppscotch-common/src/components/smart/EnvInput.vue` (+12 -8) 📝 `packages/hoppscotch-common/src/composables/codemirror.ts` (+13 -9) </details> ### 📄 Description Closes HFE-403 ### Description This PR fixes the issue where sometimes when scrolling the response section while the context menu is opened will break the syntax highlighting in codemirror. This PR also fixes a slight change in context menu position due the the z position of the splitpanes changed. ### Checks - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [x] All the tests have passed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 02:05:18 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hoppscotch#4561
No description provided.