Skip to content

feat: overhaul session UI#22

Merged
MaximilianAnzinger merged 58 commits into
mainfrom
feature/session-overhaul
Jan 13, 2026
Merged

feat: overhaul session UI#22
MaximilianAnzinger merged 58 commits into
mainfrom
feature/session-overhaul

Conversation

@markstockhausen

@markstockhausen markstockhausen commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

Description

Redesigns the session mapping UI with side‑by‑side competency cards, a prominent relationship selector with shortcuts, and swap direction controls. Updates tooltip positioning and adds custom thin scrollbars.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as
    expected)
  • 📝 Documentation update
  • 🎨 Style/formatting changes (no functional changes)
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvements
  • 🔧 Configuration changes
  • 🧪 Test additions or modifications
  • 🚀 Deployment/DevOps changes

Related Issues

Changes Made

  • Client: new session page layout, relationship selector UI with icons/tooltips/keyboard shortcuts, swap direction button, updated tooltip behavior
  • Server: N/A
  • Database: N/A
  • Other:

Testing

Not run (not requested).

Test Cases

  • Existing functionality still works
  • New functionality works as expected
  • Edge cases handled appropriately

Manual Testing

  • Tested in development environment
  • Tested with different user roles/permissions (if applicable)

Screenshots/Videos

Before

Before

After

image

vtotalova and others added 30 commits November 25, 2025 12:50
- Add unique constraint on CompetencyRelationship to prevent duplicates
- Add database indexes on CompetencyResourceLink foreign keys for performance
- Make userId optional in CompetencyResourceLink with SetNull on delete
- Add error logging and visibility in migration system
- Move database docs to root as DATABASE.md
- Sync package-lock.json with package.json (TypeScript 5.9.3)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Maximilian Anzinger <44003963+MaximilianAnzinger@users.noreply.github.com>
Co-authored-by: Maximilian Anzinger <44003963+MaximilianAnzinger@users.noreply.github.com>
…om:ls1intum/memo into feature/setup-domain-core-db-communication
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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


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

Comment thread app/session/page.tsx Outdated
Comment thread app/session/page.tsx
Comment thread components/ui/tooltip.tsx Outdated
Comment thread prisma/schema.prisma
Comment thread components/ui/tooltip.tsx Outdated
Comment thread app/session/page.tsx Outdated
Comment thread app/session/page.tsx
Comment thread app/session/page.tsx Outdated
Comment thread app/session/page.tsx
Comment thread prisma/schema.prisma
@ls1intum ls1intum deleted a comment from Copilot AI Jan 12, 2026
@ls1intum ls1intum deleted a comment from Copilot AI Jan 12, 2026
@ls1intum ls1intum deleted a comment from Copilot AI Jan 12, 2026
@ls1intum ls1intum deleted a comment from Copilot AI Jan 12, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@vtotalova vtotalova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please note that the review comments are for every React.Something, it could be simplified

Comment thread components/ui/tooltip.tsx
Comment thread components/ui/tooltip.tsx Outdated
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@markstockhausen

Copy link
Copy Markdown
Contributor Author

@MaximilianAnzinger, can you merge this PR, please?

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@MaximilianAnzinger
MaximilianAnzinger merged commit 9c7cdd0 into main Jan 13, 2026
8 checks passed
@MaximilianAnzinger
MaximilianAnzinger deleted the feature/session-overhaul branch January 13, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client-application 🎨 Client-side/UI changes database 🗃️ Database related changes docker 🐳 Docker/deployment changes enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants