Skip to content

feat(ui): use branded placeholders for images and profile avatars - #64

Merged
CowboyGH merged 1 commit into
developfrom
feature/app-image-placeholders
Apr 4, 2026
Merged

CowboyGH merged 1 commit into
developfrom
feature/app-image-placeholders

Conversation

@CowboyGH

@CowboyGH CowboyGH commented Apr 4, 2026 •

Copy link
Copy Markdown
Owner

🚀 Summary

Implemented branded image fallbacks across the app UI.

The branch replaces the generic icon-based image fallback with the new placeholder assets for remote raster images, and adds a profile-local avatar image widget so user avatars use their own dedicated placeholder artwork without expanding the shared image widget API. This updates the profile user section and edit-profile preview while keeping the solution localized and consistent with the new design assets.

Summary by CodeRabbit

  • New Features

    • Added dedicated avatar image component with improved loading and error handling.
    • Implemented consistent placeholder images across the app.
  • Improvements

    • Enhanced image error handling with automatic icon fallback if assets fail to load.

@CowboyGH CowboyGH self-assigned this Apr 4, 2026
@CowboyGH CowboyGH added area: ui/ux Widgets, layout, animations, or design type: feature New feature or request labels Apr 4, 2026
@coderabbitai

coderabbitai Bot commented Apr 4, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 52a43cb1-d3e1-412d-a2f4-50d249491ad8

📥 Commits

Reviewing files that changed from the base of the PR and between a1151c4 and 6e18f7b.

⛔ Files ignored due to path filters (2)
  • assets/images/avatar_placeholder.png is excluded by !**/*.png
  • assets/images/image_placeholder.png is excluded by !**/*.png
📒 Files selected for processing (5)
  • lib/core/constants/app_assets.dart
  • lib/features/profile/presentation/widgets/edit_profile_dialog.dart
  • lib/features/profile/presentation/widgets/profile_avatar_image.dart
  • lib/features/profile/presentation/widgets/user_section_widget.dart
  • lib/uikit/images/network_image_widget.dart

📝 Walkthrough

Walkthrough

This change introduces two new asset path constants (imagePlaceholder and avatarPlaceholder) and consolidates image placeholder handling across the profile feature. A new ProfileAvatarImage widget is added to replace NetworkImageWidget usage in profile-related screens. The NetworkImageWidget is updated to use asset-based placeholders instead of icon-based ones. The refactoring unifies placeholder rendering behavior across image loading components.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A rabbit hops through flutter code,
New placeholders light the road! ✨
ProfileAvatarImage takes its place,
Cache and assets interlace,
Icons fallback, images align—
Network widgets now refine! 🎨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: replacing generic icon-based placeholders with branded image and avatar placeholders across the UI, which is reflected in all modified files.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/app-image-placeholders

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.

@CowboyGH
CowboyGH merged commit c590b1f into develop Apr 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui/ux Widgets, layout, animations, or design type: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant