Merge branch 'minor-fixes' into 'master'

Minor fixes

See merge request sahkoinsinoorikilta/vtmk/web2.0-frontend!62

src/components/Footer/FooterContent.tsx

@@ -86,6 +86,8 @@ const FooterContent: React.FC = () => (
             <Link to="https://api.sahkoinsinoorikilta.fi/members/application/">Jäseneksi</Link>
             <Link to="mailto:hallitus@sahkoinsinoorikilta.fi">Palaute</Link>
             <Link to="https://static.sahkoinsinoorikilta.fi">Dokumenttiarkisto</Link>
+            <Link to="https://sik.kuvat.fi">Kuvagalleria</Link>
+            <Link to="https://static.sahkoinsinoorikilta.fi/logot-ja-grafiikka/">Logot ja grafiikka</Link>
           </div>
         </Columns>
       </MarginSpace>

src/components/Navigation.tsx

@@ -6,20 +6,21 @@ import Icon, { IconType } from "./Icon";
 import NavbarDropdownLink from "./NavbarDropdownLink";
 import NavbarChildLink from "./NavbarChildLink";
 
-const renderNavigationDesktopItems = () => (
+export const renderNavigationItems = (mobile = false) => (
   <>
-    <NavbarDropdownLink to="/kilta" text="Kilta ›">
+    <NavbarDropdownLink to="/kilta" text="Kilta ›" exploded={mobile}>
       <NavbarChildLink to="/kilta/toiminta">Toiminta</NavbarChildLink>
       <NavbarChildLink to="/kilta/fuksi">Fuksi</NavbarChildLink>
-      <NavbarChildLink to="/kilta/kunnia">Kunnianosoitukset</NavbarChildLink>
+      <NavbarChildLink to="/kilta/kunnianosoitukset">Kunnianosoitukset</NavbarChildLink>
       <NavbarChildLink to="https://static.sahkoinsinoorikilta.fi">Dokumenttiarkisto</NavbarChildLink>
+      <NavbarChildLink to="https://sik.kuvat.fi">Kuvagalleria</NavbarChildLink>
     </NavbarDropdownLink>
-    <NavbarDropdownLink to="/opinnot_ja_ura" text="Opinnot ja ura" />
-    <NavbarDropdownLink to="/yritysyhteistyo" text="Yritysyhteistyö" />
-    <NavbarDropdownLink to="/yhteystiedot" text="Yhteystiedot">
+    <NavbarDropdownLink to="/opinnot_ja_ura" text="Opinnot ja ura" exploded={mobile} />
+    <NavbarDropdownLink to="/yritysyhteistyo" text="Yritysyhteistyö" exploded={mobile} />
+    <NavbarDropdownLink to="/yhteystiedot" text="Yhteystiedot" exploded={mobile}>
       {/* <NavbarChildLink to="https://en.wikipedia.org/wiki/Gay">Simo Höglund</NavbarChildLink> */}
     </NavbarDropdownLink>
-    <NavbarDropdownLink to="/in_english" text="In English" />
+    <NavbarDropdownLink to="/in_english" text="In English" exploded={mobile} />
   </>
 );
 
@@ -104,7 +105,7 @@ interface NavigationProps {
 }
 
 const Navigation: React.FC<NavigationProps> = ({ onMobileMenuOpen }) => {
-  const desktopItems = renderNavigationDesktopItems();
+  const desktopItems = renderNavigationItems();
   return (
     <Nav>
       <DesktopContainer>

src/components/NavigationMobile.tsx

@@ -1,25 +1,7 @@
 import React from "react";
 import styled from "styled-components";
 import { colors } from "@theme/colors";
-import NavbarDropdownLink from "./NavbarDropdownLink";
-import NavbarChildLink from "./NavbarChildLink";
-
-const renderNavigationMobileItems = () => (
-  <>
-    <NavbarDropdownLink to="/kilta" text="Kilta ›" exploded>
-      <NavbarChildLink to="/kilta/toiminta">Toiminta</NavbarChildLink>
-      <NavbarChildLink to="/kilta/fuksi">Fuksi</NavbarChildLink>
-      <NavbarChildLink to="/kilta/kunnia">Kunnianosoitukset</NavbarChildLink>
-      <NavbarChildLink to="https://static.sahkoinsinoorikilta.fi">Dokumenttiarkisto</NavbarChildLink>
-    </NavbarDropdownLink>
-    <NavbarDropdownLink to="/opinnot_ja_ura" text="Opinnot ja ura" exploded />
-    <NavbarDropdownLink to="/yritysyhteistyo" text="Yritysyhteistyö" exploded />
-    <NavbarDropdownLink to="/yhteystiedot" text="Yhteystiedot" exploded>
-      {/* <NavbarChildLink to="https://en.wikipedia.org/wiki/Gay">Simo Höglund</NavbarChildLink> */}
-    </NavbarDropdownLink>
-    <NavbarDropdownLink to="/in_english" text="In English" exploded />
-  </>
-);
+import { renderNavigationItems } from "./Navigation";
 
 const Nav = styled.nav`
   padding: 1rem 2rem;
@@ -37,7 +19,7 @@ interface NavigationMobileProps {
 
 const NavigationMobile: React.FC<NavigationMobileProps> = ({ mobileMenuOpen }) => (
   <Nav hidden={!mobileMenuOpen}>
-    {renderNavigationMobileItems()}
+    {renderNavigationItems(true)}
   </Nav>
 );
 

src/pages/_app.tsx

@@ -5,6 +5,7 @@ import Head from "next/head";
 import styled, { createGlobalStyle } from "styled-components";
 import { ToastContainer } from "react-toastify";
 import { colors } from "@theme/colors";
+import breakpoints from "@theme/breakpoints";
 import LocaleStore from "../i18n";
 
 import "react-mde/lib/styles/css/react-mde-all.css";
@@ -25,17 +26,17 @@ html {
   font-size: ${fontSize * 0.75}pt;
   line-height: ${lineHeight};
 
-  @media screen and (min-width: 1200px) {
+  @media screen and (min-width: ${breakpoints.medium}) {
     /* 16px */
     font-size: ${fontSize}pt;
   }
 
-  @media screen and (min-width: 1920px) {
+  @media screen and (min-width: ${breakpoints.large}) {
     /* 20px */
     font-size: ${fontSize * 1.25}pt;
   }
 
-  @media screen and (min-width: 2560px) {
+  @media screen and (min-width: ${breakpoints.xlarge}) {
     /* 24px */
     font-size: ${fontSize * 1.5}pt;
   }

src/pages/kilta/kunnianosoitukset.tsx

@@ -7,7 +7,7 @@ import PageWrapper from "@views/common/PageWrapper";
 const HonoraryPage: NextPage = () => (
   <>
     <Head>
-      <link rel="canonical" href={`${process.env.NEXT_PUBLIC_SITE_URL}/kilta/kunnia`} />
+      <link rel="canonical" href={`${process.env.NEXT_PUBLIC_SITE_URL}/kilta/kunnianosoitukset`} />
     </Head>
     <PageWrapper>
       <HonoraryPageView />

src/theme/breakpoints.ts

@@ -1,10 +1,10 @@
 type Keys = "mobile" | "medium" | "large" | "xlarge";
 
 const breakpointsNumber: Record<Keys, number> = {
-  mobile: 800,
-  medium: 1200,
-  large: 1920,
-  xlarge: 2560,
+  mobile: 801,
+  medium: 1201,
+  large: 1921,
+  xlarge: 2561,
 };
 
 const breakpoints = Object.entries(breakpointsNumber).map(([k, v]) => [k, `${v}px`]).reduce((a, [k, v]) => ({ ...a, [k]: v }), {});

src/views/StudiesPage/StudiesPageView.tsx

@@ -49,16 +49,16 @@ const StudiesPageView: React.FC = () => (
         <p>
           Fuksivuoden loppupuolella aloin haaveilla oman alan hommista ja päätin laittaa kesätyöhakemuksia yrityksiin ihan laidasta laitaan.
           Lopulta taisin laittaa niitä jopa 50… Ei mennyt kauaa, kun Profit Softwarelta oltiin yhteydessä ja pääsin sinne haastatteluun.
-          Ennen pitkää olikin jo kesä ja olin päässyt osaksi Profit Softwaren automaatiotraineeteamiä.
-          Jatkoin kesän jälkeen osa-aikaisena traineenä ja seuraavana kesänä pääsinkin scrum master -roolissa ohjaamaan useita erilaisia traineeprojekteja.
+          Ennen pitkää olikin jo kesä ja olin päässyt osaksi Profit Softwaren automaatio-trainee-teamiä.
+          Jatkoin kesän jälkeen osa-aikaisena traineenä ja seuraavana kesänä pääsinkin scrum master -roolissa ohjaamaan useita erilaisia trainee-projekteja.
           Ihastuin projektienhallintaan ja jatkan projektitaitojen harjoittamista töissä edelleen.
         </p>
 
         <h6>Toni</h6>
         <p>
-          Kuulin fuksivuotenani erään kurssin vierasluennoitsijalta Fingrid Datahub hankkeesta.
-          Projekti vaikutti mielenkiintoiselta, koska se tulee muuttamaan suomen sähkömarkkinoiden viestintää.
-          Toisen vuoden opiskelijana kesätöitä hakemassa törmäsin työilmoitukseen, jossa haettiin Fingrid Datahub hankkeeseen testaajia.
+          Kuulin fuksivuotenani erään kurssin vierasluennoitsijalta Fingrid Datahub -hankkeesta.
+          Projekti vaikutti mielenkiintoiselta, koska se tulee muuttamaan Suomen sähkömarkkinoiden viestintää.
+          Toisen vuoden opiskelijana kesätöitä hakemassa törmäsin työilmoitukseen, jossa haettiin Fingrid Datahub -hankkeeseen testaajia.
           Päätin laittaa hakemuksen vetämään ja onnekseni sain työpaikan.
           Toimin koko kesän sovellustestauksen parissa sekä kehittäen useita erilaisia sovelluksia testauksen automatisoimiseksi.
           Kesän päätyttyä sain onnekseni jatkoa työsopimukselle ja toimin sovellustestaajana kyseisessä projektissa edelleen.