Skip to content

Conversation

@usu
Copy link
Member

@usu usu commented Sep 28, 2024

Implements

  • printing of checklist ContentNodes (in normal program overview)
  • new printing config, to print activity overview (including only contentTypes LearningObjective, LearningTopic & Checklist)

All is only implemented in nuxt print. React print would need a separate implementation.

Normal program print

image

Dedicated activity overview print for courses

image

@usu usu requested a review from a team September 28, 2024 23:09
@pmattmann pmattmann added the deploy! Creates a feature branch deployment for this PR label Sep 28, 2024
@github-actions
Copy link

github-actions bot commented Sep 28, 2024

Feature branch deployment currently inactive.

If the PR is still open, you can add the deploy! label to this PR to trigger a feature branch deployment.

@usu usu temporarily deployed to feature-branch September 28, 2024 23:42 — with GitHub Actions Inactive
@usu usu temporarily deployed to feature-branch September 28, 2024 23:53 — with GitHub Actions Inactive
@usu usu temporarily deployed to feature-branch September 29, 2024 00:06 — with GitHub Actions Inactive
@usu usu temporarily deployed to feature-branch September 29, 2024 00:58 — with GitHub Actions Inactive
@pmattmann pmattmann requested a review from a team September 29, 2024 06:57
design: {
multiple: true,
},
repairConfig(config, camp) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here some tests would be good

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed with c940e55

<div class="tw-font-bold">{{ entry.checklist.name }}</div>
<ul class="tw-list-disc tw-list-inside">
<li v-for="item in entry.items" :key="item.id">
<span class="tw-mr-1">{{ item.number }}</span> {{ item.text }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good if the text would only wrap to the start of the text.
and maybe we don't need the list style (and just use the numbering, without the dot at the start)

image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

b2a175a

New visualization:
image

@usu usu temporarily deployed to feature-branch September 29, 2024 08:39 — with GitHub Actions Inactive
@pmattmann pmattmann added this pull request to the merge queue Sep 29, 2024
Merged via the queue into ecamp:devel with commit 582d0a2 Sep 29, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy! Creates a feature branch deployment for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants