Small layout tweaks for project summary view by annavik · Pull Request #1304 · RolnickLab/antenna

annavik · 2026-05-14T08:42:45Z

Some small tweaks for the new project summary view (introduced in #1296). We now use the space in a bit more effective way and present users without name set as "Anonymous user". Also we streamline font weights.

Before:

After:

Summary by CodeRabbit

Style
- Updated text styling in project summary cards with refined typography and simplified layout for improved readability.
- Enhanced visual presentation of occurrence list items with refined text formatting.
Chores
- Added localization support for user display labels, replacing hardcoded text with translated identifiers for better internationalization support.

netlify · 2026-05-14T08:42:50Z

✅ Deploy Preview for antenna-ssec ready!

Name	Link
🔨 Latest commit	`0fbd95d`
🔍 Latest deploy log	https://app.netlify.com/projects/antenna-ssec/deploys/6a058b0818a8cd0008581459
😎 Deploy Preview	https://deploy-preview-1304--antenna-ssec.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

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

netlify · 2026-05-14T08:42:51Z

✅ Deploy Preview for antenna-preview ready!

Name	Link
🔨 Latest commit	`0fbd95d`
🔍 Latest deploy log	https://app.netlify.com/projects/antenna-preview/deploys/6a058b08f38b3100081cbcdf
😎 Deploy Preview	https://deploy-preview-1304--antenna-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 63 (🔴 down 2 from production) Accessibility: 89 (no change from production) Best Practices: 92 (🔴 down 8 from production) SEO: 92 (no change from production) PWA: 80 (no change from production) View the detailed breakdown and full score reports
🤖 Make changes	Run an agent on this branch

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

coderabbitai · 2026-05-14T08:42:58Z

📝 Walkthrough

Walkthrough

The PR introduces localized handling of anonymous user labels throughout the occurrence and summary features. A new ANONYMOUS_USER string constant is defined with English translation, then consistently applied in data models and summary UI components, accompanied by a ListItem component refactor to improve rendering and styling.

Changes

Localized Anonymous User Labeling and Summary Display

Layer / File(s)	Summary
Localized Anonymous User String `ui/src/utils/language.ts`	`STRING` enum gains `ANONYMOUS_USER` value; `ENGLISH_STRINGS` maps it to "Anonymous user".
Data Model Fallbacks `ui/src/data-services/models/occurrence-details.ts`, `ui/src/data-services/models/occurrence.ts`	Both files import localization helpers and replace hardcoded anonymous/unknown user strings with `translate(STRING.ANONYMOUS_USER)` in fallback paths for missing user names.
ListItem Component Rendering `ui/src/pages/project/summary/list-item.tsx`	Title and text rendering simplified: title becomes a bare truncated span; text uses `body-small text-muted-foreground` for improved typography and muting.
Summary Page Wiring and Styling `ui/src/pages/project/summary/summary.tsx`	Empty-state paragraphs switch from `text-small` to `body-small`; three summary cards rewired to pass item.title and item.text correctly: LatestOccurrences maps dateLabel to text and taxon to title; MostIdentifications applies localized anonymous-user fallback; MostObservedTaxa passes species name via title.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

mihow

🐰 Anonymous users now have a voice,
Localized and true, they can rejoice!
Summary cards shine with cleaner spans,
Each translation fits perfectly in our plans.
The UI hops forward with style and with grace! 🌿✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main changes: layout tweaks and styling adjustments for the project summary view.
Description check	✅ Passed	The description includes a summary with context (`#1296`), screenshots showing before/after changes, and explains the key improvements. However, it lacks formal sections matching the template structure, testing instructions, and deployment notes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch layout/project-overview

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.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@ui/src/pages/project/summary/summary.tsx`:
- Around line 179-181: The current title expression uses user.name?.length so
whitespace-only names like "   " bypass the anonymous fallback; update the
ternary to check the trimmed name and use the trimmed value for display, e.g.
compute const displayName = user.name?.trim(); then set title:
displayName?.length ? displayName : translate(STRING.ANONYMOUS_USER)
(referencing user.name, title, and translate(STRING.ANONYMOUS_USER) to locate
the change).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72d2f6bb-769b-4ce7-8e57-83d2d66ed79f

📥 Commits

Reviewing files that changed from the base of the PR and between 5d2145d and 0fbd95d.

📒 Files selected for processing (5)

ui/src/data-services/models/occurrence-details.ts
ui/src/data-services/models/occurrence.ts
ui/src/pages/project/summary/list-item.tsx
ui/src/pages/project/summary/summary.tsx
ui/src/utils/language.ts

coderabbitai · 2026-05-14T08:45:59Z

+              title: user.name?.length
+                ? user.name
+                : translate(STRING.ANONYMOUS_USER),


⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Handle whitespace-only names in anonymous fallback.

Line 179 uses .length, so a value like " " bypasses the fallback and renders as an empty-looking label. Trim before checking.

Suggested fix

- title: user.name?.length - ? user.name + title: user.name?.trim().length + ? user.name.trim() : translate(STRING.ANONYMOUS_USER),

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

title: user.name?.length

? user.name

: translate(STRING.ANONYMOUS_USER),

title: user.name?.trim().length

? user.name.trim()

: translate(STRING.ANONYMOUS_USER),

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@ui/src/pages/project/summary/summary.tsx` around lines 179 - 181, The current title expression uses user.name?.length so whitespace-only names like " " bypass the anonymous fallback; update the ternary to check the trimmed name and use the trimmed value for display, e.g. compute const displayName = user.name?.trim(); then set title: displayName?.length ? displayName : translate(STRING.ANONYMOUS_USER) (referencing user.name, title, and translate(STRING.ANONYMOUS_USER) to locate the change).

annavik added 3 commits May 14, 2026 10:32

layout: streamline summary view layout and styles

ef9159f

copy: add "Anonymous user" to translated strings

8e098a5

fix: present "Anonymous user" in summary view when user is missing name

0fbd95d

coderabbitai Bot reviewed May 14, 2026

View reviewed changes

annavik merged commit aeb57c1 into main May 14, 2026
7 checks passed

annavik deleted the layout/project-overview branch May 14, 2026 09:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Small layout tweaks for project summary view#1304

Small layout tweaks for project summary view#1304
annavik merged 3 commits into
mainfrom
layout/project-overview

annavik commented May 14, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

netlify Bot commented May 14, 2026 •

edited

Loading

Uh oh!

netlify Bot commented May 14, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented May 14, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot May 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

annavik commented May 14, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

netlify Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for antenna-ssec ready!

Uh oh!

netlify Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for antenna-preview ready!

Uh oh!

coderabbitai Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot May 14, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

annavik commented May 14, 2026 •

edited by coderabbitai Bot

Loading

netlify Bot commented May 14, 2026 •

edited

Loading

netlify Bot commented May 14, 2026 •

edited

Loading

coderabbitai Bot commented May 14, 2026 •

edited

Loading