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

Allow small image size on project page? #4

Closed
renyuneyun opened this issue May 6, 2019 · 3 comments
Closed

Allow small image size on project page? #4

renyuneyun opened this issue May 6, 2019 · 3 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@renyuneyun
Copy link

Hi there,
Thank you for creating this theme. It's elegant and I'm using it for my portfolio page (yet my blog is in pelican 🤐 ).

I'm wondering would you be willing to allow smaller cover images for each project (on the project page, of course)?
Currently, if I put a small image there, it will be resized to match the width (I suppose?). But I'm thinking of using different-sized images to represent different emphases of projects.

I see the reason of resizing: making them more uniformed, especially for larger pictures. And I'm aware there is a potential problem if the images are kept of their original sizes for responsive design.
But maybe allowing manual overridden of ratio may solve the problem? Say, by default 100%, but the user could manually override to, say, 40% and (the width of) that project card is resized.

@yousinix yousinix self-assigned this May 19, 2019
@yousinix yousinix added the good first issue Good for newcomers label May 19, 2019
@yousinix
Copy link
Owner

yousinix commented May 19, 2019

Thanks @renyuneyun, Will surely do this soon!
Please provide me with more details if possible (a screenshot would be a great choice).

@yousinix
Copy link
Owner

If you mean the card cover image, I am afraid that I cannot fix this.
But the good new is that you can still resize your image using any photo editor software like Adobe Photoshop for example.

card-header

@renyuneyun
Copy link
Author

I was thinking about resizing too, but test shows me that this doesn't work. The image will be stretched to the full width (tested it again today)...

@yousinix yousinix added help wanted Extra attention is needed and removed good first issue Good for newcomers labels Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants