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

Created a new layout for GitHub projects #2702

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

KingHowler
Copy link
Contributor

Preview

Screenshot 2024-09-16 205708
Screenshot 2024-09-16 205724
Screenshot 2024-09-16 205832
Screenshot 2024-09-16 205819

Additions

  1. Add repositories, type you repo name in the front matter as the following
repositories:
  - KingHowler/Arduino-Language-Support
  - KingHowler/Oscilloscope-Online
  - alshedivat/al-folio
  1. Add icons to your title, to show which languages and tools were used
icons:
  - https://skillicons.dev/icons?i=processing
  - javascript/javascript-original.svg # Note: You don't have to specify the full link to devicons, only the foldername and filename
  - html5/html5-original.svg

@KingHowler
Copy link
Contributor Author

KingHowler commented Sep 16, 2024

Fixed layout issues for mobile phones

Preview

@KingHowler
Copy link
Contributor Author

@george-gca kindly review this as well

@KingHowler
Copy link
Contributor Author

KingHowler commented Sep 16, 2024

I know this was previously discussed, but this is the updated version. I am using this for my projects section and it felt like a waste for it to not be a feature.

Next I am thinking about adding some default badges to include
for example I am using this in my about page :

Light Mode

image

Dark Mode

image

the above are the ones for my GitHub stars

Light Mode

Screenshot 2024-09-16 220821

Dark Mode

Screenshot 2024-09-16 220830

And these 2 are for installs on my VS extension

I was thinking about adding a little bit more like these so that they can make them easily? or should they do this themselves?

@george-gca
Copy link
Collaborator

george-gca commented Sep 17, 2024

I was thinking about adding a little bit more like these so that they can make them easily? or should they do this themselves?

I believe they should add them themselves.

I am not sure about the layout of the icons in the desktop version, but tbh I don't currently have a counter-proposal. Have you tested it in the description as smaller icons maybe? But I like the idea of this being a different layout available for the users. Can you create (or maybe modify) a project so it uses this layout, so it can be an example of usage?

{% if include.height %}
height="{{ include.height }}"
{% else %}
height="40"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe it is best to use rem whenever possible. This calculator can help.

_includes/icon.liquid Show resolved Hide resolved
_includes/icon.liquid Show resolved Hide resolved
_includes/icon.liquid Show resolved Hide resolved
_layouts/github-project.liquid Show resolved Hide resolved
@KingHowler
Copy link
Contributor Author

KingHowler commented Sep 17, 2024

I was thinking about adding a little bit more like these so that they can make them easily? or should they do this themselves?

I believe they should add them themselves.

I am not sure about the layout of the icons in the desktop version, but tbh I don't currently have a counter-proposal. Have you tested it in the description as smaller icons maybe? But I like the idea of this being a different layout available for the users. Can you create (or maybe modify) a project so it uses this layout, so it can be an example of usage?

Then should I center align the title and add the icons below it? Like in the mobile version?

In my opinion it will make the description look awkward, it should be with the title but if we center align it, I don't think it will give a professional look

@george-gca
Copy link
Collaborator

I think the current desktop version doesn't look very professional, but the mobile one looks nice. I agree to not center align it in the desktop version. What about putting it after the description and before the repo card?

@KingHowler
Copy link
Contributor Author

I think the current desktop version doesn't look very professional, but the mobile one looks nice. I agree to not center align it in the desktop version. What about putting it after the description and before the repo card?

That was how I did it initially but it looked like I was just dumping everything to the left side

@KingHowler
Copy link
Contributor Author

This is the new design I have come up with

PC Version

image

Mobile Version

image

The icons and the repositories can be any number the user wills it to be

The {{ page.icon }} shown in the project card at the all projects page is displayed as an icon on the page

@KingHowler
Copy link
Contributor Author

KingHowler commented Sep 19, 2024

If the new design is better, should I add the description before or after the icons?

Preview : Description before code icons

image

@KingHowler
Copy link
Contributor Author

I am also thinking about adding a divider right before the repositories, it will keep it separated from the center-aligned content

@KingHowler
Copy link
Contributor Author

I am using the design on my website, you can take a look directly if you want to
https://kinghowler.github.io/projects/

@george-gca
Copy link
Collaborator

george-gca commented Sep 20, 2024

Can you make it look like this?

image

I believe left aligned keeps the layout more similar to the other ones we are already using. If the user provides an image it could be shown in the left previous to the title like in this image, if not, it could be hidden and the title and everything would be left aligned.

Also I agree to adding a divider before and after the repo cards.

@KingHowler
Copy link
Contributor Author

I'll try to make it look like that

@alshedivat alshedivat changed the base branch from master to main October 20, 2024 02:47
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