msd-d / PetHub

An app for animal adoption.
1 stars 0 forks source link

General styling and core functionality for animal profiles #66

Closed SnaBe closed 2 years ago

SnaBe commented 2 years ago
Kechr commented 2 years ago

The birthDate calculation needs to be refactored to a cleaner implementation

Is this a new issue?

SnaBe commented 2 years ago

The birthDate calculation needs to be refactored to a cleaner implementation

Is this a new issue?

It was, but has been fixed by a commit.

SnaBe commented 2 years ago

Looking good! But, should we not include the breed on the animal profile? image image

Also, the buttons for phone and email are ellipses, and not circles (on my screen at least) image

Of course, I just decided to commit and make a PR with the work I got done in class, as I won’t be by my computer for the rest of the day. Feel free to mark it as a draft or pull and commit directly.

Joergenson commented 2 years ago

Screenshot_1636830410 Looking at the options; just use the icon with the darkgrey or pethubpink color

SnaBe commented 2 years ago

Screenshot_1636830410 Looking at the options; just use the icon with the darkgrey or pethubpink color

Not sure what you mean, do you want me to replicate your design? Or change the color?

Joergenson commented 2 years ago

Screenshot_1636830410 Looking at the options; just use the icon with the darkgrey or pethubpink color

Not sure what you mean, do you want me to replicate your design? Or change the color?

I mean just use the icon. not buttons.

<TouchableOpacity
  onpress={owo}
>
   <Icon>
</TouchableOpacity>
SnaBe commented 2 years ago

Screenshot_1636830410 Looking at the options; just use the icon with the darkgrey or pethubpink color

Not sure what you mean, do you want me to replicate your design? Or change the color?

I mean just use the icon. not buttons.

<TouchableOpacity
  onpress={owo}
>
   <Icon>
</TouchableOpacity>

Ah got you, I'll get that done no worries.