Skip to content

Field icons - #355

Open
jzgom067 wants to merge 5 commits into
v0.4.7from
field-icons
Open

Field icons#355
jzgom067 wants to merge 5 commits into
v0.4.7from
field-icons

Conversation

@jzgom067

Copy link
Copy Markdown
Member

This PR adds icons to various fields on the site for visual reinforcement, including on the event editor and dashboard.

Additionally, the stroke width was adjusted on the event editor warning icons to match the bold text on each field.

@jzgom067 jzgom067 added this to the v0.4.5 milestone Jun 23, 2026
@jzgom067 jzgom067 added cleanup Involves refactoring or small changes frontend Related to frontend stuff labels Jun 23, 2026
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 20150f18-cb1d-489c-8282-f9bea619e89a

📥 Commits

Reviewing files that changed from the base of the PR and between 5e7277c and 82f0c4d.

📒 Files selected for processing (4)
  • frontend/src/features/dashboard/components/event.tsx
  • frontend/src/features/event/editor/advanced-options.tsx
  • frontend/src/features/event/editor/date-range/selector.tsx
  • frontend/src/features/event/editor/editor.tsx

📝 Walkthrough

Walkthrough

Four frontend components receive new lucide-react icon imports and updated JSX layouts: the dashboard event card adds CalendarRangeIcon and UsersIcon with flex-row wrappers; the event editor's advanced options, date-range selector, and times block each add a contextual icon (GlobeIcon, LinkIcon, CalendarsIcon, CalendarRangeIcon, ClockIcon) alongside strokeWidth={2} on existing alert icons.

Changes

Icon additions to event UI components

Layer / File(s) Summary
Dashboard event card icon layout
frontend/src/features/dashboard/components/event.tsx
Import expanded to include CalendarRangeIcon and UsersIcon; date-range row wrapped in a flex container pairing the icon with content; participants section adds UsersIcon and a new inner div carries the ResizeObserver ref around ParticipantRow.
Event editor form section icons
frontend/src/features/event/editor/advanced-options.tsx, frontend/src/features/event/editor/date-range/selector.tsx, frontend/src/features/event/editor/editor.tsx
Imports extended with GlobeIcon, LinkIcon, CalendarsIcon, CalendarRangeIcon, and ClockIcon; Timezone and Custom Event Code labels, Event Type and date-range status row, and Possible Times block each gain an icon and flex-row layout; TriangleAlertIcon gains strokeWidth={2} throughout.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
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.


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

@jzgom067
jzgom067 requested a review from mirmirmirr June 23, 2026 18:25
@jzgom067
jzgom067 changed the base branch from v0.4.5 to v0.4.6 June 29, 2026 17:39
@jzgom067 jzgom067 modified the milestones: v0.4.5, v0.4.6 Jun 29, 2026
@jzgom067 jzgom067 modified the milestones: v0.4.6, v0.4.7 Jul 20, 2026
@jzgom067
jzgom067 changed the base branch from v0.4.6 to v0.4.7 July 20, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Involves refactoring or small changes frontend Related to frontend stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant