Skip to content
Snippets Groups Projects
Commit b1126ef8 authored by Jack Reed's avatar Jack Reed
Browse files

Update to use a MiradorIcon component rather than the text button

parent 815120b0
No related branches found
No related tags found
No related merge requests found
import React from 'react'; import React from 'react';
import { shallow } from 'enzyme'; import { shallow } from 'enzyme';
import Button from '@material-ui/core/Button'; import { MiradorMenuButton } from 'mirador/dist/es/src/components/MiradorMenuButton';
import miradorAnnotationPlugin from '../src/plugins/miradorAnnotationPlugin'; import miradorAnnotationPlugin from '../src/plugins/miradorAnnotationPlugin';
import AnnotationCreation from '../src/AnnotationCreation'; import AnnotationCreation from '../src/AnnotationCreation';
...@@ -20,14 +20,14 @@ describe('MiradorAnnotation', () => { ...@@ -20,14 +20,14 @@ describe('MiradorAnnotation', () => {
let wrapper; let wrapper;
it('renders a create new button', () => { it('renders a create new button', () => {
wrapper = createWrapper(); wrapper = createWrapper();
expect(wrapper.find(Button).text()).toBe('Create New'); expect(wrapper.find(MiradorMenuButton).props()['aria-label']).toBe('Create new annotation');
}); });
it('opens a new companionWindow when clicked', () => { it('opens a new companionWindow when clicked', () => {
const mockAddCompanionWindow = jest.fn(); const mockAddCompanionWindow = jest.fn();
wrapper = createWrapper({ wrapper = createWrapper({
addCompanionWindow: mockAddCompanionWindow, addCompanionWindow: mockAddCompanionWindow,
}); });
wrapper.find(Button).simulate('click'); wrapper.find(MiradorMenuButton).simulate('click');
expect(mockAddCompanionWindow).toHaveBeenCalledWith( expect(mockAddCompanionWindow).toHaveBeenCalledWith(
'custom', 'custom',
{ {
......
import React, { Component } from 'react'; import React, { Component } from 'react';
import PropTypes from 'prop-types'; import PropTypes from 'prop-types';
import * as actions from 'mirador/dist/es/src/state/actions'; import * as actions from 'mirador/dist/es/src/state/actions';
import Button from '@material-ui/core/Button'; import AddBoxIcon from '@material-ui/icons/AddBox';
import { MiradorMenuButton } from 'mirador/dist/es/src/components/MiradorMenuButton';
import { getVisibleCanvases } from 'mirador/dist/es/src/state/selectors/canvases'; import { getVisibleCanvases } from 'mirador/dist/es/src/state/selectors/canvases';
import AnnotationCreation from '../AnnotationCreation'; import AnnotationCreation from '../AnnotationCreation';
...@@ -42,9 +43,12 @@ class MiradorAnnotation extends Component { ...@@ -42,9 +43,12 @@ class MiradorAnnotation extends Component {
{...targetProps} // eslint-disable-line react/jsx-props-no-spreading {...targetProps} // eslint-disable-line react/jsx-props-no-spreading
/> />
<div> <div>
<Button variant="contained" color="primary" size="small" onClick={this.openCreateAnnotationCompanionWindow}> <MiradorMenuButton
Create New aria-label="Create new annotation"
</Button> onClick={this.openCreateAnnotationCompanionWindow}
>
<AddBoxIcon />
</MiradorMenuButton>
</div> </div>
</div> </div>
); );
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment