Skip to content

Avatar component implementation #2285

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Mil4n0r merged 31 commits into from
Oct 14, 2025
Merged

Avatar component implementation #2285

Mil4n0r merged 31 commits into from
Oct 14, 2025

Conversation

@PelayoFelgueroso
Copy link
Collaborator

@PelayoFelgueroso PelayoFelgueroso commented Sep 15, 2025

Checklist
(Check off all the items before submitting)

  • Build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Purpose
Implementation of the new Avatar component to our Design system, with its corresponding tests and documentation.

@Mil4n0r Mil4n0r self-requested a review September 15, 2025 10:53
@Mil4n0r Mil4n0r self-assigned this Sep 15, 2025
Mil4n0r
Mil4n0r requested changes Sep 15, 2025
View reviewed changes
Mil4n0r
Mil4n0r requested changes Sep 16, 2025
View reviewed changes
PelayoFelgueroso and others added 22 commits September 16, 2025 11:40
@PelayoFelgueroso
Fix based on feedback
bad82a6
@PelayoFelgueroso
Fix border and outline of the component states
6e218a7
@PelayoFelgueroso
Fix focus and active color
cdcde7c
@PelayoFelgueroso
Update storybook with all states
7466164
@PelayoFelgueroso
Fix story book errors by dividing storybook into 3 different pages
b19ebff
@PelayoFelgueroso
Merge branch 'master' into PelayoFelgueroso-avatar
bde8a68
@PelayoFelgueroso
Reduce storybook variants
4df408d
@PelayoFelgueroso
Merge branch 'PelayoFelgueroso-avatar' of https://github.com/dxc-tech…
13d138f 
…nology/halstack-react into PelayoFelgueroso-avatar
@PelayoFelgueroso
Avatar tooltip is displayed by default and reordered avatar circle an…
364a08f 
…d square storybook
@Mil4n0r
Added new logic to ease displaying all the variants in stories
1a6f5f0
@Mil4n0r
Removed comment
8060acf
@PelayoFelgueroso
Added status variant to pseudoStates story and added color variants t…
b3e64ed 
…o statuses story
@PelayoFelgueroso
Merge branch 'master' into PelayoFelgueroso-avatar
6402845
@PelayoFelgueroso
Add DxcTypography & some code improvements
ec9be78
@PelayoFelgueroso
Merge branch 'master' into PelayoFelgueroso-avatar
1dde622
@PelayoFelgueroso
Merge branch 'master' into PelayoFelgueroso-avatar
f8b7e15
@PelayoFelgueroso
Added new semantic token and changed color variants to fit same guide…
13f2c21 
…lines as the Badge component
@PelayoFelgueroso
Merge branch 'master' into PelayoFelgueroso-avatar
eb5bd12
@PelayoFelgueroso
Minor fix on href prop
8f53c5e
@PelayoFelgueroso
Added React.AnchorHTMLAttributes<HTMLAnchorElement> to Avatar.tsx
7f3f77a
@PelayoFelgueroso
Merge branch 'PelayoFelgueroso-avatar' of https://github.com/dxc-tech…
33decde 
…nology/halstack-react into PelayoFelgueroso-avatar
@PelayoFelgueroso
Improve code quality
c765e92
@PelayoFelgueroso PelayoFelgueroso marked this pull request as ready for review October 14, 2025 10:38
Mil4n0r
Mil4n0r requested changes Oct 14, 2025
View reviewed changes
@Mil4n0r Mil4n0r merged commit 163a7b2 into master Oct 14, 2025
5 checks passed
@Mil4n0r Mil4n0r deleted the PelayoFelgueroso-avatar branch October 14, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mil4n0r Mil4n0r Mil4n0r approved these changes

Assignees

@Mil4n0r Mil4n0r

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@PelayoFelgueroso @Mil4n0r