Skip to content
Snippets Groups Projects

Update achievements

Merged Annabel Dunstone Gray requested to merge achievements into master

What does this MR do and why?

Update achievements

  • Make icons larger
  • Move counter badges
  • Remove outline
  • Remove max number (limit is no longer needed)

Screenshots or screen recordings (pretend these are different achievements with counters > 1)

Before After
Screenshot_2024-07-22_at_3.42.44_PM Screenshot_2024-07-23_at_10.28.11_AM

How to set up and validate locally

  • Look at user profile achievements
  • I didn't have multiple achievements of the same type locally, so removed v-if="showCountBadge(userAchievement.count)" on the badge in app/assets/javascripts/profile/components/user_achievements.vue

Links

Edited by Annabel Dunstone Gray

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading