#24 Added ImageView #235

Closed
a24danwe wants to merge 0 commits from #24-Card-for-displaying-images into team_2_week_1
Collaborator

Add ImageView component with dynamic props

  • Display image, title, and description
  • Create ImageView.js
  • Add Image1 SVG to assets
Add ImageView component with dynamic props - Display image, title, and description - Create ImageView.js - Add Image1 SVG to assets
Add ImageView component with dynamic props

- Display image, title, and description
- Create ImageView.js
- Add Image1 SVG to assets
Collaborator

Code looks good. But the image shouldn't get to occupy the entire screen, maybe you could pass a parameter for max size or something like that.

Code looks good. But the image shouldn't get to occupy the entire screen, maybe you could pass a parameter for max size or something like that.
Added max width for the picture
Collaborator

Did a quick check of the changes done to Image view.
The settings change make sense, and works for me.

I will have to merge manually!

Did a quick check of the changes done to Image view. The settings change make sense, and works for me. I will have to merge manually!
Collaborator

Now merged

Now merged
c24elipe closed this pull request 2026-04-13 08:45:52 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!235
No description provided.