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

fix: image size based on parent #383

Merged
merged 1 commit into from
Sep 18, 2024
Merged

fix: image size based on parent #383

merged 1 commit into from
Sep 18, 2024

Conversation

SamStaijen
Copy link
Member

Currently, when fetching an image from Contentfull, the image size is proportional to image size.

After this PR; default image witdth should be based on the size of the parent element

@TobiasDeBruijn
Copy link
Member

Need to test it, but otherwise looks good!

Copy link
Member

@SilasPeters SilasPeters left a comment

Choose a reason for hiding this comment

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

I would suggest putting the 3 pictures side-by-side, but I'm bad at CSS

@SilasPeters SilasPeters merged commit e150032 into development Sep 18, 2024
2 checks passed
TobiasDeBruijn added a commit that referenced this pull request Sep 23, 2024
* New workflow for adding newly opened issues to the CommITCrowd project

    Perhaps the title says it all. This is a really simple workflow, and it's open for any changes or adjustments

* feat: Hide phone number if not filled in in Contentful

* Add introduction information to the website (#376)

* make default image with based on size of parent (#383)

* Update board color (#382)

* Automatically compute CYMK, RGB color from HEX (#385)

---------

Co-authored-by: Silas Peters <[email protected]>
Co-authored-by: Sem van Nieuwenhuizen <[email protected]>
Co-authored-by: sam32123 <[email protected]>
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.

3 participants