-
Notifications
You must be signed in to change notification settings - Fork 16
Add Badges #359
Conversation
-Add three possible badges to the list of Forum Helpers Page. - Dashboard Admin - Site Contributor - Original Studio Member - The people who should get badges have their badges automatically added after the page loads. - To update who gets what badge, the memberData.js file will have to be updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Also, @Accio1 I was adding some dashboard things and was wondering if you could rename wangat to KangaCoder? Thanks |
Done. In the future if I'm not active or whatever you can achieve the same result by removing the user with the original name and then adding another user with the new name. It should be a fairly rare occurrence though because of how infrequently username changes occur on scratch. |
I'm pretty sure though that would move the user to the bottom of the list though? |
Right, I hadn't considered that. It's such a rare occurrence that I don't mind just directly editing the database when it occurs since adding a username change option to the dashboard would be a bit of a hassle. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
badgeBlock.innerHTML = badgeBlock.innerHTML + '<img src="../resources/admin.png" class="memberBadge" width="32px" height="32px" title="Dashboard Admin">'; | ||
} | ||
if (memberSiteContributor.includes(name)) { | ||
badgeBlock.innerHTML = badgeBlock.innerHTML + '<img src="../resources/coder.png" class="memberBadge" width="32px" height="32px" title="Site Contributer">'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Site Contributer
typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#361 created
Resolves:
Resolves #341
Changes:
Local Tests:
Tested Locally, screenshots attached

On a computer
Mobile dimensions

@leahcimto @penguinmoose