fix: UX/styling improvements based on PR feedback

This commit is contained in:
Livio Bieri 2024-03-06 20:20:06 +01:00
parent 2b92928060
commit 42211db6ed
2 changed files with 18 additions and 14 deletions

View File

@ -131,7 +131,9 @@ onUnmounted(() => {
:show-start-button="false" :show-start-button="false"
:show-previous-button="showPreviousButton" :show-previous-button="showPreviousButton"
:base-url="props.learningUnit.evaluate_url" :base-url="props.learningUnit.evaluate_url"
:close-button-variant="learningUnitHasFeedbackPage ? 'close' : 'mark_as_done'" :close-button-variant="
learningUnitHasFeedbackPage || isReadOnly ? 'close' : 'mark_as_done'
"
:end-badge-text=" :end-badge-text="
learningUnitHasFeedbackPage ? $t('general.submission') : undefined learningUnitHasFeedbackPage ? $t('general.submission') : undefined
" "

View File

@ -36,10 +36,10 @@ const selectDetails = () => {
<template> <template>
<CockpitProfileContent> <CockpitProfileContent>
<template #side> <template #side>
<div v-for="(entry, index) in MENU_ENTRIES" :key="index" class="mb-4"> <div v-for="(entry, index) in MENU_ENTRIES" :key="index" class="mb-2">
<button <button
class="flex w-full items-center space-x-2 p-2 pr-4 hover:bg-gray-200 lg:pr-8" class="flex w-full items-center space-x-2 p-2 pr-4 text-blue-900 hover:bg-gray-200 lg:pr-8"
:class="{ 'bg-gray-200': active.type === entry.type }" :class="{ 'text-bold bg-gray-200': active.type === entry.type }"
@click="active = entry" @click="active = entry"
> >
<span>{{ $t(entry.label) }}</span> <span>{{ $t(entry.label) }}</span>
@ -47,16 +47,18 @@ const selectDetails = () => {
</div> </div>
</template> </template>
<template #main> <template #main>
<SelfEvaluationAndFeedbackOverview <div class="container-large">
v-if="active.type === 'OVERVIEW'" <SelfEvaluationAndFeedbackOverview
:profile-user-id="props.userId" v-if="active.type === 'OVERVIEW'"
@show-all="selectDetails" :profile-user-id="props.userId"
/> @show-all="selectDetails"
<SelfEvaluationAndFeedbackList />
v-else-if="active.type === 'DETAILS'" <SelfEvaluationAndFeedbackList
class="w-full" v-else-if="active.type === 'DETAILS'"
:profile-user-id="props.userId" class="w-full"
/> :profile-user-id="props.userId"
/>
</div>
</template> </template>
</CockpitProfileContent> </CockpitProfileContent>
</template> </template>