-
Notifications
You must be signed in to change notification settings - Fork 4k
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.
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
Migrate sprites from css to a Sprite vue component #15242
base: develop
Are you sure you want to change the base?
Conversation
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.
LGTM, just one issue found
@@ -168,34 +174,40 @@ export default { | |||
isAllowedToFeed () { | |||
return isAllowedToFeed(this.item, this.userItems); | |||
}, | |||
getPetItemClass () { | |||
itemClass () { | |||
if (this.isOwned() || (this.mountOwned() && this.isHatchable()) || this.isSpecial()) { |
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.
missing the "isHatchable" alone - see https://github.com/HabitRPG/habitica/pull/15242/files#diff-4b9c11aea9816b0f238d1da7970e671a1189835e1b0bfe7aa061b48bd3401262L190 so I guess this &&
can be changed to a ||
and then it should be fine :)
44f6031
to
b200a2f
Compare
Creates a new Sprite component that loads images from aws similar to how we handle it in the mobile apps
Started migrating several areas
Still needs to be done (Not in this PR)