diff --git a/plugin-api/beta/client/hocs/withReaction.js b/plugin-api/beta/client/hocs/withReaction.js index f7be6e701..40ee6e14c 100644 --- a/plugin-api/beta/client/hocs/withReaction.js +++ b/plugin-api/beta/client/hocs/withReaction.js @@ -54,6 +54,13 @@ export default (reaction) => (WrappedComponent) => { id: fragmentId }); + if (!data) { + if (self) { + throw new Error(`Comment ${action.item_id} was not found`); + } + return; + } + // Add our comment from the mutation to the end. let idx = data.action_summaries.findIndex(isReaction); @@ -96,6 +103,13 @@ export default (reaction) => (WrappedComponent) => { id: fragmentId }); + if (!data) { + if (self) { + throw new Error(`Comment ${action.item_id} was not found`); + } + return; + } + // Check whether we liked this comment. const idx = data.action_summaries.findIndex(isReaction); diff --git a/plugins/coral-plugin-offtopic/client/components/styles.css b/plugins/coral-plugin-offtopic/client/components/styles.css index 94d7eaa85..24da7f587 100644 --- a/plugins/coral-plugin-offtopic/client/components/styles.css +++ b/plugins/coral-plugin-offtopic/client/components/styles.css @@ -8,10 +8,10 @@ } .tag { - background: #3D73D5; + background: #D2D7D3; font-size: 12px; font-weight: bold; - color: white; + color: #4A4A4A; display: inline-block; margin: 0px 5px; padding: 5px 5px; diff --git a/plugins/coral-plugin-viewing-options/client/actions.js b/plugins/coral-plugin-viewing-options/client/actions.js index 8a6473e3c..fbac224bf 100644 --- a/plugins/coral-plugin-viewing-options/client/actions.js +++ b/plugins/coral-plugin-viewing-options/client/actions.js @@ -1,9 +1,9 @@ -import {VIEWING_OPTIONS_OPEN, VIEWING_OPTIONS_CLOSE} from './constants'; +import {OPEN_MENU, CLOSE_MENU} from './constants'; -export const openViewingOptions = () => ({ - type: VIEWING_OPTIONS_OPEN +export const openMenu = () => ({ + type: OPEN_MENU, }); -export const closeViewingOptions = () => ({ - type: VIEWING_OPTIONS_CLOSE +export const closeMenu = () => ({ + type: CLOSE_MENU, }); diff --git a/plugins/coral-plugin-viewing-options/client/components/ViewingOptions.js b/plugins/coral-plugin-viewing-options/client/components/ViewingOptions.js index 13b3094a1..1d4708250 100644 --- a/plugins/coral-plugin-viewing-options/client/components/ViewingOptions.js +++ b/plugins/coral-plugin-viewing-options/client/components/ViewingOptions.js @@ -8,15 +8,15 @@ import {Slot, ClickOutside} from 'plugin-api/beta/client/components'; const ViewingOptions = (props) => { const toggleOpen = () => { if (!props.open) { - props.openViewingOptions(); + props.openMenu(); } else { - props.closeViewingOptions(); + props.closeMenu(); } }; const handleClickOutside = () => { if (props.open) { - props.closeViewingOptions(); + props.closeMenu(); } }; diff --git a/plugins/coral-plugin-viewing-options/client/constants.js b/plugins/coral-plugin-viewing-options/client/constants.js index 586fe7a28..857a76e1b 100644 --- a/plugins/coral-plugin-viewing-options/client/constants.js +++ b/plugins/coral-plugin-viewing-options/client/constants.js @@ -1,2 +1,5 @@ -export const VIEWING_OPTIONS_OPEN = 'VIEWING_OPTIONS_OPEN'; -export const VIEWING_OPTIONS_CLOSE = 'VIEWING_OPTIONS_CLOSE'; +const prefix = 'TALK_VIEWING_OPTIONS'; + +export const OPEN_MENU = `${prefix}_OPEN_MENU`; +export const CLOSE_MENU = `${prefix}_CLOSE_MENU`; + diff --git a/plugins/coral-plugin-viewing-options/client/containers/ViewingOptions.js b/plugins/coral-plugin-viewing-options/client/containers/ViewingOptions.js index cbe1e7fa3..84900b9c6 100644 --- a/plugins/coral-plugin-viewing-options/client/containers/ViewingOptions.js +++ b/plugins/coral-plugin-viewing-options/client/containers/ViewingOptions.js @@ -1,13 +1,13 @@ import {connect} from 'plugin-api/beta/client/hocs'; import {bindActionCreators} from 'redux'; import ViewingOptions from '../components/ViewingOptions'; -import {openViewingOptions, closeViewingOptions} from '../actions'; +import {openMenu, closeMenu} from '../actions'; const mapStateToProps = ({coralPluginViewingOptions: state}) => ({ open: state.open }); const mapDispatchToProps = (dispatch) => - bindActionCreators({openViewingOptions, closeViewingOptions}, dispatch); + bindActionCreators({openMenu, closeMenu}, dispatch); export default connect(mapStateToProps, mapDispatchToProps)(ViewingOptions); diff --git a/plugins/coral-plugin-viewing-options/client/reducer.js b/plugins/coral-plugin-viewing-options/client/reducer.js index ab9f077df..687de803a 100644 --- a/plugins/coral-plugin-viewing-options/client/reducer.js +++ b/plugins/coral-plugin-viewing-options/client/reducer.js @@ -1,17 +1,17 @@ -import {VIEWING_OPTIONS_OPEN, VIEWING_OPTIONS_CLOSE} from './constants'; +import {OPEN_MENU, CLOSE_MENU} from './constants'; const initialState = { open: false }; -export default function offTopic (state = initialState, action) { +export default function reducer(state = initialState, action) { switch (action.type) { - case VIEWING_OPTIONS_OPEN: + case OPEN_MENU: return { ...state, open: true }; - case VIEWING_OPTIONS_CLOSE: + case CLOSE_MENU: return { ...state, open: false diff --git a/plugins/talk-plugin-featured-comments/client/constants.js b/plugins/talk-plugin-featured-comments/client/constants.js index 7aad99014..c03bf4839 100644 --- a/plugins/talk-plugin-featured-comments/client/constants.js +++ b/plugins/talk-plugin-featured-comments/client/constants.js @@ -1,2 +1,4 @@ -export const SHOW_TOOLTIP = 'SHOW_TOOLTIP'; -export const HIDE_TOOLTIP = 'HIDE_TOOLTIP'; +const prefix = 'TALK_FEATURED_COMMENTS'; + +export const SHOW_TOOLTIP = `${prefix}_SHOW_TOOLTIP`; +export const HIDE_TOOLTIP = `${prefix}_HIDE_TOOLTIP`;