Release a new version of the debugger #3098
Conversation
jasonLaster
commented
Jun 6, 2017
- Updates from the past week
- Small fixes for the editor footer and a couple small details for tests
Codecov Report
@@ Coverage Diff @@
## master #3098 +/- ##
=======================================
Coverage 66.28% 66.28%
=======================================
Files 76 76
Lines 2714 2714
Branches 549 549
=======================================
Hits 1799 1799
Misses 915 915
Continue to review full report at Codecov.
|
@@ -1,6 +1,9 @@ | |||
.editor-wrapper { | |||
--debug-line-background: rgba(226, 236, 247, 0.5); | |||
--debug-line-border: rgb(145, 188, 219); | |||
--editor-searchbar-height: 27px; | |||
--editor-second-searchbar-height: 27px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i know this is copy pasted, but do we need two separate declarations of the same value? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably not. that can be tweaked when we have a function search modal :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good!
src/components/Editor/Tabs.js
Outdated
@@ -50,7 +50,8 @@ function getHiddenTabs(sourceTabs: SourcesList, sourceTabEls) { | |||
return sourceTabs.filter((tab, index) => { | |||
// adding 10px helps account for cases where the tab might be offset by | |||
// styling such as selected tabs which don't have a border. | |||
return sourceTabEls[index].getBoundingClientRect().top > tabTopOffset + 10; | |||
const el = sourceTabEls[index]; | |||
return el && el.getBoundingClientRect().top > tabTopOffset + 10; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! i would split this up more even, so that we know what is happening.
const element = sourceTabEls[index];
const hasTopOffset = (el) => (el.getBoundingClientRect().top > tabTopOffset + 10)
return element && hasTopOffset(element);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's a good idea. I took it a couple step further lol.