WIP: Add projects page, rename git server link to 'Git' #6

Draft
albert wants to merge 5 commits from albert/dev into main
First-time contributor

build files in public were updated as well

build files in public were updated as well
Owner

I like the direction this is headed in and before I merge it, I would like to ask for a few things to be addressed:

  1. For the dropdown, let's put 'Projects' after 'Socials' as I believe we should prioritize giving members a pipeline into our communications
    image

  2. It's a minor thing, but let's change all instances of 'git' to 'Git' for correction capitalization.
    image
    image

  3. For the images (particularly the second and third), download the actual images, place them in static/images and ideally in a subfolder like logos. Also, it might be a good idea to address the inconsistent space each logo is being given currently.
    image

If we can address these three issues, it should be good to go.

I like the direction this is headed in and before I merge it, I would like to ask for a few things to be addressed: 1. For the dropdown, let's put 'Projects' after 'Socials' as I believe we should prioritize giving members a pipeline into our communications ![image](/attachments/47984f3c-00ef-4e51-9d1a-b3595e8a31e0) 2. It's a minor thing, but let's change all instances of 'git' to 'Git' for correction capitalization. ![image](/attachments/2796221a-e749-494f-b4fc-85a81df34379) ![image](/attachments/cf4a277a-5979-492a-9cb1-cbf6c1441b9b) 3. For the images (particularly the second and third), download the actual images, place them in `static/images` and ideally in a subfolder like `logos`. Also, it might be a good idea to address the inconsistent space each logo is being given currently. ![image](/attachments/e28b8ef8-186f-481f-910e-26e98ec3d913) If we can address these three issues, it should be good to go.
Author
First-time contributor

Let me know if the images still look inconsistent since it didn't look like that originally on my end oddly enough.

Let me know if the images still look inconsistent since it didn't look like that originally on my end oddly enough.
Owner

@albert wrote in #6 (comment):

Let me know if the images still look inconsistent since it didn't look like that originally on my end oddly enough.

image

It still has that issue. What engine is your browser user? This is me testing on Brave, which is Chromium-based

@albert wrote in https://git.alugatuci.org/ALUGatUCI/alugatuci.org/pulls/6#issuecomment-98: > Let me know if the images still look inconsistent since it didn't look like that originally on my end oddly enough. ![image](/attachments/c5f4c7f3-ff49-40b4-b78b-3e8545e79a3a) It still has that issue. What engine is your browser user? This is me testing on Brave, which is Chromium-based
image.png
111 KiB
Author
First-time contributor

That is so weird, I've tried Firefox, Chrome, and Brave and they look fine to me. Maybe the aspect ratio in this css class is the culprit:

.social-logo, .image-card-image-container {
  display: flex;
  align-items: center;
  justify-content: center;
  aspect-ratio: 16 / 10;
  background-color: #2d2d4d;
}

Would you mind tinkering around with this on your end?

That is so weird, I've tried Firefox, Chrome, and Brave and they look fine to me. Maybe the aspect ratio in this css class is the culprit: ``` .social-logo, .image-card-image-container { display: flex; align-items: center; justify-content: center; aspect-ratio: 16 / 10; background-color: #2d2d4d; } ``` Would you mind tinkering around with this on your end?
Owner

Ended up fixing it on my end. I think I will merge it now

Ended up fixing it on my end. I think I will merge it now
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin "albert/dev:albert/dev"
git switch "albert/dev"

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch "main"
git merge --no-ff "albert/dev"
git switch "albert/dev"
git rebase "main"
git switch "main"
git merge --ff-only "albert/dev"
git switch "albert/dev"
git rebase "main"
git switch "main"
git merge --no-ff "albert/dev"
git switch "main"
git merge --squash "albert/dev"
git switch "main"
git merge --ff-only "albert/dev"
git switch "main"
git merge "albert/dev"
git push origin "main"
Sign in to join this conversation.
No description provided.