From 34adf49df9f2903f85b302b10d92bf4e12d4c4a4 Mon Sep 17 00:00:00 2001 From: James Date: Sat, 16 Feb 2019 14:37:55 -0500 Subject: [PATCH 1/7] misc leftover locales --- src/components/modules/Localizer/index.js | 2 +- src/components/utilities/SetLocale/index.js | 2 +- src/middleware/locale.js | 11 ++++++----- 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/components/modules/Localizer/index.js b/src/components/modules/Localizer/index.js index 623dc4249d..b9adfe4506 100644 --- a/src/components/modules/Localizer/index.js +++ b/src/components/modules/Localizer/index.js @@ -42,7 +42,7 @@ class Localizer extends Component { const newParams = { ...this.props.searchParams, - locale: locale + locale }; const search = qs.stringify(newParams); diff --git a/src/components/utilities/SetLocale/index.js b/src/components/utilities/SetLocale/index.js index d854ae78f6..039f52e7d2 100644 --- a/src/components/utilities/SetLocale/index.js +++ b/src/components/utilities/SetLocale/index.js @@ -24,7 +24,7 @@ class SetLocale extends Component { const { searchParams, config, setLocale } = this.props; if (searchParams) { - if (searchParams.locales && config.localization.locales.indexOf(searchParams.locales) > -1) { + if (searchParams.locale && config.localization.locales.indexOf(searchParams.locale) > -1) { setLocale(searchParams.locale); } else if (!this.state.init) { setLocale(config.localization.defaultLocale); diff --git a/src/middleware/locale.js b/src/middleware/locale.js index 044028161c..f7560003ad 100644 --- a/src/middleware/locale.js +++ b/src/middleware/locale.js @@ -6,27 +6,28 @@ import languageParser from 'accept-language-parser'; * @param localization * @returns {Function} */ + export default function locale(localization) { return function (req, res, next) { let setLocale; if (req.query.locale) { - setLocale = localization.languages.find(search => search === req.query.locale); + setLocale = localization.locales.find(search => search === req.query.locale); if (setLocale) { req.locale = setLocale; } if (req.query.locale === '*' || req.query.locale === 'all') return next(); } - if (req.body.locale){ - setLocale = localization.languages.find(search => search === req.body.locale); + if (req.body.locale) { + setLocale = localization.locales.find(search => search === req.body.locale); if (setLocale) { req.locale = setLocale; } } if (!req.locale && req.headers['accept-language']) - req.locale = languageParser.pick(localization.languages, req.headers['accept-language']); + req.locale = languageParser.pick(localization.locales, req.headers['accept-language']); if (!req.locale) - req.locale = localization.defaultLanguage; + req.locale = localization.defaultLocale; next(); } From a90b1ae5b50d810d9ee076a81e822a78418ff96c Mon Sep 17 00:00:00 2001 From: James Date: Sat, 16 Feb 2019 14:46:44 -0500 Subject: [PATCH 2/7] further ux refinements to localizer --- src/components/modules/Localizer/index.js | 2 +- src/components/modules/Localizer/index.scss | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/components/modules/Localizer/index.js b/src/components/modules/Localizer/index.js index b9adfe4506..775ea04426 100644 --- a/src/components/modules/Localizer/index.js +++ b/src/components/modules/Localizer/index.js @@ -49,7 +49,7 @@ class Localizer extends Component { return (
  • - + {locale}
  • diff --git a/src/components/modules/Localizer/index.scss b/src/components/modules/Localizer/index.scss index 53765bf759..0094f0af74 100644 --- a/src/components/modules/Localizer/index.scss +++ b/src/components/modules/Localizer/index.scss @@ -21,6 +21,14 @@ height: rem(.25); margin-right: rem(.375); } + + &:hover { + color: $primary; + + svg { + @include color-svg($primary); + } + } } ul { From 103d22e07cb8a3147ae8b14c211f2d9829ff4be4 Mon Sep 17 00:00:00 2001 From: James Date: Sat, 16 Feb 2019 15:24:07 -0500 Subject: [PATCH 3/7] removes dynamic collection routes, uses route param instead --- demo/Page/components/Edit/index.js | 137 ++++++++++------------- demo/client/index.js | 8 +- demo/client/scss/overrides.scss | 1 + src/components/data/edit.js | 6 +- src/components/layout/Sidebar/index.js | 6 +- src/components/modules/APIUrl/index.js | 20 +++- src/components/modules/Status/index.scss | 5 +- src/components/routes/Collections.js | 42 +++---- 8 files changed, 112 insertions(+), 113 deletions(-) diff --git a/demo/Page/components/Edit/index.js b/demo/Page/components/Edit/index.js index ceb203019b..bee2cd52ae 100644 --- a/demo/Page/components/Edit/index.js +++ b/demo/Page/components/Edit/index.js @@ -1,4 +1,4 @@ -import React, { Component } from 'react'; +import React from 'react'; import { withEditData, EditView, @@ -13,99 +13,78 @@ import { FormSubmit, Repeater } from 'payload/components'; -import { toKebabCase } from 'payload/utils'; -class Edit extends Component { +const Edit = props => { - constructor(props) { - super(props); + const { id } = props.match.params; - const entrySlug = this.props.match.params.slug; + const { data, collection, config, locale } = props; - this.state = { - uid: entrySlug, - slug: entrySlug, - action: `${this.props.config.serverUrl}/${this.props.collection.slug}${entrySlug ? `/${entrySlug}` : ''}`, - method: entrySlug ? 'put' : 'post' + const sampleRepeaterValue = [ + { + content: 'here\'s some test content' + }, { + content: 'here\'s some more test content' } - } + ]; - setSlug = e => { - this.setState({ slug: toKebabCase(e.target.value) }); - } + return ( + +
    + + } action={ + + + Save + + } /> - render() { + - const initialData = this.props.data ? this.props.data : {}; + - const sampleRepeaterValue = [ - { - content: 'here\'s some test content' - }, { - content: 'here\'s some more test content' - } - ]; + - return ( - - - - } action={ - - - Save - - } /> +