Skip to content

Conversation

@pravinthan
Copy link

@pravinthan pravinthan commented Mar 11, 2021

Added implementation for factions.

Note: In Dev, the pics flicker when the state is changed (for whatever reason). If you run a Prod build, then it's all good.

@pravinthan pravinthan changed the title Added implementation for factions HTV Factions Mar 11, 2021
@pravinthan pravinthan requested a review from fpunny March 11, 2021 05:49
@pravinthan pravinthan linked an issue Mar 11, 2021 that may be closed by this pull request
Copy link
Member

@fpunny fpunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The animations are a bit rough (Missing transitions going "backwards") but on the right track. Some healthy sprinkles of js might elevate it a bit. Also consider include opacity into the transition so it doesnt seem like it just "pops" into existence.

I would not pay too much attention to it however, animations can come later owo. lets focus on fixing everything else first

@fpunny
Copy link
Member

fpunny commented Mar 11, 2021

image
Maybe instead of that. Add a button underneath the content (the spacing would be the same as text + heading) instead would look better

@pravinthan
Copy link
Author

pravinthan commented Mar 12, 2021

image
Maybe instead of that. Add a button underneath the content (the spacing would be the same as text + heading) instead would look better

Yeah this looks good now

image

@pravinthan
Copy link
Author

pravinthan commented Mar 12, 2021

Everything looks a lot better now, thanks for the review.

Also added the themed colours for the logo (since it's much easier to do when the svg is inline)

@fpunny
Copy link
Member

fpunny commented Mar 13, 2021

image
mfw 1% opacity xD, but okay its there

{
health: file(relativePath: { eq: "factions-health.png" }) {
childImageSharp {
gatsbyImageData(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can also look into a fragment, but this would do owo

Copy link
Member

@fpunny fpunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Let's roll this out first, and we can touch up later. Thanks :DD

@pravinthan
Copy link
Author

image
mfw 1% opacity xD, but okay its there

oops opacity cant be % lol

@pravinthan pravinthan merged commit b3a9fae into master Mar 13, 2021
@pravinthan pravinthan deleted the HTV-5 branch March 13, 2021 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Factions Section

3 participants