Skip to content

Commit

Permalink
Fix query editor cursor alignment by removing monaco css overrides (o…
Browse files Browse the repository at this point in the history
…pensearch-project#8912)

* Fix query editor cursor alignment by removing monaco css overrides

Monaco editor computes the cursor position internally and does not account for any css overrides
that change the position of the line (padding, margin, etc). This change removes the css override
while we implement these styles with the monaco api directly.

Signed-off-by: Daniel Rowe <[email protected]>

* Changeset file for PR opensearch-project#8912 created/updated

* fix linting

Signed-off-by: Daniel Rowe <[email protected]>

---------

Signed-off-by: Daniel Rowe <[email protected]>
Signed-off-by: Daniel Rowe <[email protected]>
Co-authored-by: Daniel Rowe <[email protected]>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
  • Loading branch information
3 people authored Nov 22, 2024
1 parent 89265b6 commit f110637
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/8912.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fix:
- Ensure query editor cursor is aligned with text ([#8912](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8912))
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,6 @@
padding: 0 $euiSizeXS;
}

.view-lines {
padding: 0 $euiSizeXS;
}

.monaco-scrollable-element {
border-radius: 0 $euiSizeXS 0 0;
}
Expand Down

0 comments on commit f110637

Please sign in to comment.