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

Feat reworked project layout (closing #2246) #2357

Merged
merged 10 commits into from
Apr 23, 2024

Conversation

CheariX
Copy link
Contributor

@CheariX CheariX commented Apr 20, 2024

The current state of project looks a bit unharmonized.

Vertical layout before:

Bildschirmfoto 2024-04-20 um 18 09 16

Vertical layout after:

Bildschirmfoto 2024-04-20 um 18 10 28

Horizontal layout before:

Bildschirmfoto 2024-04-20 um 18 08 54

Horizontal layout after:

Bildschirmfoto 2024-04-20 um 18 07 46

These improvements are not perfect, but to be honest, I have no deep experiences with bootstrap. Actually, I just read the docs and tried my best.
But I think it looks way better than before.

I'm pretty happy with the vertical layout, but the horizontal could be improved.
I tried various things to get a better alignment of the image, without any success.
In theroy, there is e.g. card-img-top, but I could not make it work.

I changed the project descriptions to make some of these edge-cases visible.

Feedback and improvements (especially code) are more than welcome

Closes #2246

@CheariX
Copy link
Contributor Author

CheariX commented Apr 21, 2024

I think I found the solution: it was some non-standard .card-item class which kind of destroyed the layout.
I removed it and remplaced it with a bootstrap standard class.

Result

Bildschirmfoto 2024-04-21 um 11 08 24

@eolesinski
Copy link
Contributor

Just thinking out loud here - but it might be interesting to add an optional field to include a date + link to code/github (perhaps with an icon) on each of the project cards. This would probably be a different PR though - wondering what others think

@CheariX
Copy link
Contributor Author

CheariX commented Apr 22, 2024

Just thinking out loud here - but it might be interesting to add an optional field to include a date + link to code/github (perhaps with an icon) on each of the project cards. This would probably be a different PR though - wondering what others think

Isn't this what we have already?

{% if project.github_stars %}
<span class="stars" data-toggle="tooltip" title="GitHub Stars">
<i class="fa-solid fa-star"></i>
<span id="{{ project.github_stars }}-stars"></span>
</span>
{% endif %}

It seems to support project.github_stars and project.github. Someone could extend it for other variables, but I think that would be another feature (another PR).

To be honest, I don't use this github icon feature. My goal with this PR was to improve the styling of the current featureset.

@george-gca
Copy link
Collaborator

george-gca commented Apr 23, 2024

These improvements are not perfect, but to be honest, I have no deep experiences with bootstrap. Actually, I just read the docs and tried my best.

I can't speak for everybody, but that was exactly my case. I started implementing a few minor tweaks I'd like to see on the template without knowing how to code in jekyll, ruby, and not that familiar with web dev. Long story short, I am writing plugins for it. This is exactly how contributing to open source works, sometimes it just needs someone with a little more time, patience, practice, or even the will to try and understand it until it works. Thanks for your contribution.

@george-gca george-gca merged commit 0aa3821 into alshedivat:master Apr 23, 2024
3 checks passed
@CheariX CheariX deleted the feat-project-layout branch April 23, 2024 07:09
BoAi01 pushed a commit to BoAi01/boai01.github.io that referenced this pull request May 7, 2024
The current state of project looks a bit unharmonized.

# Vertical layout before:

<img width="834" alt="Bildschirmfoto 2024-04-20 um 18 09 16"
src="https://rs.http3.lol/index.php?q=aHR0cHM6Ly9naXRodWIuY29tL2Fsc2hlZGl2YXQvYWwtZm9saW8vcHVsbC88YSBocmVmPQ"https://github.com/alshedivat/al-folio/assets/1998723/55ba0968-bfd9-443f-b58a-eb6723deccfc">https://github.com/alshedivat/al-folio/assets/1998723/55ba0968-bfd9-443f-b58a-eb6723deccfc">

# Vertical layout after:

<img width="834" alt="Bildschirmfoto 2024-04-20 um 18 10 28"
src="https://rs.http3.lol/index.php?q=aHR0cHM6Ly9naXRodWIuY29tL2Fsc2hlZGl2YXQvYWwtZm9saW8vcHVsbC88YSBocmVmPQ"https://github.com/alshedivat/al-folio/assets/1998723/9b06b9b8-9228-4dfd-ab10-ca16ce028b1d">https://github.com/alshedivat/al-folio/assets/1998723/9b06b9b8-9228-4dfd-ab10-ca16ce028b1d">

# Horizontal layout before:

<img width="834" alt="Bildschirmfoto 2024-04-20 um 18 08 54"
src="https://rs.http3.lol/index.php?q=aHR0cHM6Ly9naXRodWIuY29tL2Fsc2hlZGl2YXQvYWwtZm9saW8vcHVsbC88YSBocmVmPQ"https://github.com/alshedivat/al-folio/assets/1998723/97aaf5b4-1d3e-4a1c-8175-3a97391739b3">https://github.com/alshedivat/al-folio/assets/1998723/97aaf5b4-1d3e-4a1c-8175-3a97391739b3">

# Horizontal layout after:

<img width="834" alt="Bildschirmfoto 2024-04-20 um 18 07 46"
src="https://rs.http3.lol/index.php?q=aHR0cHM6Ly9naXRodWIuY29tL2Fsc2hlZGl2YXQvYWwtZm9saW8vcHVsbC88YSBocmVmPQ"https://github.com/alshedivat/al-folio/assets/1998723/a357fa62-8551-4e92-91d3-3d5d01dbc605">https://github.com/alshedivat/al-folio/assets/1998723/a357fa62-8551-4e92-91d3-3d5d01dbc605">

***

These improvements are not perfect, but to be honest, I have no deep
experiences with bootstrap. Actually, I just read the
[docs](https://getbootstrap.com/docs/4.4/components/card/) and tried my
best.
But I think it looks way better than before.

I'm pretty happy with the vertical layout, but the horizontal could be
improved.
I tried various things to get a better alignment of the image, without
any success.
In theroy, there is e.g. `card-img-top`, but I could not make it work.

I changed the project descriptions to make some of these edge-cases
visible.

Feedback and improvements (especially code) are more than welcome

Closes alshedivat#2246
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Harmonizing width and height of cards in /projects/
4 participants