feat: improve readability of the activity indicators

This commit is contained in:
swve 2025-05-25 21:32:39 +02:00
parent 405bdf7503
commit 6aca58ece7
2 changed files with 105 additions and 48 deletions

View file

@ -532,6 +532,7 @@ function ActivityClient(props: ActivityClientProps) {
current_activity={activityid} current_activity={activityid}
orgslug={orgslug} orgslug={orgslug}
course={course} course={course}
enableNavigation={true}
/> />
<div className="flex justify-between items-center w-full"> <div className="flex justify-between items-center w-full">

View file

@ -1,6 +1,6 @@
'use client' 'use client'
import { BookOpenCheck, Check, FileText, Layers, Video } from 'lucide-react' import { BookOpenCheck, Check, FileText, Layers, Video, ChevronLeft, ChevronRight } from 'lucide-react'
import React, { useMemo, memo } from 'react' import React, { useMemo, memo, useState } from 'react'
import ToolTip from '@components/Objects/StyledElements/Tooltip/Tooltip' import ToolTip from '@components/Objects/StyledElements/Tooltip/Tooltip'
import { getUriWithOrg } from '@services/config/config' import { getUriWithOrg } from '@services/config/config'
import Link from 'next/link' import Link from 'next/link'
@ -10,6 +10,7 @@ interface Props {
orgslug: string orgslug: string
course_uuid: string course_uuid: string
current_activity?: string current_activity?: string
enableNavigation?: boolean
} }
// Helper functions // Helper functions
@ -98,6 +99,9 @@ function ActivityIndicators(props: Props) {
const course = props.course const course = props.course
const orgslug = props.orgslug const orgslug = props.orgslug
const courseid = props.course_uuid.replace('course_', '') const courseid = props.course_uuid.replace('course_', '')
const enableNavigation = props.enableNavigation || false
const [currentIndex, setCurrentIndex] = useState(0)
const done_activity_style = 'bg-teal-600 hover:bg-teal-700' const done_activity_style = 'bg-teal-600 hover:bg-teal-700'
const black_activity_style = 'bg-zinc-300 hover:bg-zinc-400' const black_activity_style = 'bg-zinc-300 hover:bg-zinc-400'
@ -105,6 +109,24 @@ function ActivityIndicators(props: Props) {
const trail = props.course.trail const trail = props.course.trail
// Flatten all activities for navigation and rendering
const allActivities = useMemo(() => {
return course.chapters.flatMap((chapter: any) =>
chapter.activities.map((activity: any) => ({
...activity,
chapterId: chapter.id
}))
)
}, [course.chapters])
// Find current activity index
const currentActivityIndex = useMemo(() => {
if (!props.current_activity) return -1
return allActivities.findIndex((activity: any) =>
activity.activity_uuid.replace('activity_', '') === props.current_activity
)
}, [allActivities, props.current_activity])
// Memoize activity status checks // Memoize activity status checks
const isActivityDone = useMemo(() => (activity: any) => { const isActivityDone = useMemo(() => (activity: any) => {
let run = props.course.trail?.runs.find( let run = props.course.trail?.runs.find(
@ -125,22 +147,47 @@ function ActivityIndicators(props: Props) {
}, [props.current_activity]); }, [props.current_activity]);
const getActivityClass = useMemo(() => (activity: any) => { const getActivityClass = useMemo(() => (activity: any) => {
const isCurrent = isActivityCurrent(activity)
if (isActivityDone(activity)) { if (isActivityDone(activity)) {
return done_activity_style return `${done_activity_style}`
} }
if (isActivityCurrent(activity)) { if (isCurrent) {
return current_activity_style return `${current_activity_style} border-2 border-gray-800 animate-pulse`
} }
return black_activity_style return `${black_activity_style}`
}, [isActivityDone, isActivityCurrent]); }, [isActivityDone, isActivityCurrent]);
const navigateToPrevious = () => {
if (currentActivityIndex > 0) {
const prevActivity = allActivities[currentActivityIndex - 1]
const activityId = prevActivity.activity_uuid.replace('activity_', '')
window.location.href = getUriWithOrg(orgslug, '') + `/course/${courseid}/activity/${activityId}`
}
}
const navigateToNext = () => {
if (currentActivityIndex < allActivities.length - 1) {
const nextActivity = allActivities[currentActivityIndex + 1]
const activityId = nextActivity.activity_uuid.replace('activity_', '')
window.location.href = getUriWithOrg(orgslug, '') + `/course/${courseid}/activity/${activityId}`
}
}
return ( return (
<div className="grid grid-flow-col justify-stretch space-x-6"> <div className="flex items-center gap-4">
{course.chapters.map((chapter: any) => { {enableNavigation && (
return ( <button
<React.Fragment key={chapter.id || `chapter-${chapter.name}`}> onClick={navigateToPrevious}
<div className="grid grid-flow-col justify-stretch space-x-2"> disabled={currentActivityIndex <= 0}
{chapter.activities.map((activity: any) => { className="p-1 rounded-full hover:bg-gray-100 disabled:opacity-50 disabled:cursor-not-allowed transition-colors flex-shrink-0"
aria-label="Previous activity"
>
<ChevronLeft size={20} className="text-gray-600" />
</button>
)}
<div className="flex items-center w-full">
{allActivities.map((activity: any) => {
const isDone = isActivityDone(activity) const isDone = isActivityDone(activity)
const isCurrent = isActivityCurrent(activity) const isCurrent = isActivityCurrent(activity)
return ( return (
@ -165,18 +212,27 @@ function ActivityIndicators(props: Props) {
'' ''
)}` )}`
} }
className={`${isCurrent ? 'flex-[2]' : 'flex-1'} mx-1`}
> >
<div <div
className={`h-[7px] w-auto ${getActivityClass(activity)} rounded-lg`} className={`h-[7px] ${getActivityClass(activity)} rounded-lg transition-all`}
></div> ></div>
</Link> </Link>
</ToolTip> </ToolTip>
) )
})} })}
</div> </div>
</React.Fragment>
) {enableNavigation && (
})} <button
onClick={navigateToNext}
disabled={currentActivityIndex >= allActivities.length - 1}
className="p-1 rounded-full hover:bg-gray-100 disabled:opacity-50 disabled:cursor-not-allowed transition-colors flex-shrink-0"
aria-label="Next activity"
>
<ChevronRight size={20} className="text-gray-600" />
</button>
)}
</div> </div>
) )
} }