Skip to content

fix(graphql): find queryname by cursor - [INS-4376]#8054

Merged
CurryYangxx merged 1 commit intodevelopfrom
fix/graphql-query-name
Oct 11, 2024
Merged

fix(graphql): find queryname by cursor - [INS-4376]#8054
CurryYangxx merged 1 commit intodevelopfrom
fix/graphql-query-name

Conversation

@CurryYangxx
Copy link
Copy Markdown
Member

@CurryYangxx CurryYangxx commented Oct 9, 2024

close #5648 : Support switch graphql query by mouse cursor.

Changes

  • add cursorActivity and focus event listener for codemirror editor
  • find queryname by cursor position and update state, sync with operation dropdown
  • query highlight
Screen.Recording.2024-10-09.at.17.00.34.mov

We have this feature previously, but remove it in this pr (for logic simplifying
reason) #5272

@CurryYangxx CurryYangxx marked this pull request as draft October 9, 2024 08:53
@CurryYangxx CurryYangxx changed the title fix(graphql): find queryname by cursor fix(graphql): find queryname by cursor - [INS-4376] Oct 9, 2024
@CurryYangxx CurryYangxx marked this pull request as ready for review October 9, 2024 09:03
@CurryYangxx CurryYangxx requested review from a team and jackkav October 9, 2024 09:03
@CurryYangxx CurryYangxx force-pushed the fix/graphql-query-name branch 2 times, most recently from cbccde7 to 40b97f4 Compare October 10, 2024 07:57
@CurryYangxx CurryYangxx force-pushed the fix/graphql-query-name branch from 40b97f4 to dbe8e0d Compare October 11, 2024 02:30
@CurryYangxx CurryYangxx force-pushed the fix/graphql-query-name branch from dbe8e0d to 9df9997 Compare October 11, 2024 02:53
@CurryYangxx CurryYangxx enabled auto-merge (squash) October 11, 2024 02:53
@CurryYangxx CurryYangxx merged commit c09ee87 into develop Oct 11, 2024
@CurryYangxx CurryYangxx deleted the fix/graphql-query-name branch October 11, 2024 03:29
@sentry
Copy link
Copy Markdown

sentry bot commented Oct 16, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: Expected fetch controller: :r64: app:///index.html View Issue

Did you find this useful? React with a 👍 or 👎

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants