Chore: Migrating annotations code into repo

Sumedha Pramod · Sumedha Pramod · commit 3521db17410e · 2017-10-24T19:47:21.000-07:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "box-annotations",
-  "version": "0.0.0",
+  "version": "0.0.9",
   "description": "Box Annotations",
   "author": "Box (https://www.box.com/)",
   "license": "SEE LICENSE IN LICENSE",
diff --git a/src/AnnotationDialog.js b/src/AnnotationDialog.js
@@ -2,7 +2,7 @@ import autobind from 'autobind-decorator';
 import EventEmitter from 'events';
 import * as annotatorUtil from './annotatorUtil';
 import * as constants from './annotationConstants';
-import { ICON_CLOSE, ICON_DELETE } from '../icons/icons';
+import { ICON_CLOSE, ICON_DELETE } from './icons/icons';
 
 const POINT_ANNOTATION_ICON_HEIGHT = 31;
 const POINT_ANNOTATION_ICON_DOT_HEIGHT = 8;
diff --git a/src/AnnotationThread.js b/src/AnnotationThread.js
@@ -3,7 +3,7 @@ import autobind from 'autobind-decorator';
 import Annotation from './Annotation';
 import AnnotationService from './AnnotationService';
 import * as annotatorUtil from './annotatorUtil';
