[Campaign Launcher UI] Redesign Campaign Details page#788
Draft
KirillKirill wants to merge 11 commits intokb/770-redesign-appfrom
Draft
[Campaign Launcher UI] Redesign Campaign Details page#788KirillKirill wants to merge 11 commits intokb/770-redesign-appfrom
KirillKirill wants to merge 11 commits intokb/770-redesign-appfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Redesigns the Campaign Details page UI in the campaign-launcher client, simplifying the details header/stats layout and removing the paid-amount chart feature (and its dependencies).
Changes:
- Refactors Campaign Details header/content layout by moving join/status controls into
CampaignInfoand restylingCampaignStats. - Removes the paid amount chart modal/chart components and related data hook.
- Drops
chart.js/react-chartjs-2dependencies and related unused icons/components.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| campaign-launcher/client/yarn.lock | Removes chart-related packages from the lockfile. |
| campaign-launcher/client/package.json | Removes chart.js, react-chartjs-2, and annotation plugin dependencies. |
| campaign-launcher/client/src/pages/CampaignDetails/index.tsx | Simplifies the page structure to CampaignInfo + CampaignStats within PageWrapper. |
| campaign-launcher/client/src/components/CampaignInfo/index.tsx | Redesigns the campaign header/info row; adds in-app “back” behavior and new inline metadata. |
| campaign-launcher/client/src/components/CampaignStats/index.tsx | Restyles stats cards and replaces target label/value logic with getTargetInfo. |
| campaign-launcher/client/src/components/CampaignStatusLabel/index.tsx | Replaces Chip-based status display with dot + text and new mapping helper. |
| campaign-launcher/client/src/components/CampaignAddress/index.tsx | Adds large size option and adjusts font size mapping. |
| campaign-launcher/client/src/components/Header/index.tsx | Adjusts nav link font weight based on active state. |
| campaign-launcher/client/src/icons/index.tsx | Removes CalendarIcon and ChartIcon exports. |
| campaign-launcher/client/src/hooks/useCampaigns.ts | Removes useCampaignDailyPaidAmounts hook. |
| campaign-launcher/client/src/components/modals/ChartModal/index.tsx | Deletes chart modal UI. |
| campaign-launcher/client/src/components/DailyAmountPaidChart/index.tsx | Deletes Chart.js-based chart component. |
| campaign-launcher/client/src/components/CampaignTypeLabel/index.tsx | Deletes campaign type Chip component. |
| campaign-launcher/client/src/components/CampaignStatusChip/index.tsx | Deletes unused status chip component variant. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Comment on lines
+128
to
+135
| {!isMobile && ( | ||
| <JoinCampaign | ||
| campaign={campaign} | ||
| joinStatus={joinStatus} | ||
| joinedAt={joinedAt} | ||
| isJoinStatusLoading={isJoinStatusLoading} | ||
| /> | ||
| )} |
Collaborator
Author
There was a problem hiding this comment.
will be implemented later once we decide on the whole 'join campaign' flow
campaign-launcher/client/src/components/CampaignStats/index.tsx
Outdated
Show resolved
Hide resolved
d6dce53 to
ad8d991
Compare
31eee14 to
58fe901
Compare
58fe901 to
d6f00e1
Compare
d6f00e1 to
d75255d
Compare
6cc3b9a to
ce85888
Compare
8681fc6 to
5e5cc9f
Compare
5e5cc9f to
2ba8fab
Compare
2ba8fab to
805e0c8
Compare
63d9cca to
69eacd4
Compare
38becbd to
da7ad34
Compare
69eacd4 to
26c831e
Compare
da7ad34 to
27cfeb9
Compare
79efe33 to
c10f9f1
Compare
…nents and chart packages
…r isnt authed; add authed user data provider
695fc94 to
672ca4b
Compare
…isc styles improvements
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue tracking
Context behind the change
How has this been tested?
Release plan
Potential risks; What to monitor; Rollback plan