Skip to content

Commit ad420fe

Browse files
Merge pull request #12 from itk-dev/feature/start-route-btn
SALG-1278: Redo placement of carousel and start btn
2 parents 74da9df + 5df3f26 commit ad420fe

File tree

9 files changed

+74
-42
lines changed

9 files changed

+74
-42
lines changed

package-lock.json

Lines changed: 17 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
},
4545
"devDependencies": {
4646
"@babel/eslint-parser": "^7.19.1",
47+
"@tailwindcss/line-clamp": "^0.4.4",
4748
"eslint-config-airbnb": "^19.0.4",
4849
"eslint-config-prettier": "^8.5.0",
4950
"eslint-plugin-cypress": "^2.12.1",

src/components/routes/Route.jsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@ function Route({ id }) {
1717

1818
return (
1919
<Link
20-
className="bg-zinc-100 dark:bg-zinc-700 flex relative"
20+
className="bg-zinc-100 dark:bg-zinc-700 flex flex-row relative h-32"
2121
to={`/route/${id}`}
2222
>
23-
<div className="w-32 flex justify-center place-items-center">
23+
<div className="flex-none w-36">
2424
<Image className="object-cover h-full" src={route.image} />
2525
</div>
26-
<div className="flex-initial text-left leading-none p-3">
27-
<span className="block mb-1">{route.name}</span>
28-
<span className="text-xs ">{route.description}</span>
26+
<div className="flex-initial text-left leading-none py-3 pl-3 pr-6 mb-6">
27+
<h2 className="mb-2 font-bold">{route.name}</h2>
28+
<div className="text-xs line-clamp-3">{route.description}</div>
2929
</div>
3030
</Link>
3131
);

src/components/routes/RouteCarousel.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import "react-responsive-carousel/lib/styles/carousel.min.css"; // requires a lo
77
function RouteCarousel({ routes, onCarouselChange }) {
88
return (
99
<Carousel
10-
className="absolute left-0 bottom-0 right-0 m-5 rounded-lg overflow-hidden max-h-96 md:max-w-lg"
10+
className="rounded-lg overflow-hidden md:max-w-lg"
1111
showThumbs={false}
1212
showStatus={false}
1313
autoplay

src/components/routes/SelectedRoute.jsx

Lines changed: 10 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@ import { React, useEffect, useState } from "react";
22
import MapWrapper from "../map/MapWrapper";
33
import { getFeaturesForMap } from "../../util/helper";
44
import BackButton from "../BackButton";
5-
import { ReactComponent as IconCirclePlay } from "../../icons/circle-play-solid.svg";
6-
import { ReactComponent as IconMap } from "../../icons/map-solid.svg";
75

86
function SelectedRoute({ selectedRoute }) {
97
const [featuresForMap, setFeaturesForMap] = useState(null);
@@ -18,17 +16,22 @@ function SelectedRoute({ selectedRoute }) {
1816

1917
if (selectedRoute === null || featuresForMap === null) return null;
2018

21-
console.log(selectedRoute)
22-
2319
return (
2420
<>
2521
<MapWrapper mapData={featuresForMap} goToView={featuresForMap[0]} />
2622
<BackButton>Kategorier</BackButton>
2723
<div className="relative ml-2">
2824
{/* TODO: show the real category instead of "Valgt kategory" */}
29-
<p className="mt-10 text-sm font-bold text-emerald-800 dark:text-emerald-400 drop-shadow">Valgt kategori</p>
30-
<h1 className="text-4xl font-bold mb-5 drop-shadow">{selectedRoute.name}</h1>
31-
<div id="info-box" className="bg-zinc-100 dark:bg-zinc-700 rounded p-3 w-32 drop-shadow">
25+
<p className="mt-10 text-sm font-bold text-emerald-800 dark:text-emerald-400 drop-shadow">
26+
Valgt kategori
27+
</p>
28+
<h1 className="text-4xl font-bold mb-5 drop-shadow">
29+
{selectedRoute.name}
30+
</h1>
31+
<div
32+
id="info-box"
33+
className="bg-zinc-100 dark:bg-zinc-700 rounded p-3 w-32 drop-shadow"
34+
>
3235
<label htmlFor="distance" className="mb-5">
3336
<span className="text-xs">Distance</span>
3437
<div id="distance" className="text-lg">
@@ -49,26 +52,6 @@ function SelectedRoute({ selectedRoute }) {
4952
<div id="length" className="text-lg" />
5053
</label>
5154
</div>
52-
<div id="buttons" className="relative mt-5 flex gap-3">
53-
<button
54-
className="p-2 rounded text-zinc-100 dark:text-zinc-800 bg-zinc-800 dark:bg-zinc-100 drop-shadow"
55-
type="button"
56-
// TODO: Change onClick to go to selected route
57-
//onClick={() => setSource(podcast)}
58-
>
59-
<IconCirclePlay className="w-6 h-6" />
60-
<span className="sr-only">Afspil</span>
61-
</button>
62-
<button
63-
className="p-2 rounded text-zinc-100 dark:text-zinc-800 bg-zinc-800 dark:bg-zinc-100 drop-shadow"
64-
type="button"
65-
// TODO: Change onClick to toggle map layer
66-
//onClick={() => setSource(podcast)}
67-
>
68-
<IconMap className="w-6 h-6" />
69-
<span className="sr-only">Vis kortet</span>
70-
</button>
71-
</div>
7255
</div>
7356
</>
7457
);

src/components/tags/Tag.jsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@ function Tag({ name, numberOfRoutes, id }) {
99
to={`/tag/${id}`}
1010
>
1111
<span className="block text-lg font-bold">{name}</span>
12-
<span className="text-s text-zinc-600 dark:text-zinc-300">{numberOfRoutes} historier</span>
12+
<span className="text-s text-zinc-600 dark:text-zinc-300">
13+
{numberOfRoutes} historier
14+
</span>
1315
</Link>
1416
);
1517
}

src/components/tags/TagPage.jsx

Lines changed: 28 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ import {
77
getIdsFromApiEndpoints,
88
} from "../../util/helper";
99
import RouteCarousel from "../routes/RouteCarousel";
10+
import { ReactComponent as IconCirclePlay } from "../../icons/circle-play-solid.svg";
11+
import { ReactComponent as IconMap } from "../../icons/map-solid.svg";
1012

1113
// A tag page displays routes connected to the tag-id
1214
function TagPage() {
@@ -50,10 +52,32 @@ function TagPage() {
5052
onCarouselChange={onCarouselChange}
5153
selectedRoute={selectedRoute}
5254
/>
53-
<RouteCarousel
54-
routes={displayedRoutes}
55-
onCarouselChange={onCarouselChange}
56-
/>
55+
<div className="absolute bottom-6 left-6 md:left-auto right-6 flex flex-col gap-3">
56+
<div id="buttons" className="flex gap-3 justify-end">
57+
<button
58+
className="p-2 rounded text-zinc-100 dark:text-zinc-800 bg-zinc-800 dark:bg-zinc-100 drop-shadow"
59+
type="button"
60+
// TODO: Change onClick to toggle map layer
61+
// onClick={() => setSource(podcast)}
62+
>
63+
<IconMap className="w-6 h-6" />
64+
<span className="sr-only">Vis kortet</span>
65+
</button>
66+
<button
67+
className="p-2 rounded text-zinc-100 dark:text-zinc-800 bg-zinc-800 dark:bg-zinc-100 drop-shadow"
68+
type="button"
69+
// TODO: Change onClick to go to selected route
70+
// onClick={() => setSource(podcast)}
71+
>
72+
<IconCirclePlay className="w-6 h-6" />
73+
<span className="sr-only">Afspil</span>
74+
</button>
75+
</div>
76+
<RouteCarousel
77+
routes={displayedRoutes}
78+
onCarouselChange={onCarouselChange}
79+
/>
80+
</div>
5781
</>
5882
);
5983
}

src/components/tags/TagsList.jsx

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,14 @@ function TagsList() {
4848
<div className="flex flex-col gap-2">
4949
<p>Vælg først en kategori på listen herover.</p>
5050
<p>Vælg derefter den ønskede rute.</p>
51-
<p>Gå hen til første del af ruten og der kan du starte afspilningen.</p>
51+
<p>
52+
Gå hen til første del af ruten og der kan du starte
53+
afspilningen.
54+
</p>
5255
<p>
5356
Lyt til vejledningen eller brug navigations pilen i bunden af
54-
rute visningen til at komme til næste punkt.
55-
Husk at være opmærksom på dine opgivelser.
57+
rute visningen til at komme til næste punkt. Husk at være
58+
opmærksom på dine opgivelser.
5659
</p>
5760
<p>God fornøjelse med din podwalk.</p>
5861
</div>

tailwind.config.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,7 @@ module.exports = {
44
theme: {
55
extend: {},
66
},
7-
plugins: [],
7+
plugins: [
8+
require('@tailwindcss/line-clamp'),
9+
],
810
};

0 commit comments

Comments
 (0)