-import { ICON_PLACED_ANNOTATION } from '../icons/icons';
+import { ICON_PLACED_ANNOTATION } from './icons/icons';
 import {
     STATES,
     TYPES,
diff --git a/src/Annotator.js b/src/Annotator.js
@@ -2,7 +2,7 @@ import EventEmitter from 'events';
 import autobind from 'autobind-decorator';
 import AnnotationService from './AnnotationService';
 import * as annotatorUtil from './annotatorUtil';
-import { ICON_CLOSE } from '../icons/icons';
+import { ICON_CLOSE } from './icons/icons';
 import './Annotator.scss';
 import {
     CLASS_ACTIVE,
diff --git a/src/Annotator.scss b/src/Annotator.scss
@@ -1,4 +1,5 @@
-@import '../boxuiVariables';
+@import 'boxui';
+@import 'boxuiVariables';
 
 $highlight-yellow: #fed94e;
 $highlight-yellow-active-hover: #ffc900;
@@ -21,7 +22,8 @@ $tablet: 'max-width: 768px';
     background-color: $fours;
 }
 
-.bp-annotate-draw-header {
+.bp-annotate-draw-header,
+.bp-header.bp-annotate-draw-header {
     background-color: $twos;
     border: none;
     position: relative;
@@ -61,6 +63,8 @@ $tablet: 'max-width: 768px';
 .bp-annotate-draw-undo-redo-container {
     margin: 0 auto;
 
+    .is-disabled.bp-btn-annotate-draw-undo,
+    .is-disabled.bp-btn-annotate-draw-redo,
     .bp-btn-annotate-draw-undo,
     .bp-btn-annotate-draw-redo {
         background: none;
@@ -450,6 +454,8 @@ $tablet: 'max-width: 768px';
 
         &:hover {
             color: $better-black;
+            padding: 8px;
+            padding-left: 4px;
         }
     }
 
diff --git a/src/BoxAnnotations.js b/src/BoxAnnotations.js
@@ -130,5 +130,4 @@ class BoxAnnotations {
     }
 }
 
-global.BoxAnnotations = BoxAnnotations;
 export default BoxAnnotations;
diff --git a/src/MobileAnnotator.scss b/src/MobileAnnotator.scss
@@ -1,4 +1,4 @@
-@import '../_boxuiVariables';
+@import 'boxuiVariables';
 
 $tablet: "(min-width: 768px)";
 
diff --git a/src/__tests__/AnnotationDialog-test.js b/src/__tests__/AnnotationDialog-test.js
@@ -16,13 +16,13 @@ let dialog;
 const sandbox = sinon.sandbox.create();
 let stubs = {};
 
-describe('lib/annotations/AnnotationDialog', () => {
+describe('AnnotationDialog', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/__tests__/AnnotationDialog-test.html');
+        fixture.load('__tests__/AnnotationDialog-test.html');
 
         dialog = new AnnotationDialog({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/__tests__/AnnotationModeController-test.js b/src/__tests__/AnnotationModeController-test.js
@@ -6,7 +6,7 @@ let annotationModeController;
 let stubs;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/AnnotationModeController', () => {
+describe('AnnotationModeController', () => {
     beforeEach(() => {
         annotationModeController = new AnnotationModeController();
         stubs = {};
diff --git a/src/__tests__/AnnotationService-test.js b/src/__tests__/AnnotationService-test.js
@@ -1,6 +1,5 @@
 /* eslint-disable no-unused-expressions */
 import fetchMock from 'fetch-mock';
-import '../../polyfill';
 import Annotation from '../Annotation';
 import AnnotationService from '../AnnotationService';
 
@@ -9,7 +8,7 @@ const API_HOST = 'https://app.box.com/api';
 let annotationService;
 let sandbox;
 
-describe('lib/annotations/AnnotationService', () => {
+describe('AnnotationService', () => {
     beforeEach(() => {
         sandbox = sinon.sandbox.create();
         annotationService = new AnnotationService({
diff --git a/src/__tests__/AnnotationThread-test.js b/src/__tests__/AnnotationThread-test.js
@@ -16,13 +16,13 @@ let thread;
 const sandbox = sinon.sandbox.create();
 let stubs = {};
 
-describe('lib/annotations/AnnotationThread', () => {
+describe('AnnotationThread', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/__tests__/AnnotationThread-test.html');
+        fixture.load('__tests__/AnnotationThread-test.html');
 
         thread = new AnnotationThread({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/__tests__/Annotator-test.js b/src/__tests__/Annotator-test.js
@@ -23,13 +23,13 @@ let annotator;
 let stubs = {};
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/Annotator', () => {
+describe('Annotator', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/__tests__/Annotator-test.html');
+        fixture.load('__tests__/Annotator-test.html');
 
         const options = {
             annotator: {
diff --git a/src/__tests__/BoxAnnotations-test.js b/src/__tests__/BoxAnnotations-test.js
@@ -8,7 +8,7 @@ let loader;
 let stubs;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotators/BoxAnnotations', () => {
+describe('BoxAnnotations', () => {
     beforeEach(() => {
         stubs = {};
         loader = new BoxAnnotations();
diff --git a/src/__tests__/CommentBox-test.js b/src/__tests__/CommentBox-test.js
@@ -6,7 +6,7 @@ import {
     SELECTOR_ANNOTATION_BUTTON_POST
 } from '../annotationConstants';
 
-describe('lib/annotations/CommentBox', () => {
+describe('CommentBox', () => {
     const sandbox = sinon.sandbox.create();
     let commentBox;
     let parentEl;
diff --git a/src/__tests__/annotatorUtil-test.js b/src/__tests__/annotatorUtil-test.js
@@ -43,16 +43,16 @@ const DIALOG_WIDTH = 81;
 const sandbox = sinon.sandbox.create();
 let stubs = {};
 
-describe('lib/annotations/annotatorUtil', () => {
+describe('annotatorUtil', () => {
     let childEl;
     let parentEl;
 
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/__tests__/annotatorUtil-test.html');
+        fixture.load('__tests__/annotatorUtil-test.html');
 
         childEl = document.querySelector('.child');
         parentEl = document.querySelector('.parent');
diff --git a/src/doc/CreateHighlightDialog.js b/src/doc/CreateHighlightDialog.js
@@ -1,5 +1,5 @@
 import EventEmitter from 'events';
-import { ICON_HIGHLIGHT, ICON_HIGHLIGHT_COMMENT } from '../../icons/icons';
+import { ICON_HIGHLIGHT, ICON_HIGHLIGHT_COMMENT } from '../icons/icons';
 import CommentBox from '../CommentBox';
 import { hideElement, showElement, generateBtn } from '../annotatorUtil';
 import * as constants from '../annotationConstants';
diff --git a/src/doc/DocAnnotator.js b/src/doc/DocAnnotator.js
@@ -1,6 +1,6 @@
 import rangy from 'rangy';
 /* eslint-disable no-unused-vars */
-// Workaround for rangy npm issue: https://github.com/timdown/rangy/issues/342
+// Workaround for rangy npm issue: https://github.com/timdown/rangy/lib/issues/342
 import rangyClassApplier from 'rangy/lib/rangy-classapplier';
 import rangyHighlight from 'rangy/lib/rangy-highlighter';
 import rangySaveRestore from 'rangy/lib/rangy-selectionsaverestore';
diff --git a/src/doc/DocDrawingDialog.js b/src/doc/DocDrawingDialog.js
@@ -1,7 +1,7 @@
 import AnnotationDialog from '../AnnotationDialog';
 import * as annotatorUtil from '../annotatorUtil';
 import * as constants from '../annotationConstants';
-import { ICON_DRAW_SAVE, ICON_DRAW_DELETE } from '../../icons/icons';
+import { ICON_DRAW_SAVE, ICON_DRAW_DELETE } from '../icons/icons';
 
 class DocDrawingDialog extends AnnotationDialog {
     /** @property {boolean} Whether or not the dialog is visible */
diff --git a/src/doc/DocHighlightDialog.js b/src/doc/DocHighlightDialog.js
@@ -2,7 +2,7 @@ import autobind from 'autobind-decorator';
 import AnnotationDialog from '../AnnotationDialog';
 import * as annotatorUtil from '../annotatorUtil';
 import * as docAnnotatorUtil from './docAnnotatorUtil';
-import { ICON_HIGHLIGHT, ICON_HIGHLIGHT_COMMENT } from '../../icons/icons';
+import { ICON_HIGHLIGHT, ICON_HIGHLIGHT_COMMENT } from '../icons/icons';
 import * as constants from '../annotationConstants';
 
 const CLASS_HIGHLIGHT_DIALOG = 'bp-highlight-dialog';
diff --git a/src/doc/__tests__/CreateHighlightDialog-test.js b/src/doc/__tests__/CreateHighlightDialog-test.js
@@ -11,7 +11,7 @@ import * as annotatorUtil from '../../annotatorUtil';
 
 const CLASS_CREATE_DIALOG = 'bp-create-annotation-dialog';
 
-describe('lib/annotations/doc/CreateHighlightDialog', () => {
+describe('doc/CreateHighlightDialog', () => {
     const sandbox = sinon.sandbox.create();
     let dialog;
     let parentEl;
@@ -21,11 +21,11 @@ describe('lib/annotations/doc/CreateHighlightDialog', () => {
     };
 
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/CreateHighlightDialog-test.html');
+        fixture.load('doc/__tests__/CreateHighlightDialog-test.html');
 
         const parentEl = document.querySelector('.bp-create-highlight-dialog-container');
         dialog = new CreateHighlightDialog(parentEl, { localized });
diff --git a/src/doc/__tests__/DocAnnotator-test.js b/src/doc/__tests__/DocAnnotator-test.js
@@ -24,13 +24,13 @@ const sandbox = sinon.sandbox.create();
 
 const CLASS_DEFAULT_CURSOR = 'bp-use-default-cursor';
 
-describe('lib/annotations/doc/DocAnnotator', () => {
+describe('doc/DocAnnotator', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocAnnotator-test.html');
+        fixture.load('doc/__tests__/DocAnnotator-test.html');
 
         const options = {
             annotator: {
diff --git a/src/doc/__tests__/DocDrawingDialog-test.js b/src/doc/__tests__/DocDrawingDialog-test.js
@@ -6,13 +6,13 @@ let dialog;
 let stubs;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/doc/DocDrawingDialog', () => {
+describe('doc/DocDrawingDialog', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocDrawingDialog-test.html');
+        fixture.load('doc/__tests__/DocDrawingDialog-test.html');
         dialog = new DocDrawingDialog({
             annotatedElement: document.querySelector('.annotated-element'),
             location: {},
diff --git a/src/doc/__tests__/DocDrawingThread-test.js b/src/doc/__tests__/DocDrawingThread-test.js
@@ -13,13 +13,13 @@ let thread;
 let stubs;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/doc/DocDrawingThread', () => {
+describe('doc/DocDrawingThread', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocDrawingThread-test.html');
+        fixture.load('doc/__tests__/DocDrawingThread-test.html');
         thread = new DocDrawingThread({
             annotationService: {
                 user: {
diff --git a/src/doc/__tests__/DocHighlightDialog-test.js b/src/doc/__tests__/DocHighlightDialog-test.js
@@ -18,13 +18,13 @@ const DATA_TYPE_HIGHLIGHT_BTN = 'highlight-btn';
 const DATA_TYPE_ADD_HIGHLIGHT_COMMENT = 'add-highlight-comment-btn';
 const PAGE_PADDING_TOP = 15;
 
-describe('lib/annotations/doc/DocHighlightDialog', () => {
+describe('doc/DocHighlightDialog', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocHighlightDialog-test.html');
+        fixture.load('doc/__tests__/DocHighlightDialog-test.html');
 
         dialog = new DocHighlightDialog({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/doc/__tests__/DocHighlightThread-test.js b/src/doc/__tests__/DocHighlightThread-test.js
@@ -15,13 +15,13 @@ import {
 let thread;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/doc/DocHighlightThread', () => {
+describe('doc/DocHighlightThread', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocHighlightThread-test.html');
+        fixture.load('doc/__tests__/DocHighlightThread-test.html');
 
         thread = new DocHighlightThread({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/doc/__tests__/DocPointDialog-test.js b/src/doc/__tests__/DocPointDialog-test.js
@@ -6,13 +6,13 @@ import * as docAnnotatorUtil from '../docAnnotatorUtil';
 let dialog;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/doc/DocPointDialog', () => {
+describe('doc/DocPointDialog', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocPointDialog-test.html');
+        fixture.load('doc/__tests__/DocPointDialog-test.html');
 
         dialog = new DocPointDialog({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/doc/__tests__/DocPointThread-test.js b/src/doc/__tests__/DocPointThread-test.js
@@ -9,13 +9,13 @@ import { STATES } from '../../annotationConstants';
 let pointThread;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/doc/DocPointThread', () => {
+describe('doc/DocPointThread', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/DocPointThread-test.html');
+        fixture.load('doc/__tests__/DocPointThread-test.html');
 
         pointThread = new DocPointThread({
             annotatedElement: document.querySelector('.annotated-element'),
diff --git a/src/doc/__tests__/docAnnotatorUtil-test.js b/src/doc/__tests__/docAnnotatorUtil-test.js
@@ -11,13 +11,13 @@ import * as annotatorUtil from '../../annotatorUtil';
 const sandbox = sinon.sandbox.create();
 let stubs = {};
 
-describe('lib/annotations/doc/docAnnotatorUtil', () => {
+describe('doc/docAnnotatorUtil', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
-        fixture.load('annotations/doc/__tests__/docAnnotatorUtil-test.html');
+        fixture.load('doc/__tests__/docAnnotatorUtil-test.html');
     });
 
     afterEach(() => {
diff --git a/src/drawing/__tests__/DrawingContainer-test.js b/src/drawing/__tests__/DrawingContainer-test.js
@@ -3,9 +3,9 @@ import DrawingContainer from '../DrawingContainer';
 let drawingContainer;
 const sandbox = sinon.sandbox.create();
 
-describe('lib/annotations/drawing/DrawingContainer', () => {
+describe('drawing/DrawingContainer', () => {
     before(() => {
-        fixture.setBase('src/lib');
+        fixture.setBase('src');
     });
 
     beforeEach(() => {
diff --git a/src/drawing/__tests__/DrawingModeController-test.js b/src/drawing/__tests__/DrawingModeController-test.js
diff --git a/src/drawing/__tests__/DrawingPath-test.js b/src/drawing/__tests__/DrawingPath-test.js
diff --git a/src/drawing/__tests__/DrawingThread-test.js b/src/drawing/__tests__/DrawingThread-test.js
diff --git a/src/image/__tests__/ImageAnnotator-test.js b/src/image/__tests__/ImageAnnotator-test.js
diff --git a/src/image/__tests__/ImagePointDialog-test.js b/src/image/__tests__/ImagePointDialog-test.js
diff --git a/src/image/__tests__/ImagePointThread-test.js b/src/image/__tests__/ImagePointThread-test.js
diff --git a/src/image/__tests__/imageAnnotatorUtil-test.js b/src/image/__tests__/imageAnnotatorUtil-test.js

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "box-annotations",`
`3`		`- "version": "0.0.0",`
	`3`	`+ "version": "0.0.9",`
`4`	`4`	`"description": "Box Annotations",`
`5`	`5`	`"author": "Box (https://www.box.com/)",`
`6`	`6`	`"license": "SEE LICENSE IN LICENSE",`
Original file line number	Diff line number	Diff line change
`@@ -130,5 +130,4 @@ class BoxAnnotations {`
`130`	`130`	`}`
`131`	`131`	`}`
`132`	`132`
`133`		`-global.BoxAnnotations = BoxAnnotations;`
`134`	`133`	`export default BoxAnnotations;`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-@import '../_boxuiVariables';`
	`1`	`+@import 'boxuiVariables';`
`2`	`2`
`3`	`3`	`$tablet: "(min-width: 768px)";`
`4`	`4`