Skip to content

fix: Update ActorCard and ActorSearch components for consistent styling#505

Merged
jirispilka merged 1 commit intomasterfrom
fix/search-actor-style
Feb 26, 2026
Merged

fix: Update ActorCard and ActorSearch components for consistent styling#505
jirispilka merged 1 commit intomasterfrom
fix/search-actor-style

Conversation

@jirispilka
Copy link
Copy Markdown
Collaborator

This was Claude work, we can discard if you don't agree

  • Increased border-radius from 8 to 12 in ActorCard and ActorSearchDetail
  • Adjusted spacing in ActorSearch for consistency.
  • Added ActorAvatarWrapper to encapsulate the ActorAvatar component

Table radius shoul be 12px

image

missing border around the actors avatar

image

spacing between cards should be 8px

Not sure how to verify

…styling

- Increased border-radius from 8 to 12 in ActorCard and ActorSearchDetail
- Adjusted spacing in ActorSearch for consistency.
- Added ActorAvatarWrapper to encapsulate the ActorAvatar componen
@jirispilka jirispilka requested a review from jmikitova February 26, 2026 12:20
@github-actions github-actions Bot added the t-ai Issues owned by the AI team. label Feb 26, 2026
@jirispilka jirispilka requested a review from adelamikk February 26, 2026 12:20
Copy link
Copy Markdown
Contributor

@jmikitova jmikitova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great! 👍 please wait for @adelamikk for confirmation

@jmikitova
Copy link
Copy Markdown
Contributor

btw. i believe you can verify spacing in browser when opening https://localhost:3000/index-actor-search.html, there should be mocked Actor cards in a list

Copy link
Copy Markdown

@adelamikk adelamikk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ❤️

@jirispilka
Copy link
Copy Markdown
Collaborator Author

btw. i believe you can verify spacing in browser when opening https://localhost:3000/index-actor-search.html, there should be mocked Actor cards in a list

Thank you!

@jirispilka jirispilka merged commit b6ee8a7 into master Feb 26, 2026
7 checks passed
@jirispilka jirispilka deleted the fix/search-actor-style branch February 26, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-ai Issues owned by the AI team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants