Clear selection on page change

packages/app/obojobo-document-engine/__tests__/Viewer/components/inline-nav-button.test.js

@@ -6,7 +6,8 @@ import NavButton from 'src/scripts/viewer/components/inline-nav-button'
 
 jest.mock('src/scripts/viewer/util/nav-util', () => ({
 	goNext: jest.fn(),
-	goPrev: jest.fn()
+	goPrev: jest.fn(),
+	clearSelection: jest.fn()
 }))
 
 const getProps = (type, disabled = false, onClick = null) => ({

packages/app/obojobo-document-engine/__tests__/Viewer/util/nav-util.test.js

@@ -517,4 +517,33 @@ describe('NavUtil', () => {
 		NavUtil.resetContext()
 		expect(Common.flux.Dispatcher.trigger).toHaveBeenCalledWith('nav:resetContext')
 	})
+
+	test('clearSelection clears the current selection', () => {
+		const mockRemoveAllRanges = jest.fn()
+		const mockEmpty = jest.fn()
+
+		// Need to make sure removeAllRanges gets called by default
+		// while empty gets called if removeAllRanges isn't available
+		window.getSelection = jest
+			.fn()
+			.mockReturnValueOnce({
+				removeAllRanges: mockRemoveAllRanges,
+				empty: null
+			})
+			.mockReturnValueOnce({
+				removeAllRanges: null,
+				empty: mockEmpty
+			})
+			.mockReturnValueOnce({
+				removeAllRanges: null,
+				empty: null
+			})
+
+		NavUtil.clearSelection()
+		NavUtil.clearSelection()
+		NavUtil.clearSelection()
+
+		expect(mockRemoveAllRanges).toHaveBeenCalledTimes(1)
+		expect(mockEmpty).toHaveBeenCalledTimes(1)
+	})
 })

packages/app/obojobo-document-engine/src/scripts/viewer/components/inline-nav-button.js

@@ -11,6 +11,10 @@ export default class InlineNavButton extends React.Component {
 	}
 
 	onClick(event) {
+		// Make sure any highlighted text gets reset
+		// when navigating to different pages
+		NavUtil.clearSelection()
+
 		if (this.props.onClick) {
 			this.props.onClick()
 			return

packages/app/obojobo-document-engine/src/scripts/viewer/util/nav-util.js

@@ -251,6 +251,16 @@ const NavUtil = {
 
 	isNavOpen(state) {
 		return state.open
+	},
+
+	clearSelection() {
+		const selection = window.getSelection()
+
+		if (selection.removeAllRanges) {
+			selection.removeAllRanges()
+		} else if (selection.empty) {
+			selection.empty()
+		}
 	}
 }