Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

Refactor code editor by moving the expand snippet out to a separate file#2864

Merged
Rokt33r merged 10 commits into
BoostIO:masterfrom
ZeroX-DG:refactor-codeEditor
Mar 20, 2019
Merged

Refactor code editor by moving the expand snippet out to a separate file#2864
Rokt33r merged 10 commits into
BoostIO:masterfrom
ZeroX-DG:refactor-codeEditor

Conversation

@ZeroX-DG

@ZeroX-DG ZeroX-DG commented Feb 5, 2019

Copy link
Copy Markdown
Member

Description

This PR attempt to refactor the CodeEditor component by moving the CodeMirror language list and expand snippet feature out to a separated file

Refactor information

  • Component: CodeEditor
  • Affected feature:
    • Expand snippet
    • Auto detect programming language
@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. needs extra review 🔎 Pull request requires review from an additional reviewer. labels Feb 5, 2019
@ZeroX-DG ZeroX-DG requested a review from Rokt33r February 5, 2019 06:12
@ZeroX-DG ZeroX-DG changed the title Refactor code editor by moving the snippet out to a separate file Feb 5, 2019
@Rokt33r Rokt33r merged commit b5b56f7 into BoostIO:master Mar 20, 2019
@Rokt33r Rokt33r removed needs extra review 🔎 Pull request requires review from an additional reviewer. approved 👍 Pull request has been approved by sufficient reviewers. labels Mar 20, 2019
@Rokt33r Rokt33r added this to the v0.11.15 milestone Mar 20, 2019
@janat08

janat08 commented Mar 21, 2019

Copy link
Copy Markdown

The master is crashing?

this.snippets = []
this.expandSnippet = this.expandSnippet.bind(this)
this.init = this.init.bind(this)
this.assignSnippet = this.assignSnippet.bind(this)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks like this is a typo. It's causing startup to fail on my local machine. This likely should be:

this.assignSnippets = this.assignSnippets.bind(this)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. I'll fix it quickly.

@Rokt33r Rokt33r mentioned this pull request Mar 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

4 participants