Repository navigation
Feedbacks resolved - #77
Conversation
WalkthroughThis pull request introduces visual updates to the application's navigation components, focusing on icon modifications across multiple files. The changes primarily involve replacing the house icon with heart icons in the tab and drawer layouts. The updates span the tab layout, drawer menu content, and collaboration details components, with a consistent theme of updating icon representations. Additionally, a style modification in the collaboration details stylesheet adjusts the modal content's background color to be theme-responsive. Changes
Suggested reviewers
Possibly related PRs
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
components/drawer-layout/DrawerMenuContent.tsx (1)
Line range hint
33-33: Remove unused HomeIcon import.The HomeIcon import is no longer used after switching to heart icons.
-import HomeIcon from "@/assets/icons/home.svg";
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
app/(main)/(drawer)/(tabs)/_layout.tsx(2 hunks)components/collaboration/collaboration-details/InvitationsTabContent.tsx(1 hunks)components/drawer-layout/DrawerMenuContent.tsx(2 hunks)shared-libs(1 hunks)styles/collaboration-details/CollaborationDetails.styles.tsx(0 hunks)
💤 Files with no reviewable changes (1)
- styles/collaboration-details/CollaborationDetails.styles.tsx
✅ Files skipped from review due to trivial changes (2)
- shared-libs
- components/collaboration/collaboration-details/InvitationsTabContent.tsx
🔇 Additional comments (4)
app/(main)/(drawer)/(tabs)/_layout.tsx (2)
14-14: LGTM: Icon imports are properly organized.The heart icons are correctly imported from their respective packages, maintaining consistency with other icon imports.
Also applies to: 19-19
53-59: LGTM: Tab icon implementation follows best practices.The icon implementation:
- Correctly uses conditional rendering based on focus state
- Maintains consistent sizing with other tab icons
- Follows the established pattern used by other tabs
components/drawer-layout/DrawerMenuContent.tsx (2)
19-19: LGTM: Icon imports are properly organized.The heart icons are correctly imported from their respective packages, maintaining consistency with other icon imports.
Also applies to: 25-25
91-96: LGTM: Drawer icon implementation follows best practices.The icon implementation:
- Correctly uses conditional rendering based on focus state
- Uses the DrawerIcon component consistently
- Matches the pattern used by other menu items
* pushed new contract screens * fixed member container * Redesign application and invitation cards (#51) * redesign application and invitation cards * added keyboard view * fixes * added underline in file attachment * ui-feedback * added work * minor changes * Repurpose messaging screen (#53) * repurpose messaging screen * accept application changes * added phases * invitationId as influencerId * fixed signup signin * fix: bug on brand creation * minor changes * minor changes * minor changes * minor changes * pushed a minor change * fix: accept application (#56) * fix: accept application * fix: managerId * ui fix * Redirection fixes (#59) * fix: initial redirections * scheme change * router navigate fix * router navigate fix * Redirection changes (#61) * redirection changes * redirection changes * image component and other fixes * No brands protected screen implementation (#64) * added no brands protected screen and shimmer ui * shimmer ui update * added loading indicator * brand protected screen bug fix * shared-uis conflict * shared-uis conflict * fix: build * pushed ui fixes * social button change * Bug fixes (#68) * bug fixes * bug fixes * message screen fix * pushed ui fix * pushed icons * Bug fixes (#69) * bug fixes * render html, location address, and rating section * refactored code * refactor * refactor * location address changes * messages icons changes * pushed new icons * pushed brand modal fixes * pushed changes * Icons changes (#73) * icons changes * icons changes * email verification * changed icon * pushed slight change to dev for a slight route change * merge conflict * refactor code (#75) * feedbacks resolved (#77) * pushed changes * Feedbacks resolved (#79) * feedbacks resolved * max width refactor * removed unused imports * reusable component fix --------- Co-authored-by: Manan Sharma <112090668+Manan-Sharma-5@users.noreply.github.com>
Summary by CodeRabbit
Release Notes
UI Updates
Styling
Chores