Skip to content
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

added an aspect ratio to the manga card. #56

Merged
merged 2 commits into from
Oct 31, 2021
Merged

added an aspect ratio to the manga card. #56

merged 2 commits into from
Oct 31, 2021

Conversation

ff2400t
Copy link
Contributor

@ff2400t ff2400t commented Oct 30, 2021

Added an aspect ratio to the MangaCard which prevents resizing of the component. Before this if an image failed to load the component would go in a Square shape if it alone in the row, If all the images in a row failed to load then the whole row would change it's height,

@ff2400t ff2400t closed this Oct 30, 2021
@ff2400t ff2400t deleted the aspect-ratio branch October 30, 2021 17:50
@ff2400t ff2400t restored the aspect-ratio branch October 30, 2021 17:51
@ff2400t ff2400t reopened this Oct 30, 2021
display: 'flex',
aspectRatio: '9/14',
Copy link
Member

Choose a reason for hiding this comment

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

how about the respected 225/350? I know it's the same ratio but it's the golden number when making manga...
also add a comment for future generations reading the code.

Comment on lines +21 to +23
// Preserve the shape of the manga cards, without this the manga card would chage it's
// if all the images in a row failed to load then the whole row would change into a
// square shape
Copy link
Member

Choose a reason for hiding this comment

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

doesn't make sense

@AriaMoradi AriaMoradi merged commit b755ad3 into Suwayomi:master Oct 31, 2021
@ff2400t ff2400t deleted the aspect-ratio branch November 1, 2021 03:35
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.

2 participants