Skip to content

Update content#40

Merged
iulusoy merged 3 commits into
mainfrom
update-content
Mar 25, 2026
Merged

Update content#40
iulusoy merged 3 commits into
mainfrom
update-content

Conversation

@iulusoy
Copy link
Copy Markdown
Member

@iulusoy iulusoy commented Mar 25, 2026

  • team description
  • image for blog
  • home page card formatting
  • links in faq text

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the frontend’s public-facing content and layout, expanding the Team page content, refining FAQ link phrasing/placement, adjusting homepage card formatting (including a blog image), and aligning i18n strings with the new UI content.

Changes:

  • Expanded /team page from a short description to a structured team/publications layout with Flowbite components and external links.
  • Updated FAQ answers to embed links inline and split some answers into multiple translation parts.
  • Adjusted homepage card text positioning and added an image to the blog card; removed an unused import in /about.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
src/frontend/src/routes/team/+page.svelte Replaces simple team description with structured sections, lists, and link buttons.
src/frontend/src/routes/faq/+page.svelte Reorders/embeds links within FAQ answers and aligns markup with updated translation keys.
src/frontend/src/routes/about/+page.svelte Removes unused navigation import.
src/frontend/src/routes/+page.svelte Tweaks homepage card text positioning and adds a blog card image plus component-scoped CSS.
src/frontend/src/lib/i18n/translations.ts Adds new team/FAQ translation keys and updates existing FAQ copy for DE/FR.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/frontend/src/routes/team/+page.svelte Outdated
Comment thread src/frontend/src/routes/team/+page.svelte Outdated
Comment thread src/frontend/src/lib/i18n/translations.ts Outdated
Comment thread src/frontend/src/routes/+page.svelte
<h3 class="text-xl text-center">
{$t.team.ResearchTeam}
</h3>
<h4 class="text-l text-center italic">
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

The Tailwind class text-l is not a standard Tailwind text-size utility, so it will be ignored and the heading will render at the default size. Use a valid class such as text-lg/text-base (or remove the size class if not needed).

Copilot uses AI. Check for mistakes.
Comment thread src/frontend/src/routes/team/+page.svelte Outdated
Comment thread src/frontend/src/routes/team/+page.svelte Outdated
Comment thread src/frontend/src/lib/i18n/translations.ts Outdated
{$t.team.PhDStudent1LinkText}
</Button>
<br>
<h4 class="text-l text-center italic">
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

The Tailwind class text-l is not a standard Tailwind text-size utility, so it will be ignored and the heading will render at the default size. Use a valid class such as text-lg/text-base (or remove the size class if not needed).

Copilot uses AI. Check for mistakes.
Comment thread src/frontend/src/routes/team/+page.svelte Outdated
@sonarqubecloud
Copy link
Copy Markdown

@iulusoy iulusoy merged commit 5ce4cf1 into main Mar 25, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants