Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Convert Editor component to JSX.#3554

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
wldcordeiro:jsx-editor
Aug 4, 2017
Merged

Convert Editor component to JSX.#3554
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
wldcordeiro:jsx-editor

Conversation

@wldcordeiro
Copy link
Copy Markdown
Contributor

Associated Issue: #3506

Summary of Changes

  • Convert the Editor component to JSX

Test Plan

Tell us a little a bit about how you tested your patch.

Example test plan:

  • Tested editor features (add breakpoint, context menu, etc)
  • Ran unit tests

Copy link
Copy Markdown
Contributor

@jasonLaster jasonLaster left a comment

Choose a reason for hiding this comment

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

Looks great!

import Preview from "./Preview";
import Breakpoints from "./Breakpoints";
import HitMarker from "./HitMarker";
import CallSites from "./CallSites";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

so nice!

// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`ast getOutOfScopeLocations with selected line 1`] = `
Object {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hmm - not sure if we need this! it's a flaky test.

Take it out and we should be able to land it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will do.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 4, 2017

Codecov Report

Merging #3554 into master will decrease coverage by 0.08%.
The diff coverage is 9.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3554      +/-   ##
==========================================
- Coverage   52.62%   52.54%   -0.09%     
==========================================
  Files         119      119              
  Lines        4650     4644       -6     
  Branches      957      957              
==========================================
- Hits         2447     2440       -7     
- Misses       2203     2204       +1
Impacted Files Coverage Δ
src/components/Editor/index.js 20.44% <9.09%> (-1.79%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b1bf36...5e2f74d. Read the comment docs.

@jasonLaster jasonLaster merged commit f60265b into firefox-devtools:master Aug 4, 2017
@wldcordeiro wldcordeiro deleted the jsx-editor branch August 4, 2017 17:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants