Skip to content

Small layout tweaks after "Teams" page release#1099

Merged
annavik merged 3 commits intomainfrom
layout/teams-tweaks
Jan 22, 2026
Merged

Small layout tweaks after "Teams" page release#1099
annavik merged 3 commits intomainfrom
layout/teams-tweaks

Conversation

@annavik
Copy link
Member

@annavik annavik commented Jan 22, 2026

Summary

Some small layout tweaks after the "Teams" page release. When seeing the updates in production I noticed a few areas in need of improvement.

List of Changes

  • Merge columns "User" and "Name" to make table more compact
  • Simplify "Create new project" dialog layout (the generic dialog layout was tweaked in the previous PR)
  • Set default sort order to "User" (this will order members by name)

Detailed Description

Screenshots

Teams table (before vs. after):
Screenshot 2026-01-22 at 10 36 54
Screenshot 2026-01-22 at 10 37 31

Dialog (before vs. after):
Screenshot 2026-01-22 at 10 23 50
Screenshot 2026-01-22 at 10 23 31

Summary by CodeRabbit

  • UI Changes

    • Team member rows now stack name (with "YOU" badge) above email; removed the separate name column.
    • Team list now defaults to sorting by member name (ascending).
  • Refactor

    • Simplified project creation form layout by removing extra wrapper elements around name, description, and pipeline fields.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify
Copy link

netlify bot commented Jan 22, 2026

Deploy Preview for antenna-ssec ready!

Name Link
🔨 Latest commit a88b0b4
🔍 Latest deploy log https://app.netlify.com/projects/antenna-ssec/deploys/6971ef8fa8c1b300086269d0
😎 Deploy Preview https://deploy-preview-1099--antenna-ssec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Jan 22, 2026

Deploy Preview for antenna-preview ready!

Name Link
🔨 Latest commit a88b0b4
🔍 Latest deploy log https://app.netlify.com/projects/antenna-preview/deploys/6971ef8f360d18000843ec13
😎 Deploy Preview https://deploy-preview-1099--antenna-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62 (🔴 down 18 from production)
Accessibility: 80 (🔴 down 9 from production)
Best Practices: 100 (🟢 up 8 from production)
SEO: 92 (🔴 down 8 from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 22, 2026

📝 Walkthrough

Walkthrough

Refactored UI layout: simplified form field wrappers in new-project-form.tsx, consolidated user display into a single stacked column and removed the separate name column in team-columns.tsx, and changed the team list default sort to name (asc) in team.tsx.

Changes

Cohort / File(s) Summary
Form Layout Simplification
ui/src/pages/project-details/new-project-form.tsx
Removed FormRow wrappers; replaced with direct FormField and FormController usage for name, description, and defaultProcessingPipeline. No changes to validation, error handling, or submission logic.
Team Column Restructuring
ui/src/pages/project/team/team-columns.tsx
Merged user display into the user column: shows name (font-medium) with optional YOU badge on the first line and email on the second. Removed the separate exported column with id: 'name'. Updated sortField for the user column from user__email to name.
Team List Default Sort
ui/src/pages/project/team/team.tsx
Changed default sort key/order for team listing from updated_at (desc) to name (asc) when initializing the sort hook.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped through fields with nimble paws,
Wrapped less, aligned with simpler laws.
Names stacked above emails, neat and bright,
Sorting by name now feels just right. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description includes a summary, list of changes, and detailed description with before/after screenshots, but lacks testing instructions, deployment notes, and checklist items required by the template. Add 'How to Test the Changes' section with testing instructions, include 'Deployment Notes' if applicable, and complete the checklist at the end of the description.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes as small layout refinements following the Teams page release, matching the PR's focus on UI tweaks.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@annavik annavik force-pushed the layout/teams-tweaks branch from 7ef2f08 to c572337 Compare January 22, 2026 09:34
@annavik annavik force-pushed the layout/teams-tweaks branch from c572337 to a88b0b4 Compare January 22, 2026 09:36
@annavik annavik merged commit f27f7b7 into main Jan 22, 2026
7 checks passed
@annavik annavik deleted the layout/teams-tweaks branch January 22, 2026 09:44
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.

1 participant

Comments