Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monaco editor version update #4010

Merged
merged 15 commits into from
Aug 31, 2023
Merged

Monaco editor version update #4010

merged 15 commits into from
Aug 31, 2023

Conversation

Aniket-Engg
Copy link
Collaborator

No description provided.

@netlify
Copy link

netlify bot commented Aug 18, 2023

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 4502cc3
🔍 Latest deploy log https://app.netlify.com/sites/remixproject/deploys/64f03c949a224e0009c32401
😎 Deploy Preview https://deploy-preview-4010--remixproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LianaHus
Copy link
Collaborator

LianaHus commented Aug 23, 2023

  • there is a border on selected line in editor
  • greater/less symbols have red color
    Frame 7 (2)

@Aniket-Engg
Copy link
Collaborator Author

  • there is a border on selected line in editor
  • greater/less symbols have red color
    Frame 7 (2)

These are default changes done in the newer version

@Aniket-Engg Aniket-Engg added ready-to-review PR ready to review and removed WIP labels Aug 23, 2023
@Aniket-Engg Aniket-Engg added the editor Related to code editor label Aug 23, 2023
@LianaHus
Copy link
Collaborator

LianaHus commented Aug 24, 2023

These are default changes done in the newer version

ok but this should be fixed. we can discuss <> but selected line looks ugly

it should be one of those two:
.focusOutline
.focusAndSelectionOutline
set it to background color, it will fix it.
https://github.com/ethereum/remix-project/blob/monacoverUpdate/libs/remix-ui/editor/src/lib/remix-ui-editor.tsx#L317

Copy link
Collaborator

@LianaHus LianaHus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed in the chat, please restore the current(live) style of the selected line. If you still suggest changing it, we can discuss it on the UX call.

'editor.lineHighlightBorder': secondaryColor,
'editor.lineHighlightBackground':
textbackground === darkColor ? lightColor : secondaryColor,
'editor.lineHighlightBorder': textbackground,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not set it the same as the highlight color? Looks narrow now

@Aniket-Engg Aniket-Engg merged commit 52e5252 into master Aug 31, 2023
6 checks passed
@Aniket-Engg Aniket-Engg deleted the monacoverUpdate branch August 31, 2023 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor Related to code editor ready-to-review PR ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants