From e24a0caa20fcd3c25ef12dac6e0459e2b91f3426 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Tue, 21 Feb 2017 12:08:02 -0700 Subject: [PATCH 01/36] add EmptyCard component. fix typos. more PropTypes. --- .../coral-admin/src/components/EmptyCard.js | 14 ++++++++++ .../ModerationQueue/ModerationQueue.js | 15 ++++++++--- .../ModerationQueue/components/Comment.js | 26 ++++++++++++++++--- .../ModerationQueue/components/FlagBox.js | 4 +-- 4 files changed, 51 insertions(+), 8 deletions(-) create mode 100644 client/coral-admin/src/components/EmptyCard.js diff --git a/client/coral-admin/src/components/EmptyCard.js b/client/coral-admin/src/components/EmptyCard.js new file mode 100644 index 000000000..e0f837e69 --- /dev/null +++ b/client/coral-admin/src/components/EmptyCard.js @@ -0,0 +1,14 @@ +import React, {PropTypes} from 'react'; +import {Card} from 'coral-ui'; + +const EmptyCard = props => ( + + {props.children} + +); + +EmptyCard.propTypes = { + children: PropTypes.node.isRequired +}; + +export default EmptyCard; diff --git a/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js b/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js index d30f24c1f..69685a22e 100644 --- a/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js +++ b/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js @@ -1,14 +1,17 @@ import React, {PropTypes} from 'react'; import Comment from './components/Comment'; +import EmptyCard from '../../components/EmptyCard'; import {actionsMap} from './helpers/moderationQueueActionsMap'; const ModerationQueue = ({activeTab = 'premod', ...props}) => { + const areComments = props.data[activeTab].length; return (
-
@@ -29,7 +33,12 @@ const ModerationQueue = ({activeTab = 'premod', ...props}) => { }; ModerationQueue.propTypes = { - data: PropTypes.object.isRequired + data: PropTypes.object.isRequired, + acceptComment: PropTypes.func.isRequired, + rejectComment: PropTypes.func.isRequired, + showBanUserDialog: PropTypes.func.isRequired, + currentAsset: PropTypes.object, + suspectWords: PropTypes.arrayOf(PropTypes.string).isRequired }; export default ModerationQueue; diff --git a/client/coral-admin/src/containers/ModerationQueue/components/Comment.js b/client/coral-admin/src/containers/ModerationQueue/components/Comment.js index f8f50c6df..c8818aab9 100644 --- a/client/coral-admin/src/containers/ModerationQueue/components/Comment.js +++ b/client/coral-admin/src/containers/ModerationQueue/components/Comment.js @@ -1,4 +1,4 @@ -import React from 'react'; +import React, {PropTypes} from 'react'; import timeago from 'timeago.js'; import Linkify from 'react-linkify'; import Highlighter from 'react-highlight-words'; @@ -17,7 +17,7 @@ const lang = new I18n(translations); const Comment = ({actions = [], ...props}) => { const links = linkify.getMatches(props.comment.body); - const actionSumaries = props.comment.action_summaries; + const actionSummaries = props.comment.action_summaries; return (
  • @@ -62,11 +62,31 @@ const Comment = ({actions = [], ...props}) => {

    - {actionSumaries && } + {actionSummaries && }
    • ); }; +Comment.propTypes = { + acceptComment: PropTypes.func.isRequired, + rejectComment: PropTypes.func.isRequired, + suspectWords: PropTypes.arrayOf(PropTypes.string).isRequired, + currentAsset: PropTypes.object, + isActive: PropTypes.bool.isRequired, + comment: PropTypes.shape({ + body: PropTypes.string.isRequired, + action_summaries: PropTypes.array, + created_at: PropTypes.string.isRequired, + user: PropTypes.shape({ + banned: PropTypes.bool + }), + asset: PropTypes.shape({ + title: PropTypes.string, + id: PropTypes.string + }) + }) +}; + const linkStyles = { backgroundColor: 'rgb(255, 219, 135)', padding: '1px 2px' diff --git a/client/coral-admin/src/containers/ModerationQueue/components/FlagBox.js b/client/coral-admin/src/containers/ModerationQueue/components/FlagBox.js index bf5a34f29..6140abd90 100644 --- a/client/coral-admin/src/containers/ModerationQueue/components/FlagBox.js +++ b/client/coral-admin/src/containers/ModerationQueue/components/FlagBox.js @@ -5,7 +5,7 @@ const FlagBox = props => (

    Flags:

    @@ -13,7 +13,7 @@ const FlagBox = props => ( ); FlagBox.propTypes = { - actionSumaries: PropTypes.array.isRequired + actionSummaries: PropTypes.array.isRequired }; export default FlagBox; From 9000778d8fac4f027e830b7ccf589b3c880b23f6 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Tue, 21 Feb 2017 12:39:29 -0700 Subject: [PATCH 02/36] empty result for users in Community --- .../coral-admin/src/components/EmptyCard.js | 2 +- .../src/containers/Community/Community.js | 19 ++++++++++--------- .../src/containers/Community/NoResults.js | 9 --------- .../ModerationQueue/ModerationQueue.js | 6 +++++- client/coral-admin/src/translations.json | 4 +++- 5 files changed, 19 insertions(+), 21 deletions(-) delete mode 100644 client/coral-admin/src/containers/Community/NoResults.js diff --git a/client/coral-admin/src/components/EmptyCard.js b/client/coral-admin/src/components/EmptyCard.js index e0f837e69..42337a892 100644 --- a/client/coral-admin/src/components/EmptyCard.js +++ b/client/coral-admin/src/components/EmptyCard.js @@ -2,7 +2,7 @@ import React, {PropTypes} from 'react'; import {Card} from 'coral-ui'; const EmptyCard = props => ( - + {props.children} ); diff --git a/client/coral-admin/src/containers/Community/Community.js b/client/coral-admin/src/containers/Community/Community.js index bad5654cf..08a962c34 100644 --- a/client/coral-admin/src/containers/Community/Community.js +++ b/client/coral-admin/src/containers/Community/Community.js @@ -1,12 +1,12 @@ import React from 'react'; import I18n from 'coral-framework/modules/i18n/i18n'; -import translations from '../../translations.json'; +import translations from 'coral-admin/src/translations.json'; import styles from './Community.css'; import Table from './Table'; import Loading from './Loading'; -import NoResults from './NoResults'; import {Pager} from 'coral-ui'; +import EmptyCard from '../../components/EmptyCard'; const lang = new I18n(translations); @@ -54,13 +54,14 @@ const Community = ({isFetching, commenters, ...props}) => {
    { isFetching && } - { !hasResults && } - { hasResults && - + { + hasResults + ?
    + : {lang.t('community.no-results')} } ( -
    - No users found with that user name or email address -
    -); - -export default NoResults; diff --git a/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js b/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js index 69685a22e..0d48acc92 100644 --- a/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js +++ b/client/coral-admin/src/containers/ModerationQueue/ModerationQueue.js @@ -3,6 +3,10 @@ import React, {PropTypes} from 'react'; import Comment from './components/Comment'; import EmptyCard from '../../components/EmptyCard'; import {actionsMap} from './helpers/moderationQueueActionsMap'; +import I18n from 'coral-framework/modules/i18n/i18n'; +import translations from 'coral-admin/src/translations'; + +const lang = new I18n(translations); const ModerationQueue = ({activeTab = 'premod', ...props}) => { const areComments = props.data[activeTab].length; @@ -25,7 +29,7 @@ const ModerationQueue = ({activeTab = 'premod', ...props}) => { currentAsset={props.currentAsset} />; }) - : No more comments to moderate! You're all caught up. Go have some ☕️ + : {lang.t('modqueue.emptyqueue')} } diff --git a/client/coral-admin/src/translations.json b/client/coral-admin/src/translations.json index 5b15564d7..debfdcc08 100644 --- a/client/coral-admin/src/translations.json +++ b/client/coral-admin/src/translations.json @@ -7,7 +7,7 @@ "admin": "Administrator", "moderator": "Moderator", "role": "Select role...", - "no-results": "No users found with that user name or email address.", + "no-results": "No users found with that user name or email address. They're hiding!", "status": "Status", "select-status": "Select status...", "active": "Active", @@ -32,6 +32,7 @@ "prevcomment": "Go to the previous comment", "singleview": "Toggle single comment edit view", "thismenu": "Open this menu", + "emptyqueue": "No more comments to moderate! You're all caught up. Go have some ☕️", "showshortcuts": "Show Shortcuts" }, "comment": { @@ -152,6 +153,7 @@ "flagged": "marcado", "shortcuts": "Atajos de teclado", "close": "Cerrar", + "emptyqueue": "No se encontro ningún usuario. Están escondidos.", "showshortcuts": "Mostrar atajos" }, "comment": { From f76befd6ac3517ade0962ab069c38eb6312378b0 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Tue, 21 Feb 2017 12:48:01 -0700 Subject: [PATCH 03/36] no comments in user list --- client/coral-settings/translations.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/client/coral-settings/translations.json b/client/coral-settings/translations.json index 2a827a4f0..e255e2b64 100644 --- a/client/coral-settings/translations.json +++ b/client/coral-settings/translations.json @@ -1,12 +1,12 @@ { "en":{ - "userNoComment": "This user has not yet left a comment.", + "userNoComment": "You've never left a comment. Join the conversation!", "allComments": "All Comments", "profileSettings": "Profile Settings", "myCommentHistory": "My comment History" }, "es":{ - "userNoComment": "Aún no ha escrito ningún comentario.", + "userNoComment": "No has dejado áun ningún comentario. ¡Unete a la conversación!", "allComments": "Todos los comentarios", "profileSettings": "Configuración del perfil", "myCommentHistory": "Mi historial de comentarios" From 02c05e8ae157f985a12f12a31ddf29dff0fbfb24 Mon Sep 17 00:00:00 2001 From: gaba Date: Tue, 21 Feb 2017 12:43:02 -0800 Subject: [PATCH 04/36] CSS, translations and code for new create username dialog. --- .../components/CreateUsernameDialog.js | 25 ++++++------ client/coral-sign-in/components/styles.css | 39 ++++++++++++++++++- client/coral-sign-in/translations.js | 14 ++++--- 3 files changed, 60 insertions(+), 18 deletions(-) diff --git a/client/coral-sign-in/components/CreateUsernameDialog.js b/client/coral-sign-in/components/CreateUsernameDialog.js index d245b38ac..457641089 100644 --- a/client/coral-sign-in/components/CreateUsernameDialog.js +++ b/client/coral-sign-in/components/CreateUsernameDialog.js @@ -10,7 +10,7 @@ const lang = new I18n(translations); const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmitUsername, handleChange, ...props}) => (
    - +

    {lang.t('createdisplay.yourusername')}

    +
    Example
    +

    {lang.t('createdisplay.ifyoudontchangeyourname')}

    { props.auth.error && {props.auth.error} }
    - - { props.errors.username && {lang.t('createdisplay.specialCharacters')} } -
    + { props.errors.username && {lang.t('createdisplay.specialCharacters')} } +
    +
    +
    diff --git a/client/coral-sign-in/components/styles.css b/client/coral-sign-in/components/styles.css index 60a252a20..77282d388 100644 --- a/client/coral-sign-in/components/styles.css +++ b/client/coral-sign-in/components/styles.css @@ -117,7 +117,7 @@ input.error{ } .action { - margin-top: 15px; + margin-top: 0px; } .passwordRequestSuccess { @@ -140,6 +140,41 @@ input.error{ display: block; } -.confirmSubmit { +/* Change username Dialog*/ +.dialogusername { + border: none; + box-shadow: 0 9px 46px 8px rgba(0, 0, 0, 0.14), 0 11px 15px -7px rgba(0, 0, 0, 0.12), 0 24px 38px 3px rgba(0, 0, 0, 0.2); + width: 400px; + top: 10px; +} + +.yourusername { + display: block; +} + +.example { + display: block; +} + +.ifyoudont { + display: block; +} + +.saveusername { + display: block; + width: 100%; +} + +.savebutton { + display: inline; + background-color: rgb(105,105,105); + color: white; +} + +.continuebutton { + background-color: black; + font-weight: bold; + color: white; + min-width: 200px; } diff --git a/client/coral-sign-in/translations.js b/client/coral-sign-in/translations.js index a2bb60819..4826f2f2c 100644 --- a/client/coral-sign-in/translations.js +++ b/client/coral-sign-in/translations.js @@ -30,9 +30,11 @@ export default { checkTheForm: 'Invalid Form. Please, check the fields' }, 'createdisplay': { - writeyourusername: 'Write your username', - yourusername: 'Your username is publicly visible on all comments you post. A username is needed before you can post your first comment.', + writeyourusername: 'Edit your username', + yourusername: 'Your username appears on every comment you post.', + ifyoudontchangeyourname: 'If you don\'t change your username at this step, your Facebook display name will appear alongside of all your comments.', username: 'Username', + continue: 'Continue with the same Facebook username', save: 'Save', requiredField: 'Required field', errorCreate: 'Error when changing username', @@ -71,9 +73,11 @@ export default { checkTheForm: 'Formulario Inválido. Por favor, completa los campos' }, 'createdisplay': { - writeyourusername: 'Escribe tu nombre', - yourusername: 'Tu nombre es visible publicamente en todos los comentarios que publiques. Es necesario tener un nombre de usuario antes de poder publicar tu primer comentario.', - username: 'Nombre a mostrar', + writeyourusername: 'Edita tu nombre', + yourusername: 'Tu nombre aparece en cada comentario que publiques.', + ifyoudontchangeyourname: 'Si no modificas tu nombre de usuario en este paso, tu nombre de Facebook aparecera al lado de cada comentario que publiques.', + username: 'Nombre', + continue: 'Continuar con nombre de Facebook', save: 'Guardar', requiredField: 'Campo necesario', errorCreate: 'Hubo un error al cambiar el nombre de usuario', From 9149b737f5bd06f7e10994110c8a1213fcbf9cce Mon Sep 17 00:00:00 2001 From: David Jay Date: Tue, 21 Feb 2017 17:23:09 -0500 Subject: [PATCH 05/36] Retrieving counts and displaying new comments to user. --- client/coral-embed-stream/src/Comment.js | 2 +- client/coral-embed-stream/src/Embed.js | 8 +- client/coral-embed-stream/src/NewCount.js | 18 ++++ client/coral-embed-stream/src/Stream.js | 22 ++++- client/coral-framework/actions/asset.js | 1 + client/coral-framework/constants/asset.js | 1 + .../coral-framework/graphql/queries/index.js | 93 ++++++++++++------- client/coral-framework/reducers/asset.js | 3 + .../reducers/assetReducer.spec.js | 19 ++++ .../reducers/notificationReducer.spec.js | 35 +++++++ 10 files changed, 165 insertions(+), 37 deletions(-) create mode 100644 client/coral-embed-stream/src/NewCount.js create mode 100644 test/client/coral-framework/reducers/assetReducer.spec.js create mode 100644 test/client/coral-framework/reducers/notificationReducer.spec.js diff --git a/client/coral-embed-stream/src/Comment.js b/client/coral-embed-stream/src/Comment.js index 9d679473e..8371cec45 100644 --- a/client/coral-embed-stream/src/Comment.js +++ b/client/coral-embed-stream/src/Comment.js @@ -177,7 +177,7 @@ class Comment extends React.Component { comment.replies &&
    comment.replies.length} diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index a7c4dcb38..6ff153a52 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -12,6 +12,7 @@ const {fetchAssetSuccess} = assetActions; import {queryStream} from 'coral-framework/graphql/queries'; import {postComment, postFlag, postLike, deleteAction} from 'coral-framework/graphql/mutations'; import {editName} from 'coral-framework/actions/user'; +import {updateCountCache} from 'coral-framework/actions/asset'; import {Notification, notificationActions, authActions, assetActions, pym} from 'coral-framework'; import Stream from './Stream'; @@ -28,6 +29,7 @@ import SettingsContainer from 'coral-settings/containers/SettingsContainer'; import RestrictedContent from 'coral-framework/components/RestrictedContent'; import ConfigureStreamContainer from 'coral-configure/containers/ConfigureStreamContainer'; import LoadMore from './LoadMore'; +import NewCount from './NewCount'; class Embed extends Component { @@ -82,7 +84,7 @@ class Embed extends Component { render () { const {activeTab} = this.state; - const {closedAt} = this.props.asset; + const {closedAt, countCache = {}} = this.props.asset; const {loading, asset, refetch} = this.props.data; const {loggedIn, isAdmin, user, showSignInDialog, signInOffset} = this.props.auth; @@ -147,6 +149,7 @@ class Embed extends Component { } {!loggedIn && } {loggedIn && user && } + ({ clearNotification: () => dispatch(clearNotification()), editName: (username) => dispatch(editName(username)), showSignInDialog: (offset) => dispatch(showSignInDialog(offset)), + updateCountCache: (id, count) => dispatch(updateCountCache(id, count)), logout: () => dispatch(logout()), dispatch: d => dispatch(d) }); diff --git a/client/coral-embed-stream/src/NewCount.js b/client/coral-embed-stream/src/NewCount.js new file mode 100644 index 000000000..712c36be3 --- /dev/null +++ b/client/coral-embed-stream/src/NewCount.js @@ -0,0 +1,18 @@ +import React, {PropTypes} from 'react'; + +const NewCount = ({commentCount, countCache}) => +
    + { + countCache && commentCount - countCache > 0 && +
    + Load {commentCount - countCache} More Comments +
    + } +
    ; + +NewCount.propTypes = { + commentCount: PropTypes.number.isRequired, + countCache: PropTypes.number +}; + +export default NewCount; diff --git a/client/coral-embed-stream/src/Stream.js b/client/coral-embed-stream/src/Stream.js index 2f0f4b01b..d59521484 100644 --- a/client/coral-embed-stream/src/Stream.js +++ b/client/coral-embed-stream/src/Stream.js @@ -17,10 +17,30 @@ class Stream extends React.Component { constructor(props) { super(props); - this.state = {activeReplyBox: ''}; + this.state = {activeReplyBox: '', countPoll: null}; this.setActiveReplyBox = this.setActiveReplyBox.bind(this); } + componentDidMount() { + const {asset, getCounts, updateCountCache} = this.props; + + updateCountCache(asset.id, asset.comments.length); + + // Note: Apollo's built-in polling doesn't work with fetchMore queries, so a + // setInterval is being used instead. + this.setState({ + countPoll: setInterval(() => getCounts({ + asset_id: asset.id, + limit: asset.comments.length, + sort: 'REVERSE_CHRONOLOGICAL' + }), 5000), + }); + } + + componentWillUnmount() { + clearInterval(this.state.countPoll); + } + setActiveReplyBox (reactKey) { if (!this.props.currentUser) { const offset = document.getElementById(`c_${reactKey}`).getBoundingClientRect().top - 75; diff --git a/client/coral-framework/actions/asset.js b/client/coral-framework/actions/asset.js index d5db289c3..02e72ea98 100644 --- a/client/coral-framework/actions/asset.js +++ b/client/coral-framework/actions/asset.js @@ -38,6 +38,7 @@ export const updateOpenStream = closedBody => (dispatch, getState) => { const openStream = () => ({type: actions.OPEN_COMMENTS}); const closeStream = () => ({type: actions.CLOSE_COMMENTS}); +export const updateCountCache = (id, count) => ({type: actions.UPDATE_COUNT_CACHE, id, count}); export const updateOpenStatus = status => dispatch => { if (status === 'open') { diff --git a/client/coral-framework/constants/asset.js b/client/coral-framework/constants/asset.js index 40f746706..234095d9d 100644 --- a/client/coral-framework/constants/asset.js +++ b/client/coral-framework/constants/asset.js @@ -8,3 +8,4 @@ export const UPDATE_ASSET_SETTINGS_FAILURE = 'UPDATE_ASSET_SETTINGS_FAILURE'; export const OPEN_COMMENTS = 'OPEN_COMMENTS'; export const CLOSE_COMMENTS = 'CLOSE_COMMENTS'; +export const UPDATE_COUNT_CACHE = 'UPDATE_COUNT_CACHE'; diff --git a/client/coral-framework/graphql/queries/index.js b/client/coral-framework/graphql/queries/index.js index 09b3159d0..494ffa30e 100644 --- a/client/coral-framework/graphql/queries/index.js +++ b/client/coral-framework/graphql/queries/index.js @@ -1,6 +1,7 @@ import {graphql} from 'react-apollo'; import STREAM_QUERY from './streamQuery.graphql'; import LOAD_MORE from './loadMore.graphql'; +import GET_COUNTS from './getCounts.graphql'; import MY_COMMENT_HISTORY from './myCommentHistory.graphql'; function getQueryVariable(variable) { @@ -17,6 +18,62 @@ function getQueryVariable(variable) { return 'http://localhost/default/stream'; } +export const getCounts = (data) => ({asset_id, limit, sort}) => { + return data.fetchMore({ + query: GET_COUNTS, + variables: { + asset_id, + limit, + sort + }, + updateQuery: (oldData, {fetchMoreResult:{data}}) => { + + return { + ...oldData, + asset: { + ...oldData.asset, + commentCount: data.asset.commentCount + } + }; + } + }); +}; + +export const loadMore = (data) => ({limit, cursor, parent_id, asset_id, sort}) => { + return data.fetchMore({ + query: LOAD_MORE, + variables: { + limit, + cursor, + parent_id, + asset_id, + sort + }, + updateQuery: (oldData, {fetchMoreResult:{data:{new_top_level_comments}}}) => + + // If loading more replies + parent_id ? { + ...oldData, + asset: { + ...oldData.asset, + comments: oldData.asset.comments.map((comment) => + comment.id === parent_id + ? {...comment, replies: [...comment.replies, ...new_top_level_comments]} + : comment) + } + } + + // If loading more top-level comments + : { + ...oldData, + asset: { + ...oldData.asset, + comments: [...oldData.asset.comments, ...new_top_level_comments] + } + } + }); +}; + export const queryStream = graphql(STREAM_QUERY, { options: () => ({ variables: { @@ -25,40 +82,8 @@ export const queryStream = graphql(STREAM_QUERY, { }), props: ({data}) => ({ data, - loadMore: ({limit, cursor, parent_id, asset_id, sort}) => { - return data.fetchMore({ - query: LOAD_MORE, - variables: { - limit, - cursor, - parent_id, - asset_id, - sort - }, - updateQuery: (oldData, {fetchMoreResult:{data:{new_top_level_comments}}}) => - - // If loading more replies - parent_id ? { - ...oldData, - asset: { - ...oldData.asset, - comments: oldData.asset.comments.map((comment) => - comment.id === parent_id - ? {...comment, replies: [...comment.replies, ...new_top_level_comments]} - : comment) - } - } - - // If loading more top-level comments - : { - ...oldData, - asset: { - ...oldData.asset, - comments: [...oldData.asset.comments, ...new_top_level_comments] - } - } - }); - } + loadMore: loadMore(data), + getCounts: getCounts(data), }) }); diff --git a/client/coral-framework/reducers/asset.js b/client/coral-framework/reducers/asset.js index f9d0a55e3..067edfc5b 100644 --- a/client/coral-framework/reducers/asset.js +++ b/client/coral-framework/reducers/asset.js @@ -19,6 +19,9 @@ export default function asset (state = initialState, action) { case actions.UPDATE_ASSET_SETTINGS_SUCCESS: return state .setIn(['settings'], action.settings); + case actions.UPDATE_COUNT_CACHE: + return state + .setIn(['countCache', action.id], action.count); default: return state; } diff --git a/test/client/coral-framework/reducers/assetReducer.spec.js b/test/client/coral-framework/reducers/assetReducer.spec.js new file mode 100644 index 000000000..c54f51737 --- /dev/null +++ b/test/client/coral-framework/reducers/assetReducer.spec.js @@ -0,0 +1,19 @@ +import {Map} from 'immutable'; +import {expect} from 'chai'; +import assetReducer from '../../../../client/coral-framework/reducers/asset'; +import * as actions from '../../../../client/coral-framework/constants/asset'; + +describe ('coral-embed-stream assetReducer', () => { + describe('UPDATE_COUNT_CACHE', () => { + it('should update the count cache', () => { + const action = { + type: actions.UPDATE_COUNT_CACHE, + id: '123', + count: 456 + }; + const store = new Map({}); + const result = assetReducer(store, action); + expect(result.getIn(['countCache', '123'])).to.equal(456); + }); + }); +}); diff --git a/test/client/coral-framework/reducers/notificationReducer.spec.js b/test/client/coral-framework/reducers/notificationReducer.spec.js new file mode 100644 index 000000000..40c50d11a --- /dev/null +++ b/test/client/coral-framework/reducers/notificationReducer.spec.js @@ -0,0 +1,35 @@ +import {Map} from 'immutable'; +import {expect} from 'chai'; +import notificationReducer from '../../../../client/coral-framework/reducers/notification'; +import * as actions from '../../../../client/coral-framework/actions/notification'; + +describe ('notificationsReducer', () => { + describe('ADD_NOTIFICATION', () => { + it('should add a notification', () => { + const action = { + type: actions.ADD_NOTIFICATION, + text: 'Test notification', + notifType: 'test' + }; + const store = new Map({}); + const result = notificationReducer(store, action); + expect(result.get('text')).to.equal(action.text); + expect(result.get('type')).to.equal(action.notifType); + }); + }); + + describe('CLEAR_NOTIFICATION', () => { + it('should clear a notification', () => { + const action = { + type: actions.CLEAR_NOTIFICATION + }; + const store = new Map({ + text: 'Test notification', + type: 'test' + }); + const result = notificationReducer(store, action); + expect(result.get('text')).to.equal(''); + expect(result.get('type')).to.equal(''); + }); + }); +}); From ca107afb9b06dad9074860f2734364241eb4ca22 Mon Sep 17 00:00:00 2001 From: David Jay Date: Tue, 21 Feb 2017 17:29:58 -0500 Subject: [PATCH 06/36] Avoiding new comment button when a comment is posted. --- client/coral-embed-stream/src/Embed.js | 2 ++ client/coral-plugin-commentbox/CommentBox.js | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index 6ff153a52..1c7898afb 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -134,6 +134,8 @@ class Embed extends Component { postItem={this.props.postItem} appendItemArray={this.props.appendItemArray} updateItem={this.props.updateItem} + updateCountCache={this.props.updateCountCache} + countCache={countCache[asset.id]} assetId={asset.id} premod={asset.settings.moderation} isReply={false} diff --git a/client/coral-plugin-commentbox/CommentBox.js b/client/coral-plugin-commentbox/CommentBox.js index 344ade150..3d895df07 100644 --- a/client/coral-plugin-commentbox/CommentBox.js +++ b/client/coral-plugin-commentbox/CommentBox.js @@ -29,6 +29,8 @@ class CommentBox extends Component { commentPostedHandler, postItem, assetId, + updateCountCache, + countCache, parentId, addNotification, authorId @@ -44,14 +46,17 @@ class CommentBox extends Component { if (this.props.charCount && this.state.body.length > this.props.charCount) { return; } + updateCountCache(assetId, countCache + 1); postItem(comment, 'comments') .then(({data}) => { const postedComment = data.createComment.comment; if (postedComment.status === 'REJECTED') { addNotification('error', lang.t('comment-post-banned-word')); + updateCountCache(assetId, countCache); } else if (postedComment.status === 'PREMOD') { addNotification('success', lang.t('comment-post-notif-premod')); + updateCountCache(assetId, countCache); } else { addNotification('success', 'Your comment has been posted.'); } From 5d0730b4d5f8d5078d65cf824e67c1afd86c2521 Mon Sep 17 00:00:00 2001 From: gaba Date: Tue, 21 Feb 2017 14:52:19 -0800 Subject: [PATCH 07/36] Adds fake comment component. --- .../components/CreateUsernameDialog.js | 14 +++-- .../coral-sign-in/components/FakeComment.js | 54 +++++++++++++++++++ client/coral-sign-in/components/styles.css | 6 +++ client/coral-sign-in/translations.js | 4 ++ 4 files changed, 75 insertions(+), 3 deletions(-) create mode 100644 client/coral-sign-in/components/FakeComment.js diff --git a/client/coral-sign-in/components/CreateUsernameDialog.js b/client/coral-sign-in/components/CreateUsernameDialog.js index 457641089..a01137733 100644 --- a/client/coral-sign-in/components/CreateUsernameDialog.js +++ b/client/coral-sign-in/components/CreateUsernameDialog.js @@ -3,7 +3,10 @@ import TextField from 'coral-ui/components/TextField'; import Alert from './Alert'; import Button from 'coral-ui/components/Button'; import {Dialog} from 'coral-ui'; +import FakeComment from './FakeComment'; + import styles from './styles.css'; + import I18n from 'coral-framework/modules/i18n/i18n'; import translations from '../translations'; const lang = new I18n(translations); @@ -25,9 +28,14 @@ const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmit
    -

    {lang.t('createdisplay.yourusername')}

    -
    Example
    -

    {lang.t('createdisplay.ifyoudontchangeyourname')}

    +

    {lang.t('createdisplay.yourusername')}

    + +

    {lang.t('createdisplay.ifyoudontchangeyourname')}

    { props.auth.error && {props.auth.error} }
    { props.errors.username && {lang.t('createdisplay.specialCharacters')} } diff --git a/client/coral-sign-in/components/FakeComment.js b/client/coral-sign-in/components/FakeComment.js new file mode 100644 index 000000000..97afa59fa --- /dev/null +++ b/client/coral-sign-in/components/FakeComment.js @@ -0,0 +1,54 @@ +import React from 'react'; +import styles from 'coral-embed-stream/src/Comment.css'; + +import PermalinkButton from 'coral-plugin-permalinks/PermalinkButton'; +import AuthorName from 'coral-plugin-author-name/AuthorName'; +import Content from 'coral-plugin-commentcontent/CommentContent'; +import PubDate from 'coral-plugin-pubdate/PubDate'; +import {ReplyButton} from 'coral-plugin-replies'; +import FlagComment from 'coral-plugin-flags/FlagComment'; +import LikeButton from 'coral-plugin-likes/LikeButton'; + +class FakeComment extends React.Component { + constructor (props) { + super(props); + } + + render () { + const {username, created_at, body} = this.props; + + return ( +
    +
    + + + +
    + {return;}} + deleteAction={()=>{return;}} + showSignInDialog={()=>{return;}} + currentUser={{}} + /> + {}} + parentCommentId={'commentID'} + currentUserId={{}} + banned={false} + /> +
    +
    + + +
    +
    + ); + } +} + +export default FakeComment; diff --git a/client/coral-sign-in/components/styles.css b/client/coral-sign-in/components/styles.css index 77282d388..81afe75e5 100644 --- a/client/coral-sign-in/components/styles.css +++ b/client/coral-sign-in/components/styles.css @@ -159,6 +159,7 @@ input.error{ .ifyoudont { display: block; + margin-top: 15px; } .saveusername { @@ -178,3 +179,8 @@ input.error{ color: white; min-width: 200px; } + +.fakeComment { + display: block; + margin-bottom: 5px; +} diff --git a/client/coral-sign-in/translations.js b/client/coral-sign-in/translations.js index 4826f2f2c..5693939af 100644 --- a/client/coral-sign-in/translations.js +++ b/client/coral-sign-in/translations.js @@ -36,6 +36,8 @@ export default { username: 'Username', continue: 'Continue with the same Facebook username', save: 'Save', + fakecommentdate: '1 minute ago', + fakecommentbody: 'This is an example comment. Readers can share their thoughts and opinions with newsrooms in the comments section.', requiredField: 'Required field', errorCreate: 'Error when changing username', checkTheForm: 'Invalid Form. Please, check the fields', @@ -79,6 +81,8 @@ export default { username: 'Nombre', continue: 'Continuar con nombre de Facebook', save: 'Guardar', + fakecommentdate: 'hace un minuto', + fakecommentbody: 'Este es un comentario de ejemplo. Las lectoras pueden compartir sus ideas y opiniones con los periodistas en la sección de comentarios.', requiredField: 'Campo necesario', errorCreate: 'Hubo un error al cambiar el nombre de usuario', checkTheForm: 'Formulario Invalido. Por favor, verifica los campos', From e7c4acde4d07c47b4e053cdf95f148713ccecf15 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Tue, 21 Feb 2017 16:11:56 -0700 Subject: [PATCH 08/36] small bugfix --- .../src/containers/ModerationQueue/components/Comment.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/client/coral-admin/src/containers/ModerationQueue/components/Comment.js b/client/coral-admin/src/containers/ModerationQueue/components/Comment.js index c8818aab9..b39eeb2f8 100644 --- a/client/coral-admin/src/containers/ModerationQueue/components/Comment.js +++ b/client/coral-admin/src/containers/ModerationQueue/components/Comment.js @@ -42,7 +42,7 @@ const Comment = ({actions = [], ...props}) => { /> )}
    - {props.comment.user.banned === 'banned' ? + {props.comment.user.status === 'banned' ? {lang.t('comment.banned_user')} @@ -78,7 +78,7 @@ Comment.propTypes = { action_summaries: PropTypes.array, created_at: PropTypes.string.isRequired, user: PropTypes.shape({ - banned: PropTypes.bool + status: PropTypes.string }), asset: PropTypes.shape({ title: PropTypes.string, From 1b79a105802ab456b16ecf6af3dc7f396e5dd398 Mon Sep 17 00:00:00 2001 From: gaba Date: Tue, 21 Feb 2017 16:36:50 -0800 Subject: [PATCH 09/36] Change username when it changes. --- client/coral-sign-in/components/CreateUsernameDialog.js | 8 +++++--- client/coral-sign-in/components/FakeComment.js | 1 - .../coral-sign-in/containers/ChangeUsernameContainer.js | 1 + 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/client/coral-sign-in/components/CreateUsernameDialog.js b/client/coral-sign-in/components/CreateUsernameDialog.js index a01137733..f0edc7398 100644 --- a/client/coral-sign-in/components/CreateUsernameDialog.js +++ b/client/coral-sign-in/components/CreateUsernameDialog.js @@ -11,7 +11,8 @@ import I18n from 'coral-framework/modules/i18n/i18n'; import translations from '../translations'; const lang = new I18n(translations); -const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmitUsername, handleChange, ...props}) => ( +const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmitUsername, handleChange, ...props}) => { + return ( {lang.t('createdisplay.yourusername')}

    @@ -54,6 +55,7 @@ const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmit     -); + ); +}; export default CreateUsernameDialog; diff --git a/client/coral-sign-in/components/FakeComment.js b/client/coral-sign-in/components/FakeComment.js index 97afa59fa..c496e69d9 100644 --- a/client/coral-sign-in/components/FakeComment.js +++ b/client/coral-sign-in/components/FakeComment.js @@ -39,7 +39,6 @@ class FakeComment extends React.Component { onClick={() => {}} parentCommentId={'commentID'} currentUserId={{}} - banned={false} />
    diff --git a/client/coral-sign-in/containers/ChangeUsernameContainer.js b/client/coral-sign-in/containers/ChangeUsernameContainer.js index 33ae63614..ca065169f 100644 --- a/client/coral-sign-in/containers/ChangeUsernameContainer.js +++ b/client/coral-sign-in/containers/ChangeUsernameContainer.js @@ -29,6 +29,7 @@ class ChangeUsernameContainer extends Component { constructor(props) { super(props); + this.initialState.formData.username = props.user.username; this.state = this.initialState; this.handleChange = this.handleChange.bind(this); this.handleSubmitUsername = this.handleSubmitUsername.bind(this); From 2c60285981581b4d3777f95fe6eacc95f43ff18a Mon Sep 17 00:00:00 2001 From: gaba Date: Wed, 22 Feb 2017 07:00:15 -0800 Subject: [PATCH 10/36] The component needs the banned value. --- client/coral-sign-in/components/FakeComment.js | 1 + 1 file changed, 1 insertion(+) diff --git a/client/coral-sign-in/components/FakeComment.js b/client/coral-sign-in/components/FakeComment.js index c496e69d9..97afa59fa 100644 --- a/client/coral-sign-in/components/FakeComment.js +++ b/client/coral-sign-in/components/FakeComment.js @@ -39,6 +39,7 @@ class FakeComment extends React.Component { onClick={() => {}} parentCommentId={'commentID'} currentUserId={{}} + banned={false} />
    From 80706d2fdff6ba547762d359270fe7d49671ac70 Mon Sep 17 00:00:00 2001 From: Wyatt Johnson Date: Wed, 22 Feb 2017 10:33:47 -0700 Subject: [PATCH 11/36] fixed issue related to action create on suspect word match --- graph/mutators/comment.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/graph/mutators/comment.js b/graph/mutators/comment.js index 33d2c4f85..5112ba829 100644 --- a/graph/mutators/comment.js +++ b/graph/mutators/comment.js @@ -150,10 +150,8 @@ const createPublicComment = (context, commentInput) => { item_id: comment.id, item_type: 'COMMENTS', action_type: 'FLAG', - metadata: { - field: 'body', - details: 'Matched suspect word filters.' - } + group_id: 'Matched suspect word filter', + metadata: {} }) .then(() => comment); } From dd09ddf968f31c6a3848a527c89707cb215cee31 Mon Sep 17 00:00:00 2001 From: David Jay Date: Wed, 22 Feb 2017 13:13:25 -0500 Subject: [PATCH 12/36] Adding NewCount component. --- client/coral-embed-stream/src/Embed.js | 6 +++++- client/coral-embed-stream/src/NewCount.js | 6 +++--- client/coral-embed-stream/src/Stream.js | 2 +- 3 files changed, 9 insertions(+), 5 deletions(-) diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index 1c7898afb..ac9acc931 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -151,7 +151,11 @@ class Embed extends Component { } {!loggedIn && } {loggedIn && user && } - + -
    +const NewCount = ({commentCount, countCache}) =>
    { countCache && commentCount - countCache > 0 &&
    @@ -12,7 +11,8 @@ const NewCount = ({commentCount, countCache}) => NewCount.propTypes = { commentCount: PropTypes.number.isRequired, - countCache: PropTypes.number + countCache: PropTypes.number, + loadMore: PropTypes.func.isRequired }; export default NewCount; diff --git a/client/coral-embed-stream/src/Stream.js b/client/coral-embed-stream/src/Stream.js index d59521484..e68a223ca 100644 --- a/client/coral-embed-stream/src/Stream.js +++ b/client/coral-embed-stream/src/Stream.js @@ -24,7 +24,7 @@ class Stream extends React.Component { componentDidMount() { const {asset, getCounts, updateCountCache} = this.props; - updateCountCache(asset.id, asset.comments.length); + updateCountCache(asset.id, asset.commentCount); // Note: Apollo's built-in polling doesn't work with fetchMore queries, so a // setInterval is being used instead. From 2acd099f3f598252e604bffcf98dc5205d4c75d6 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Wed, 22 Feb 2017 11:15:13 -0700 Subject: [PATCH 13/36] add empty state for asset search --- .../src/containers/Streams/Streams.js | 36 ++++++++++--------- client/coral-admin/src/translations.json | 2 ++ 2 files changed, 22 insertions(+), 16 deletions(-) diff --git a/client/coral-admin/src/containers/Streams/Streams.js b/client/coral-admin/src/containers/Streams/Streams.js index 62f2f009e..96130fcba 100644 --- a/client/coral-admin/src/containers/Streams/Streams.js +++ b/client/coral-admin/src/containers/Streams/Streams.js @@ -8,6 +8,7 @@ import {Link} from 'react-router'; import {Pager, Icon} from 'coral-ui'; import {DataTable, TableHeader, RadioGroup, Radio} from 'react-mdl'; +import EmptyCard from 'coral-admin/src/components/EmptyCard'; const limit = 25; @@ -142,22 +143,25 @@ class Streams extends Component { {lang.t('streams.oldest')}
    -
    - - {lang.t('streams.article')} - - {lang.t('streams.pubdate')} - - - {lang.t('streams.status')} - - - -
    + { + assetsIds.length + ?
    + + {lang.t('streams.article')} + + {lang.t('streams.pubdate')} + + + {lang.t('streams.status')} + + + +
    + : {lang.t('streams.empty_result')} + }
    ); } diff --git a/client/coral-admin/src/translations.json b/client/coral-admin/src/translations.json index debfdcc08..7219a22ca 100644 --- a/client/coral-admin/src/translations.json +++ b/client/coral-admin/src/translations.json @@ -114,6 +114,7 @@ "comment_count": "Comments" }, "streams": { + "empty_result": "No assets match this search. Maybe try widening your search?", "search": "Search", "filter-streams": "Filter Streams", "stream-status": "Stream Status", @@ -222,6 +223,7 @@ "comment_count": "Comentarios" }, "streams": { + "empty_result": "No se encuentro articulo con esta busqueda. Tal vez extender la busqueda?", "search": "", "filter-streams": "", "stream-status": "", From 096bd12df6e2b73c6d3878544a2dd9039952c960 Mon Sep 17 00:00:00 2001 From: David Jay Date: Wed, 22 Feb 2017 13:45:02 -0500 Subject: [PATCH 14/36] Loading new comments. --- client/coral-embed-stream/src/Embed.js | 7 ++++ client/coral-embed-stream/src/LoadMore.js | 2 +- client/coral-embed-stream/src/NewCount.js | 35 ++++++++++++++----- .../coral-framework/graphql/queries/index.js | 5 +-- graph/typeDefs.graphql | 2 +- 5 files changed, 38 insertions(+), 13 deletions(-) diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index ac9acc931..487437203 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -100,6 +100,10 @@ class Embed extends Component { return ; } + // Find the created_at date of the first comment. If no comments exist, set the date to a week ago. + const firstCommentDate = asset.comments[0] ? asset.comments[0].created_at + : new Date(Date.now() - 604800000); + return (
    @@ -155,6 +159,9 @@ class Embed extends Component { commentCount={asset.commentCount} countCache={countCache[asset.id]} loadMore={this.props.loadMore} + firstCommentDate={firstCommentDate} + assetId={asset.id} + updateCountCache={this.props.updateCountCache} /> { } const cursor = parentId - ? comments[1].created_at + ? comments[0].created_at : comments[comments.length - 1].created_at; loadMore({ diff --git a/client/coral-embed-stream/src/NewCount.js b/client/coral-embed-stream/src/NewCount.js index ded006325..99e1297e9 100644 --- a/client/coral-embed-stream/src/NewCount.js +++ b/client/coral-embed-stream/src/NewCount.js @@ -1,18 +1,35 @@ import React, {PropTypes} from 'react'; -const NewCount = ({commentCount, countCache}) =>
    - { - countCache && commentCount - countCache > 0 && -
    - Load {commentCount - countCache} More Comments -
    - } -
    ; +const onLoadMoreClick = ({loadMore, commentCount, firstCommentDate, assetId, updateCountCache}) => (e) => { + e.preventDefault(); + updateCountCache(assetId, commentCount); + loadMore({ + limit: 500, + cursor: firstCommentDate, + assetId, + sort: 'CHRONOLOGICAL' + }, true); +}; + +const NewCount = (props) => { + const newComments = props.commentCount - props.countCache; + + return
    + { + props.countCache && newComments > 0 && +
    + Load {newComments} More Comments +
    + } +
    ; +}; NewCount.propTypes = { commentCount: PropTypes.number.isRequired, countCache: PropTypes.number, - loadMore: PropTypes.func.isRequired + loadMore: PropTypes.func.isRequired, + assetId: PropTypes.string.isRequired, + firstCommentDate: PropTypes.string.isRequired }; export default NewCount; diff --git a/client/coral-framework/graphql/queries/index.js b/client/coral-framework/graphql/queries/index.js index 494ffa30e..9b8a55134 100644 --- a/client/coral-framework/graphql/queries/index.js +++ b/client/coral-framework/graphql/queries/index.js @@ -39,7 +39,7 @@ export const getCounts = (data) => ({asset_id, limit, sort}) => { }); }; -export const loadMore = (data) => ({limit, cursor, parent_id, asset_id, sort}) => { +export const loadMore = (data) => ({limit, cursor, parent_id, asset_id, sort}, newComments) => { return data.fetchMore({ query: LOAD_MORE, variables: { @@ -68,7 +68,8 @@ export const loadMore = (data) => ({limit, cursor, parent_id, asset_id, sort}) = ...oldData, asset: { ...oldData.asset, - comments: [...oldData.asset.comments, ...new_top_level_comments] + comments: newComments ? [...new_top_level_comments.reverse(), ...oldData.asset.comments] + : [...oldData.asset.comments, ...new_top_level_comments] } } }); diff --git a/graph/typeDefs.graphql b/graph/typeDefs.graphql index 72a54a0cd..7db4dbad8 100644 --- a/graph/typeDefs.graphql +++ b/graph/typeDefs.graphql @@ -448,7 +448,7 @@ type RootQuery { # Comments returned based on a query. comments(query: CommentsQuery!): [Comment] - # Returne the count of comments satisfied by the query. Note that this edge is + # Return the count of comments satisfied by the query. Note that this edge is # expensive as it is not batched. Requires the `ADMIN` role. commentCount(query: CommentCountQuery!): Int From 32bc754bf7435e3caa92c466f79a912c208052a4 Mon Sep 17 00:00:00 2001 From: gaba Date: Wed, 22 Feb 2017 11:22:51 -0800 Subject: [PATCH 15/36] It needs to send the translation key --- client/coral-framework/helpers/response.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/coral-framework/helpers/response.js b/client/coral-framework/helpers/response.js index 64803faef..0fce878cd 100644 --- a/client/coral-framework/helpers/response.js +++ b/client/coral-framework/helpers/response.js @@ -45,7 +45,7 @@ const handleResp = res => { } if (err.error && err.error.translation_key) { - message = err.error.translation_key; + error.translation_key = err.error.translation_key; } error.message = message; From 6becb88b560306842726acd90a54f07ad9d93696 Mon Sep 17 00:00:00 2001 From: David Jay Date: Wed, 22 Feb 2017 15:07:32 -0500 Subject: [PATCH 16/36] Adding style to new comments button. --- client/coral-embed-stream/src/NewCount.js | 13 +++++++++---- client/coral-embed-stream/style/default.css | 12 ++++++++++-- client/coral-framework/translations.json | 3 +++ 3 files changed, 22 insertions(+), 6 deletions(-) diff --git a/client/coral-embed-stream/src/NewCount.js b/client/coral-embed-stream/src/NewCount.js index 99e1297e9..429ecc113 100644 --- a/client/coral-embed-stream/src/NewCount.js +++ b/client/coral-embed-stream/src/NewCount.js @@ -1,4 +1,7 @@ import React, {PropTypes} from 'react'; +import I18n from 'coral-framework/modules/i18n/i18n'; +import translations from 'coral-framework/translations.json'; +const lang = new I18n(translations); const onLoadMoreClick = ({loadMore, commentCount, firstCommentDate, assetId, updateCountCache}) => (e) => { e.preventDefault(); @@ -14,12 +17,14 @@ const onLoadMoreClick = ({loadMore, commentCount, firstCommentDate, assetId, upd const NewCount = (props) => { const newComments = props.commentCount - props.countCache; - return
    + return
    { props.countCache && newComments > 0 && -
    - Load {newComments} More Comments -
    + }
    ; }; diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 1deae6026..0e4f7e4a6 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -336,18 +336,26 @@ button.coral-load-more { text-align: center; color: #FFF; background-color: #2376D8; + cursor: pointer; } button.coral-load-more:hover { background-color: #4399FF; } -.coral-load-more-replies { +.coral-load-more-replies, .coral-new-comments { width: 100%; display: flex; justify-content: center; + cursor: pointer; } -.coral-load-more-replies button.coral-load-more { +.coral-new-comments { + position: relative; + top: 1.8em; + z-index: 100; +} + +.coral-load-more-replies button.coral-load-more, .coral-new-comments button.coral-load-more{ width: initial; } diff --git a/client/coral-framework/translations.json b/client/coral-framework/translations.json index 2884178b6..23405d2d8 100644 --- a/client/coral-framework/translations.json +++ b/client/coral-framework/translations.json @@ -11,6 +11,9 @@ "button": "Submit", "error": "Usernames can contain letters, numbers and _ only" }, + "newCount": "View {0} more {1}", + "comment": "comment", + "comments": "comments", "error": { "emailNotVerified": "Email address {0} not verified.", "email": "Not a valid E-Mail", From bb0b3f461a7a7809f7adf5821e59dc7e37c55317 Mon Sep 17 00:00:00 2001 From: David Jay Date: Wed, 22 Feb 2017 15:11:50 -0500 Subject: [PATCH 17/36] Moving polling interval to constant. --- client/coral-embed-stream/src/Stream.js | 3 ++- client/coral-framework/constants/comments.js | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/client/coral-embed-stream/src/Stream.js b/client/coral-embed-stream/src/Stream.js index e68a223ca..89a637ceb 100644 --- a/client/coral-embed-stream/src/Stream.js +++ b/client/coral-embed-stream/src/Stream.js @@ -1,5 +1,6 @@ import React, {PropTypes} from 'react'; import Comment from './Comment'; +import {NEW_COMMENT_COUNT_POLL_INTERVAL} from 'coral-framework/constants/comments'; class Stream extends React.Component { @@ -33,7 +34,7 @@ class Stream extends React.Component { asset_id: asset.id, limit: asset.comments.length, sort: 'REVERSE_CHRONOLOGICAL' - }), 5000), + }), NEW_COMMENT_COUNT_POLL_INTERVAL), }); } diff --git a/client/coral-framework/constants/comments.js b/client/coral-framework/constants/comments.js index 17ea2223e..28db9cdf9 100644 --- a/client/coral-framework/constants/comments.js +++ b/client/coral-framework/constants/comments.js @@ -1 +1,2 @@ export const ADDTL_COMMENTS_ON_LOAD_MORE = 10; +export const NEW_COMMENT_COUNT_POLL_INTERVAL = 20000; From 27bd1404b38181b15e7ea31528215a8841f566f8 Mon Sep 17 00:00:00 2001 From: David Jay Date: Wed, 22 Feb 2017 15:13:46 -0500 Subject: [PATCH 18/36] Adding es translation. --- client/coral-framework/translations.json | 3 +++ 1 file changed, 3 insertions(+) diff --git a/client/coral-framework/translations.json b/client/coral-framework/translations.json index 23405d2d8..c817925eb 100644 --- a/client/coral-framework/translations.json +++ b/client/coral-framework/translations.json @@ -42,6 +42,9 @@ "bannedAccountMsg": "Tu cuenta se encuentra suspendida. Esto significa que no puedes dar Like, Marcar o escribir commentarios. Por favor, contacta moderator@fakeurl for more information", "editNameMsg": "", "loadMore": "Ver más", + "newCount": "Ver {0} {1} más", + "comment": "commentario", + "comments": "commentarios", "error": { "emailNotVerified": "Dirección de correo electrónico {0} no verificada.", "email": "No es un email válido", From 7ae81fd17269e3954a233fcb28b3a9959f9b147b Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Thu, 23 Feb 2017 09:43:50 -0700 Subject: [PATCH 19/36] fixing mixed types --- client/coral-embed-stream/src/Embed.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index 487437203..5e979d243 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -101,8 +101,9 @@ class Embed extends Component { } // Find the created_at date of the first comment. If no comments exist, set the date to a week ago. - const firstCommentDate = asset.comments[0] ? asset.comments[0].created_at - : new Date(Date.now() - 604800000); + const firstCommentDate = asset.comments[0] + ? asset.comments[0].created_at + : new Date(Date.now() - 1000 * 60 * 60 * 24 * 7).toISOString(); return (
    From e3b9799d4a2125377679652723eeaf8b3679fe50 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Thu, 23 Feb 2017 09:58:31 -0700 Subject: [PATCH 20/36] nested ternaries hurt me --- .../coral-framework/graphql/queries/index.js | 49 +++++++++++-------- 1 file changed, 29 insertions(+), 20 deletions(-) diff --git a/client/coral-framework/graphql/queries/index.js b/client/coral-framework/graphql/queries/index.js index 9b8a55134..c993dfb6a 100644 --- a/client/coral-framework/graphql/queries/index.js +++ b/client/coral-framework/graphql/queries/index.js @@ -49,29 +49,38 @@ export const loadMore = (data) => ({limit, cursor, parent_id, asset_id, sort}, n asset_id, sort }, - updateQuery: (oldData, {fetchMoreResult:{data:{new_top_level_comments}}}) => + updateQuery: (oldData, {fetchMoreResult:{data:{new_top_level_comments}}}) => { - // If loading more replies - parent_id ? { - ...oldData, - asset: { - ...oldData.asset, - comments: oldData.asset.comments.map((comment) => - comment.id === parent_id - ? {...comment, replies: [...comment.replies, ...new_top_level_comments]} - : comment) - } + let updatedAsset; + + if (parent_id) { + + // If loading more replies + updatedAsset = { + ...oldData, + asset: { + ...oldData.asset, + comments: oldData.asset.comments.map((comment) => + comment.id === parent_id + ? {...comment, replies: [...comment.replies, ...new_top_level_comments]} + : comment) + } + }; + } else { + + // If loading more top-level comments + updatedAsset = { + ...oldData, + asset: { + ...oldData.asset, + comments: newComments ? [...new_top_level_comments.reverse(), ...oldData.asset.comments] + : [...oldData.asset.comments, ...new_top_level_comments] + } + }; } - // If loading more top-level comments - : { - ...oldData, - asset: { - ...oldData.asset, - comments: newComments ? [...new_top_level_comments.reverse(), ...oldData.asset.comments] - : [...oldData.asset.comments, ...new_top_level_comments] - } - } + return updatedAsset; + } }); }; From 5d0ea6a880c20e0655e2df080adbd3caefd24a22 Mon Sep 17 00:00:00 2001 From: gaba Date: Thu, 23 Feb 2017 09:48:59 -0800 Subject: [PATCH 21/36] Rmove continue button, fix bug with facebook signup. --- .../components/CreateUsernameDialog.js | 1 - client/coral-sign-in/components/styles.css | 7 ------ .../containers/ChangeUsernameContainer.js | 2 +- routes/api/auth/index.js | 23 ++++++++----------- services/users.js | 5 +++- 5 files changed, 14 insertions(+), 24 deletions(-) diff --git a/client/coral-sign-in/components/CreateUsernameDialog.js b/client/coral-sign-in/components/CreateUsernameDialog.js index f0edc7398..d0e348046 100644 --- a/client/coral-sign-in/components/CreateUsernameDialog.js +++ b/client/coral-sign-in/components/CreateUsernameDialog.js @@ -50,7 +50,6 @@ const CreateUsernameDialog = ({open, handleClose, offset, formData, handleSubmit />
    -
    diff --git a/client/coral-sign-in/components/styles.css b/client/coral-sign-in/components/styles.css index 81afe75e5..617c8d1cb 100644 --- a/client/coral-sign-in/components/styles.css +++ b/client/coral-sign-in/components/styles.css @@ -173,13 +173,6 @@ input.error{ color: white; } -.continuebutton { - background-color: black; - font-weight: bold; - color: white; - min-width: 200px; -} - .fakeComment { display: block; margin-bottom: 5px; diff --git a/client/coral-sign-in/containers/ChangeUsernameContainer.js b/client/coral-sign-in/containers/ChangeUsernameContainer.js index ca065169f..c4450aa56 100644 --- a/client/coral-sign-in/containers/ChangeUsernameContainer.js +++ b/client/coral-sign-in/containers/ChangeUsernameContainer.js @@ -104,7 +104,7 @@ class ChangeUsernameContainer extends Component { return (
    (err, user) => { /** * Returns the response to the login attempt via a popup callback with some JS. */ + const HandleAuthPopupCallback = (req, res, next) => (err, user) => { if (err) { return res.render('auth-callback', {err: JSON.stringify(err), data: null}); @@ -70,20 +70,15 @@ const HandleAuthPopupCallback = (req, res, next) => (err, user) => { return res.render('auth-callback', {err: JSON.stringify(errors.ErrNotAuthorized), data: null}); } - // Authorize the user to edit their username. - UsersService.toggleNameEdit(user.id, true) - .then(() => { - - // Perform the login of the user! - req.logIn(user, (err) => { - if (err) { - return res.render('auth-callback', {err: JSON.stringify(err), data: null}); - } + // Perform the login of the user! + req.logIn(user, (err) => { + if (err) { + return res.render('auth-callback', {err: JSON.stringify(err), data: null}); + } - // We logged in the user! Let's send back the user data. - res.render('auth-callback', {err: null, data: JSON.stringify(user)}); - }); - }); + // We logged in the user! Let's send back the user data. + res.render('auth-callback', {err: null, data: JSON.stringify(user), changeusername: true}); + }); }; /** diff --git a/services/users.js b/services/users.js index cf3a0063d..b58776a75 100644 --- a/services/users.js +++ b/services/users.js @@ -124,6 +124,8 @@ module.exports = class UsersService { return user; } + // User does not exist and need to be created. + let username = UsersService.castUsername(displayName); // The user was not found, lets create them! @@ -131,7 +133,8 @@ module.exports = class UsersService { username, lowercaseUsername: username.toLowerCase(), roles: [], - profiles: [{id, provider}] + profiles: [{id, provider}], + canEditName: true }); return user.save(); From da732ece14e6cb940db0227ac1abbbc33e62daa9 Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 15:03:09 -0500 Subject: [PATCH 22/36] Updating button padding and font size. --- client/coral-embed-stream/style/default.css | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 1deae6026..a99ec1bc7 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -7,7 +7,7 @@ body { font-family: 'Lato', sans-serif; font-family: 'Open Sans', sans-serif; width: 100%; - font-size: 12px; + font-size: 14px; margin: 0px; padding: 0px 0px 50px 0px; } @@ -18,9 +18,10 @@ body { button { padding: 5px 10px; - margin: 5px; + margin: 5px 10px 5px 0px; background: none; border: none; + font-size: inherit; } button:hover { @@ -208,6 +209,7 @@ hr { .coral-plugin-pubdate-text { color: #CCC; display: inline-block; + font-size: .75rem; } .coral-plugin-permalinks-container { From d73a6a87a4ed7592bda1dddc495a47db4142c3ff Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 15:19:38 -0500 Subject: [PATCH 23/36] Adding border radius to tag label. --- client/coral-embed-stream/style/default.css | 12 ++++++++++++ client/coral-plugin-tag-label/TagLabel.js | 4 +--- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index a99ec1bc7..315db2dd3 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -171,6 +171,18 @@ hr { float: right; } +/* Tag Labels */ + +.coral-plugin-tag-label { + background-color: #4C1066; + color: white; + display: inline-block; + margin: 10px 10px; + padding: 5px 5px; + border-radius: 2px; +} + + /* Reply styles */ diff --git a/client/coral-plugin-tag-label/TagLabel.js b/client/coral-plugin-tag-label/TagLabel.js index 0bceca225..ba7403f5c 100644 --- a/client/coral-plugin-tag-label/TagLabel.js +++ b/client/coral-plugin-tag-label/TagLabel.js @@ -1,8 +1,6 @@ import React from 'react'; -import styles from './styles.css'; - -const TagLabel = ({isStaff}) =>
    +const TagLabel = ({isStaff}) =>
    {isStaff ? 'Staff' : ''}
    ; From 471607403ca184f6bea4e4485d87f80704252a50 Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 15:26:02 -0500 Subject: [PATCH 24/36] Switching font. --- client/coral-embed-stream/style/default.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 315db2dd3..375dd0097 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -4,8 +4,8 @@ html, body { } body { - font-family: 'Lato', sans-serif; font-family: 'Open Sans', sans-serif; + font-family: 'Lato', sans-serif; width: 100%; font-size: 14px; margin: 0px; From 4cc3426af483983ae469ccf54b796ef88ad0e53b Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 16:07:41 -0500 Subject: [PATCH 25/36] Adding bolding, adding translations to settings page. --- client/coral-embed-stream/style/default.css | 9 ++++----- client/coral-plugin-author-name/AuthorName.js | 4 +--- client/coral-plugin-tag-label/styles.css | 7 ------- client/coral-settings/components/NotLoggedIn.js | 10 +++++----- client/coral-settings/translations.json | 10 ++++++++-- client/coral-ui/components/Tab.css | 1 + views/embed/stream.ejs | 2 +- 7 files changed, 20 insertions(+), 23 deletions(-) delete mode 100644 client/coral-plugin-tag-label/styles.css diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 375dd0097..35305f309 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -66,10 +66,9 @@ hr { background: rgb(35,118,216); color: white; width: 100%; - text-align: center; + text-align: left; padding: 10px; margin-bottom: 10px; - font-weight: bold; display: block; } @@ -163,8 +162,8 @@ hr { .coral-plugin-author-name-text { display: inline-block; - margin-right: 10px; - font-weight: bolder; + margin: 10px 8px 10px 0; + font-weight: bold; } .coral-plugin-author-name-bio-flag { @@ -219,7 +218,7 @@ hr { } .coral-plugin-pubdate-text { - color: #CCC; + color: #696969; display: inline-block; font-size: .75rem; } diff --git a/client/coral-plugin-author-name/AuthorName.js b/client/coral-plugin-author-name/AuthorName.js index bca1d716b..67f61bc33 100644 --- a/client/coral-plugin-author-name/AuthorName.js +++ b/client/coral-plugin-author-name/AuthorName.js @@ -1,6 +1,5 @@ import React, {Component} from 'react'; const packagename = 'coral-plugin-author-name'; -import styles from './styles.css'; export default class AuthorName extends Component { @@ -24,8 +23,7 @@ export default class AuthorName extends Component { const {author} = this.props; return (
    + className={`${packagename}-text`}> {author && author.name}
    ); diff --git a/client/coral-plugin-tag-label/styles.css b/client/coral-plugin-tag-label/styles.css deleted file mode 100644 index 03dc3dbc8..000000000 --- a/client/coral-plugin-tag-label/styles.css +++ /dev/null @@ -1,7 +0,0 @@ -.staff { - background-color: #4C1066; - color: white; - display: inline-block; - margin: 10px 10px; - padding: 5px 5px; -} diff --git a/client/coral-settings/components/NotLoggedIn.js b/client/coral-settings/components/NotLoggedIn.js index 095d43a8f..2e2b64181 100644 --- a/client/coral-settings/components/NotLoggedIn.js +++ b/client/coral-settings/components/NotLoggedIn.js @@ -1,6 +1,9 @@ import React from 'react'; import styles from './NotLoggedIn.css'; import SignInContainer from '../../coral-sign-in/containers/SignInContainer'; +import translations from '../translations'; +import I18n from 'coral-framework/modules/i18n/i18n'; +const lang = new I18n(translations); export default ({showSignInDialog}) => (
    @@ -9,13 +12,10 @@ export default ({showSignInDialog}) => ( { console.log('Signin click'); showSignInDialog(); - }}>Sign In to access Settings + }}>{lang.t('signIn')} {lang.t('toAccess')}
    - From the Settings Page you can -
      -
    • See your comment history
      • -
    + {lang.t('fromSettingsPage')}
    ); diff --git a/client/coral-settings/translations.json b/client/coral-settings/translations.json index 2a827a4f0..953708e61 100644 --- a/client/coral-settings/translations.json +++ b/client/coral-settings/translations.json @@ -3,12 +3,18 @@ "userNoComment": "This user has not yet left a comment.", "allComments": "All Comments", "profileSettings": "Profile Settings", - "myCommentHistory": "My comment History" + "myCommentHistory": "My comment History", + "signIn": "Sign in", + "toAccess": " to access Settings", + "fromSettingsPage": "From the Settings Page you can see your comment history." }, "es":{ "userNoComment": "Aún no ha escrito ningún comentario.", "allComments": "Todos los comentarios", "profileSettings": "Configuración del perfil", - "myCommentHistory": "Mi historial de comentarios" + "myCommentHistory": "Mi historial de comentarios", + "signIn": "Entrar", + "toAccess": "para acceder a la configuración", + "fromSettingsPage": "Desde la peagina de configuración puede ver su historia de comentarios." } } diff --git a/client/coral-ui/components/Tab.css b/client/coral-ui/components/Tab.css index dfd8d65f1..1d353c8fe 100644 --- a/client/coral-ui/components/Tab.css +++ b/client/coral-ui/components/Tab.css @@ -1,5 +1,6 @@ li.base--active { background: white; + font-weight: bold; } li.material--active { diff --git a/views/embed/stream.ejs b/views/embed/stream.ejs index 77b392b5f..fe7baf919 100644 --- a/views/embed/stream.ejs +++ b/views/embed/stream.ejs @@ -3,7 +3,7 @@ - + <% if (locals.customCssUrl) { %> From c1e3aec1816b4b463f18456cb34987c49024cc09 Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 16:20:20 -0500 Subject: [PATCH 26/36] Updating styles. --- client/coral-embed-stream/style/default.css | 7 +++++++ client/coral-settings/components/NotLoggedIn.css | 1 - client/coral-sign-in/components/styles.css | 1 + 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 35305f309..9d6723082 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -59,6 +59,13 @@ hr { font-weight: bold; } +/* Coral sign in button */ + +#coralSignInButton { + background-color: #2a2a2a; + color: #FFF; +} + /* Info Box Styles */ .coral-plugin-infobox-info { top: 0; diff --git a/client/coral-settings/components/NotLoggedIn.css b/client/coral-settings/components/NotLoggedIn.css index f65e96507..c15c44c98 100644 --- a/client/coral-settings/components/NotLoggedIn.css +++ b/client/coral-settings/components/NotLoggedIn.css @@ -11,5 +11,4 @@ cursor: pointer; margin: 0px; padding-bottom: 2px; - border-bottom: solid 1px black; } diff --git a/client/coral-sign-in/components/styles.css b/client/coral-sign-in/components/styles.css index 60a252a20..87e762ebf 100644 --- a/client/coral-sign-in/components/styles.css +++ b/client/coral-sign-in/components/styles.css @@ -36,6 +36,7 @@ .signInButton { margin-top: 10px; + background-color: #2a2a2a; } .close { From 022667c9aed1f778f2a07a1171116ba2e7fd2dd5 Mon Sep 17 00:00:00 2001 From: gaba Date: Thu, 23 Feb 2017 13:55:51 -0800 Subject: [PATCH 27/36] Brings some of the components into FakeComment --- .../coral-sign-in/components/FakeComment.js | 38 ++++++++++++------- client/coral-sign-in/translations.js | 12 +++++- 2 files changed, 36 insertions(+), 14 deletions(-) diff --git a/client/coral-sign-in/components/FakeComment.js b/client/coral-sign-in/components/FakeComment.js index 97afa59fa..75d4e51f1 100644 --- a/client/coral-sign-in/components/FakeComment.js +++ b/client/coral-sign-in/components/FakeComment.js @@ -1,13 +1,15 @@ import React from 'react'; import styles from 'coral-embed-stream/src/Comment.css'; -import PermalinkButton from 'coral-plugin-permalinks/PermalinkButton'; import AuthorName from 'coral-plugin-author-name/AuthorName'; import Content from 'coral-plugin-commentcontent/CommentContent'; import PubDate from 'coral-plugin-pubdate/PubDate'; import {ReplyButton} from 'coral-plugin-replies'; -import FlagComment from 'coral-plugin-flags/FlagComment'; -import LikeButton from 'coral-plugin-likes/LikeButton'; + +import I18n from 'coral-framework/modules/i18n/i18n'; +import translations from '../translations'; + +const lang = new I18n(translations); class FakeComment extends React.Component { constructor (props) { @@ -27,14 +29,13 @@ class FakeComment extends React.Component {
    - {return;}} - deleteAction={()=>{return;}} - showSignInDialog={()=>{return;}} - currentUser={{}} - /> +
    + +
    ; {}} parentCommentId={'commentID'} @@ -43,8 +44,19 @@ class FakeComment extends React.Component { />
    - - +
    + +
    +
    + +
    ); diff --git a/client/coral-sign-in/translations.js b/client/coral-sign-in/translations.js index 5693939af..7a946c0bb 100644 --- a/client/coral-sign-in/translations.js +++ b/client/coral-sign-in/translations.js @@ -42,7 +42,12 @@ export default { errorCreate: 'Error when changing username', checkTheForm: 'Invalid Form. Please, check the fields', specialCharacters: 'Usernames can contain letters, numbers and _ only' - } + }, + 'permalink': { + permalink: 'Link' + }, + 'report': 'Report', + 'like': 'Like', }, es: { 'signIn': { @@ -88,5 +93,10 @@ export default { checkTheForm: 'Formulario Invalido. Por favor, verifica los campos', specialCharacters: 'Sólo pueden contener letras, números y _' }, + 'permalink': { + permalink: 'Enlace' + }, + 'report': 'Informe', + 'like': 'Me gusta', } }; From e735fb800abfdbc81a513cd8e7c241e0a779a2e8 Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 17:02:35 -0500 Subject: [PATCH 28/36] Updating button and signin style. --- client/coral-embed-stream/style/default.css | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index 9d6723082..a4bec2400 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -17,17 +17,16 @@ body { } button { - padding: 5px 10px; margin: 5px 10px 5px 0px; background: none; + padding: 0px; border: none; font-size: inherit; } button:hover { border-radius: 2px; - color: #FFF; - background-color: rgb(155, 155, 155); + color: #767676; } button i { @@ -66,6 +65,10 @@ hr { color: #FFF; } +#coralSignInButton:hover { + background-color: #767676; +} + /* Info Box Styles */ .coral-plugin-infobox-info { top: 0; From 39536046c969b7de2ec3509e82ffefe5a5d7b09b Mon Sep 17 00:00:00 2001 From: David Jay Date: Thu, 23 Feb 2017 17:08:35 -0500 Subject: [PATCH 29/36] Fixing permalink icon and adding padding to comment content. --- client/coral-embed-stream/style/default.css | 2 +- client/coral-plugin-commentcontent/CommentContent.js | 2 +- client/coral-plugin-permalinks/PermalinkButton.js | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/client/coral-embed-stream/style/default.css b/client/coral-embed-stream/style/default.css index a4bec2400..61965519b 100644 --- a/client/coral-embed-stream/style/default.css +++ b/client/coral-embed-stream/style/default.css @@ -167,7 +167,7 @@ hr { } .coral-plugin-commentcontent-text { - margin-bottom: 10px; + margin-bottom: 7px; } .coral-plugin-author-name-text { diff --git a/client/coral-plugin-commentcontent/CommentContent.js b/client/coral-plugin-commentcontent/CommentContent.js index 2dab237dd..501005b3d 100644 --- a/client/coral-plugin-commentcontent/CommentContent.js +++ b/client/coral-plugin-commentcontent/CommentContent.js @@ -1,5 +1,5 @@ import React from 'react'; -const name = 'coral-plugin-content'; +const name = 'coral-plugin-commentcontent'; const Content = ({body, styles}) => { const textbreaks = body.split('\n'); diff --git a/client/coral-plugin-permalinks/PermalinkButton.js b/client/coral-plugin-permalinks/PermalinkButton.js index 1a99a5627..4c6cbb571 100644 --- a/client/coral-plugin-permalinks/PermalinkButton.js +++ b/client/coral-plugin-permalinks/PermalinkButton.js @@ -49,8 +49,8 @@ class PermalinkButton extends React.Component { return (
    Date: Fri, 24 Feb 2017 11:11:18 -0700 Subject: [PATCH 30/36] translations. remove console.log on frontend. remove notification on POST moderation --- .../containers/Configure/CommentSettings.js | 1 - .../ModerationQueue/ModerationContainer.js | 1 - client/coral-embed-stream/src/Embed.js | 9 +- client/coral-framework/actions/auth.js | 3 +- client/coral-framework/translations.json | 2 + client/coral-plugin-commentbox/CommentBox.js | 3 - .../RileysAwesomeCommentBox.js | 58 ---------- client/coral-plugin-stream/Stream.js | 100 ------------------ .../coral-settings/components/NotLoggedIn.js | 1 - .../{SettingsHeader.css => ProfileHeader.css} | 0 .../components/ProfileHeader.js | 12 +++ .../components/SettingsHeader.js | 14 --- ...ttingsContainer.js => ProfileContainer.js} | 8 +- client/coral-settings/translations.json | 8 +- client/coral-sign-in/translations.js | 4 +- 15 files changed, 32 insertions(+), 192 deletions(-) delete mode 100644 client/coral-plugin-stream/RileysAwesomeCommentBox.js delete mode 100644 client/coral-plugin-stream/Stream.js rename client/coral-settings/components/{SettingsHeader.css => ProfileHeader.css} (100%) create mode 100644 client/coral-settings/components/ProfileHeader.js delete mode 100644 client/coral-settings/components/SettingsHeader.js rename client/coral-settings/containers/{SettingsContainer.js => ProfileContainer.js} (92%) diff --git a/client/coral-admin/src/containers/Configure/CommentSettings.js b/client/coral-admin/src/containers/Configure/CommentSettings.js index 5a14cf565..90057c5fe 100644 --- a/client/coral-admin/src/containers/Configure/CommentSettings.js +++ b/client/coral-admin/src/containers/Configure/CommentSettings.js @@ -52,7 +52,6 @@ const updateClosedMessage = (updateSettings) => (event) => { }; const updateCustomCssUrl = (updateSettings) => (event) => { - console.log('updateCustomCssUrl', event.target.value); const customCssUrl = event.target.value; updateSettings({customCssUrl}); }; diff --git a/client/coral-admin/src/containers/ModerationQueue/ModerationContainer.js b/client/coral-admin/src/containers/ModerationQueue/ModerationContainer.js index f685ac094..b1a44533e 100644 --- a/client/coral-admin/src/containers/ModerationQueue/ModerationContainer.js +++ b/client/coral-admin/src/containers/ModerationQueue/ModerationContainer.js @@ -54,7 +54,6 @@ class ModerationContainer extends Component { } if (data.error) { - console.log(data); return
    Error
    ; } diff --git a/client/coral-embed-stream/src/Embed.js b/client/coral-embed-stream/src/Embed.js index 5e979d243..5c72e65d6 100644 --- a/client/coral-embed-stream/src/Embed.js +++ b/client/coral-embed-stream/src/Embed.js @@ -2,6 +2,9 @@ import React, {Component} from 'react'; import {compose} from 'react-apollo'; import {connect} from 'react-redux'; import isEqual from 'lodash/isEqual'; +import I18n from 'coral-framework/modules/i18n/i18n'; +import translations from 'coral-framework/translations'; +const lang = new I18n(translations); import {TabBar, Tab, TabContent, Spinner} from 'coral-ui'; @@ -25,7 +28,7 @@ import UserBox from 'coral-sign-in/components/UserBox'; import SignInContainer from 'coral-sign-in/containers/SignInContainer'; import SuspendedAccount from 'coral-framework/components/SuspendedAccount'; import ChangeUsernameContainer from '../../coral-sign-in/containers/ChangeUsernameContainer'; -import SettingsContainer from 'coral-settings/containers/SettingsContainer'; +import ProfileContainer from 'coral-settings/containers/ProfileContainer'; import RestrictedContent from 'coral-framework/components/RestrictedContent'; import ConfigureStreamContainer from 'coral-configure/containers/ConfigureStreamContainer'; import LoadMore from './LoadMore'; @@ -110,7 +113,7 @@ class Embed extends Component {
    - Settings + {lang.t('profile')} Configure Stream {loggedIn && } @@ -190,7 +193,7 @@ class Embed extends Component { loadMore={this.props.loadMore}/> - dispatch => { dispatch(verifyEmailSuccess()); }) .catch(err => { - console.log('failed to send email verification', err); // email might have already been verifyed - dispatch(verifyEmailFailure()); + dispatch(verifyEmailFailure(err)); }); }; diff --git a/client/coral-framework/translations.json b/client/coral-framework/translations.json index c817925eb..d115aedca 100644 --- a/client/coral-framework/translations.json +++ b/client/coral-framework/translations.json @@ -1,5 +1,6 @@ { "en": { + "profile": "Profile", "successUpdateSettings": "The changes you have made have been applied to the comment stream on this article", "successNameUpdate": "Your username has been updated", "contentNotAvailable": "This content is not available", @@ -36,6 +37,7 @@ } }, "es": { + "profile": "Perfil", "successUpdateSettings": "La configuración de este articulo fue actualizada", "successBioUpdate": "Tu bio fue actualizada", "contentNotAvailable": "El contenido no se encuentra disponible", diff --git a/client/coral-plugin-commentbox/CommentBox.js b/client/coral-plugin-commentbox/CommentBox.js index 3d895df07..e1b04582c 100644 --- a/client/coral-plugin-commentbox/CommentBox.js +++ b/client/coral-plugin-commentbox/CommentBox.js @@ -57,8 +57,6 @@ class CommentBox extends Component { } else if (postedComment.status === 'PREMOD') { addNotification('success', lang.t('comment-post-notif-premod')); updateCountCache(assetId, countCache); - } else { - addNotification('success', 'Your comment has been posted.'); } if (commentPostedHandler) { @@ -110,7 +108,6 @@ class CommentBox extends Component { cStyle='darkGrey' className={`${name}-cancel-button`} onClick={() => { - console.log('cancel button in comment box'); cancelButtonClicked(''); }}> {lang.t('cancel')} diff --git a/client/coral-plugin-stream/RileysAwesomeCommentBox.js b/client/coral-plugin-stream/RileysAwesomeCommentBox.js deleted file mode 100644 index 73fa6407d..000000000 --- a/client/coral-plugin-stream/RileysAwesomeCommentBox.js +++ /dev/null @@ -1,58 +0,0 @@ -import React, {Component} from 'react'; -import {graphql} from 'react-apollo'; -import gql from 'graphql-tag'; - -export class RileysAwesomeCommentBox extends Component { - - postComment() { - console.log(this.props); - console.log('postComment', this.props.asset_id); - this.props.mutate({ - variables: { - asset_id: this.props.asset_id, - body: this.textarea.value, - parent_id: null - } - }).then(({data}) => { - console.log('it workt'); - console.log(data); - }); - } - - render() { - return
    - - -
    ; - } -} - -const postComment = gql` - fragment commentView on Comment { - id - body - user { - name: username - } - actions { - type: action_type - count - current: current_user { - id - created_at - } - } - } - - mutation CreateComment ($asset_id: ID!, $parent_id: ID, $body: String!) { - createComment(asset_id:$asset_id, parent_id:$parent_id, body:$body) { - ...commentView - } - } -`; - -const RileysAwesomeCommentBoxWithData = graphql( - postComment -)(RileysAwesomeCommentBox); - -export default RileysAwesomeCommentBoxWithData; diff --git a/client/coral-plugin-stream/Stream.js b/client/coral-plugin-stream/Stream.js deleted file mode 100644 index 72e03d4ae..000000000 --- a/client/coral-plugin-stream/Stream.js +++ /dev/null @@ -1,100 +0,0 @@ -import React, {Component} from 'react'; -import {graphql} from 'react-apollo'; -import gql from 'graphql-tag'; -import {fetchSignIn} from 'coral-framework/actions/auth'; -import RileysAwesomeCommentBox from 'coral-plugin-stream/RileysAwesomeCommentBox'; - -const assetID = '6187a94b-0b6d-4a96-ac6b-62b529cd8410'; - -// MyComponent is a "presentational" or apollo-unaware component, -// It could be a simple React class: -class Stream extends Component { - - constructor(props) { - super(props); - } - - logMeIn() { - fetchSignIn({email: 'your@example.com', password: 'dfasidfaisdufoiausdfoiuaspdoifas'})(() => {}); - } - - render() { - const {data} = this.props; - return
    - - { - data.loading - ? 'loading!' - :
    - -

    Asset ID: {data.asset.id}

    -
      - { - data.asset.comments.map(comment => { - return
    • - {comment.body} [{comment.id}] -
        - { - comment.replies.map(reply => { - return
      • {reply.body}
        • ; - }) - } -
      -
      • ; - }) - } -
    -
    - } -
    ; - } -} - -// Initialize GraphQL queries or mutations with the gql tag -const StreamQuery = gql`fragment commentView on Comment { - id - body - user { - name: username - } - tags { - name - } - actions { - type: action_type - count - current: current_user { - id - created_at - } - } -} - -query AssetQuery($asset_id: ID!) { - asset(id: $asset_id) { - id - title - url - commentCount - comments { - ...commentView - replies { - ...commentView - } - } - } -}`; - -// We then can use `graphql` to pass the query results returned by MyQuery -// to MyComponent as a prop (and update them as the results change) -const StreamWithData = graphql( - StreamQuery, { - options: { - variables: { - asset_id: assetID - } - } - } -)(Stream); - -export default StreamWithData; diff --git a/client/coral-settings/components/NotLoggedIn.js b/client/coral-settings/components/NotLoggedIn.js index 2e2b64181..acd3ae7e6 100644 --- a/client/coral-settings/components/NotLoggedIn.js +++ b/client/coral-settings/components/NotLoggedIn.js @@ -10,7 +10,6 @@ export default ({showSignInDialog}) => ( diff --git a/client/coral-settings/components/SettingsHeader.css b/client/coral-settings/components/ProfileHeader.css similarity index 100% rename from client/coral-settings/components/SettingsHeader.css rename to client/coral-settings/components/ProfileHeader.css diff --git a/client/coral-settings/components/ProfileHeader.js b/client/coral-settings/components/ProfileHeader.js new file mode 100644 index 000000000..24b2222bd --- /dev/null +++ b/client/coral-settings/components/ProfileHeader.js @@ -0,0 +1,12 @@ +import React, {PropTypes} from 'react'; +import styles from './ProfileHeader.css'; + +const ProfileHeader = ({username}) => ( +
    +

    {username}

    +
    +); + +ProfileHeader.propTypes = {username: PropTypes.string.isRequired}; + +export default ProfileHeader; diff --git a/client/coral-settings/components/SettingsHeader.js b/client/coral-settings/components/SettingsHeader.js deleted file mode 100644 index 34d347314..000000000 --- a/client/coral-settings/components/SettingsHeader.js +++ /dev/null @@ -1,14 +0,0 @@ -import React from 'react'; -import styles from './SettingsHeader.css'; - -export default ({userData}) => ( -
    -

    {userData.username}

    - - { - - // Hiding display of users ID unless there's a use case for it. - //

    {userData.profiles.map(profile => profile.id)}

    - } -
    -); diff --git a/client/coral-settings/containers/SettingsContainer.js b/client/coral-settings/containers/ProfileContainer.js similarity index 92% rename from client/coral-settings/containers/SettingsContainer.js rename to client/coral-settings/containers/ProfileContainer.js index 9e72cdadc..4a98ed8ff 100644 --- a/client/coral-settings/containers/SettingsContainer.js +++ b/client/coral-settings/containers/ProfileContainer.js @@ -8,13 +8,13 @@ import {myCommentHistory} from 'coral-framework/graphql/queries'; import {link} from 'coral-framework/services/PymConnection'; import NotLoggedIn from '../components/NotLoggedIn'; import {Spinner} from 'coral-ui'; -import SettingsHeader from '../components/SettingsHeader'; +import ProfileHeader from '../components/ProfileHeader'; import CommentHistory from 'coral-plugin-history/CommentHistory'; import translations from '../translations'; const lang = new I18n(translations); -class SettingsContainer extends Component { +class ProfileContainer extends Component { constructor (props) { super(props); this.state = { @@ -44,7 +44,7 @@ class SettingsContainer extends Component { return (
    - + { // Hiding bio until moderation can get figured out @@ -88,4 +88,4 @@ const mapDispatchToProps = () => ({ export default compose( connect(mapStateToProps, mapDispatchToProps), myCommentHistory -)(SettingsContainer); +)(ProfileContainer); diff --git a/client/coral-settings/translations.json b/client/coral-settings/translations.json index c8398bd7b..945d71b87 100644 --- a/client/coral-settings/translations.json +++ b/client/coral-settings/translations.json @@ -1,20 +1,22 @@ { "en":{ + "profile": "Profile", "userNoComment": "You've never left a comment. Join the conversation!", "allComments": "All Comments", "profileSettings": "Profile Settings", "myCommentHistory": "My comment History", "signIn": "Sign in", - "toAccess": " to access Settings", - "fromSettingsPage": "From the Settings Page you can see your comment history." + "toAccess": " to access Profile", + "fromSettingsPage": "From the Profile Page you can see your comment history." }, "es":{ + "profile": "Perfil", "userNoComment": "No has dejado áun ningún comentario. ¡Unete a la conversación!", "allComments": "Todos los comentarios", "profileSettings": "Configuración del perfil", "myCommentHistory": "Mi historial de comentarios", "signIn": "Entrar", - "toAccess": "para acceder a la configuración", + "toAccess": "para acceder a al perfil", "fromSettingsPage": "Desde la peagina de configuración puede ver su historia de comentarios." } } diff --git a/client/coral-sign-in/translations.js b/client/coral-sign-in/translations.js index a2bb60819..08c8bbd3f 100644 --- a/client/coral-sign-in/translations.js +++ b/client/coral-sign-in/translations.js @@ -10,7 +10,7 @@ export default { facebookSignIn: 'Sign in with Facebook', facebookSignUp: 'Sign up with Facebook', logout: 'Logout', - signIn: 'Sign In', + signIn: 'Sign in to join the conversation', or: 'Or', email: 'E-mail Address', password: 'Password', @@ -51,7 +51,7 @@ export default { facebookSignIn: 'Entrar con Facebook', facebookSignUp: 'Regístrate con Facebook', logout: 'Salir', - signIn: 'Entrar', + signIn: 'Entrar para Unirte a la Conversación', or: 'o', email: 'E-mail', password: 'Contraseña', From 6566505fc1ccbb0ec256381338a4446f351cee99 Mon Sep 17 00:00:00 2001 From: Wyatt Johnson Date: Fri, 24 Feb 2017 11:15:46 -0700 Subject: [PATCH 31/36] Adjusted route --- app.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app.json b/app.json index e498c5168..5faa23bc2 100644 --- a/app.json +++ b/app.json @@ -29,5 +29,5 @@ "as": "POSTMARK" }], "image": "heroku/nodejs", - "success_url": "/admin/setup" + "success_url": "/admin/install" } From 5c441e90e7cf932de9103b13399ea8b65e36a6e5 Mon Sep 17 00:00:00 2001 From: Riley Davis Date: Fri, 24 Feb 2017 13:08:48 -0700 Subject: [PATCH 32/36] do not update countCache if the CommentBox is in a reply --- client/coral-plugin-commentbox/CommentBox.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/client/coral-plugin-commentbox/CommentBox.js b/client/coral-plugin-commentbox/CommentBox.js index 3d895df07..f7444437e 100644 --- a/client/coral-plugin-commentbox/CommentBox.js +++ b/client/coral-plugin-commentbox/CommentBox.js @@ -30,6 +30,7 @@ class CommentBox extends Component { postItem, assetId, updateCountCache, + isReply, countCache, parentId, addNotification, @@ -46,17 +47,17 @@ class CommentBox extends Component { if (this.props.charCount && this.state.body.length > this.props.charCount) { return; } - updateCountCache(assetId, countCache + 1); + !isReply && updateCountCache(assetId, countCache + 1); postItem(comment, 'comments') .then(({data}) => { const postedComment = data.createComment.comment; if (postedComment.status === 'REJECTED') { addNotification('error', lang.t('comment-post-banned-word')); - updateCountCache(assetId, countCache); + !isReply && updateCountCache(assetId, countCache); } else if (postedComment.status === 'PREMOD') { addNotification('success', lang.t('comment-post-notif-premod')); - updateCountCache(assetId, countCache); + !isReply && updateCountCache(assetId, countCache); } else { addNotification('success', 'Your comment has been posted.'); } From d650779c1a3b83ea035ff9b0705874f15cf96996 Mon Sep 17 00:00:00 2001 From: gaba Date: Fri, 24 Feb 2017 16:08:25 -0800 Subject: [PATCH 33/36] Removes ; --- client/coral-sign-in/components/FakeComment.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/coral-sign-in/components/FakeComment.js b/client/coral-sign-in/components/FakeComment.js index 75d4e51f1..b5af6bf1c 100644 --- a/client/coral-sign-in/components/FakeComment.js +++ b/client/coral-sign-in/components/FakeComment.js @@ -35,7 +35,7 @@ class FakeComment extends React.Component { thumb_up -
    ; +
    {}} parentCommentId={'commentID'} From a429af3c0e9da6b9c7ab981ca8dabeab8fa969d2 Mon Sep 17 00:00:00 2001 From: gaba Date: Fri, 24 Feb 2017 16:09:22 -0800 Subject: [PATCH 34/36] Removes varialbe not used. --- routes/api/auth/index.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routes/api/auth/index.js b/routes/api/auth/index.js index 8bd7a1263..fc3b87e51 100644 --- a/routes/api/auth/index.js +++ b/routes/api/auth/index.js @@ -77,7 +77,7 @@ const HandleAuthPopupCallback = (req, res, next) => (err, user) => { } // We logged in the user! Let's send back the user data. - res.render('auth-callback', {err: null, data: JSON.stringify(user), changeusername: true}); + res.render('auth-callback', {err: null, data: JSON.stringify(user)}); }); }; From f85a6d4da172f9cb2d192b3e6cac72773b536862 Mon Sep 17 00:00:00 2001 From: Belen Curcio Date: Mon, 27 Feb 2017 11:49:53 -0300 Subject: [PATCH 35/36] Drawer --- .../coral-admin/src/components/ui/Drawer.css | 0 .../coral-admin/src/components/ui/Drawer.js | 49 +++++++++++++++---- .../coral-admin/src/components/ui/Header.css | 18 +++++++ .../coral-admin/src/components/ui/Header.js | 2 +- client/coral-admin/src/components/ui/Logo.js | 4 +- client/coral-admin/src/index.js | 3 ++ 6 files changed, 64 insertions(+), 12 deletions(-) create mode 100644 client/coral-admin/src/components/ui/Drawer.css diff --git a/client/coral-admin/src/components/ui/Drawer.css b/client/coral-admin/src/components/ui/Drawer.css new file mode 100644 index 000000000..e69de29bb diff --git a/client/coral-admin/src/components/ui/Drawer.js b/client/coral-admin/src/components/ui/Drawer.js index 2003ed34d..8fdf4f19d 100644 --- a/client/coral-admin/src/components/ui/Drawer.js +++ b/client/coral-admin/src/components/ui/Drawer.js @@ -1,18 +1,49 @@ import React from 'react'; import {Navigation, Drawer} from 'react-mdl'; -import {Link} from 'react-router'; -import styles from './Header.css'; +import {IndexLink, Link} from 'react-router'; +import styles from './Drawer.css'; import I18n from 'coral-framework/modules/i18n/i18n'; import translations from '../../translations.json'; -export default () => ( - - - {lang.t('configure.moderate')} - {lang.t('configure.community')} - {lang.t('configure.configure')} - +export default ({handleLogout, restricted = false}) => ( + + { !restricted ? +
    + + + {lang.t('configure.dashboard')} + + + {lang.t('configure.moderate')} + + + {lang.t('configure.streams')} + + + {lang.t('configure.community')} + + + {lang.t('configure.configure')} + + Sign Out + {`v${process.env.VERSION}`} + +
    : null }     ); const lang = new I18n(translations); + diff --git a/client/coral-admin/src/components/ui/Header.css b/client/coral-admin/src/components/ui/Header.css index e6cf85d23..e30cb46d7 100644 --- a/client/coral-admin/src/components/ui/Header.css +++ b/client/coral-admin/src/components/ui/Header.css @@ -1,7 +1,25 @@ +@custom-media --table-viewport (max-width: 1024px); + +@media (--table-viewport) { + .logo { + margin-left: 58px; + h1 { + background: #696969; + } + span { + color: #fcfcfc; + } + } + .nav { + display: none; + } +} + .header { background-color: transparent; box-shadow: none; min-height: 58px; + display: block; } .header > div { diff --git a/client/coral-admin/src/components/ui/Header.js b/client/coral-admin/src/components/ui/Header.js index 141d7fecf..556b33b6c 100644 --- a/client/coral-admin/src/components/ui/Header.js +++ b/client/coral-admin/src/components/ui/Header.js @@ -8,7 +8,7 @@ import {Logo} from './Logo'; export default ({handleLogout, restricted = false}) => (
    - + { !restricted ?
    diff --git a/client/coral-admin/src/components/ui/Logo.js b/client/coral-admin/src/components/ui/Logo.js index 05a7707d7..8c57cde11 100644 --- a/client/coral-admin/src/components/ui/Logo.js +++ b/client/coral-admin/src/components/ui/Logo.js @@ -2,8 +2,8 @@ import React from 'react'; import styles from './Logo.css'; import {CoralLogo} from 'coral-ui'; -export const Logo = () => ( -
    +export const Logo = ({className = ''}) => ( +

    Talk diff --git a/client/coral-admin/src/index.js b/client/coral-admin/src/index.js index 306461dd0..00190847a 100644 --- a/client/coral-admin/src/index.js +++ b/client/coral-admin/src/index.js @@ -7,6 +7,9 @@ import store from './services/store'; import App from './components/App'; +import 'react-mdl/extra/material.css'; +import 'react-mdl/extra/material.js'; + render( From e8a92b768c4851d1efd79b8cd9828dcb99a14551 Mon Sep 17 00:00:00 2001 From: Belen Curcio Date: Mon, 27 Feb 2017 11:52:21 -0300 Subject: [PATCH 36/36] logout and version in drawer --- client/coral-admin/src/components/ui/Layout.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/client/coral-admin/src/components/ui/Layout.js b/client/coral-admin/src/components/ui/Layout.js index 46c7aa7fa..f28e8dd80 100644 --- a/client/coral-admin/src/components/ui/Layout.js +++ b/client/coral-admin/src/components/ui/Layout.js @@ -6,8 +6,8 @@ import styles from './Layout.css'; export const Layout = ({children, ...props}) => ( -
    - +
    +
    {children}