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

Responsive Design Issue: div.subtlecircle Overflow on Small Screens (< 767px) #1367

Closed
3 tasks done
VitthalGund opened this issue Oct 3, 2023 · 9 comments
Closed
3 tasks done

Comments

@VitthalGund
Copy link
Contributor

🗹 Requirements

  • I have searched the issues for my issue and found nothing related and/or helpful
  • I have searched the FAQ for help
  • I have searched the Wiki for help

🎯 Subject of the issue

Experienced behavior:
I have observed the div.subtlecircle element overflows out of its parent container below 767px width. This same problem also occurs in the Terminal Fonts Examples container, where the div.subtlecircle images overflow in the same manner as in the feature section. This is not good for smaller screen sizes less than 767px, which breaks the design of the website for devices with screen sizes less than 767px.

Expected behavior:
The rounded logo should be overflow for screen sizes less than 767px. This can be solved by applying and modifying the existing CSS on that page.

🔧 Your Setup

  • Which terminal emulator are you using?

    • Google Chrome
  • Which operating system are you using?

    • Windows 11

I want to work on this issue, can assign it to me and also based on my observation the I am not able to find the appropriate CSS file to make changes, can get any kind of document which explain the structure, or it would be great if any explain comments only.

★ Screenshots (Optional)

Experienced behavior:
image

Expected behavior:
image

@Finii Finii added the confirmed label Oct 4, 2023
@Finii
Copy link
Collaborator

Finii commented Oct 4, 2023

Thanks for reporting the issue! I never noticed that.

Interestingly the other circle seems to be less broken:

image

But then, the upper circle (with the icons) is even broken on wider widths: It is not centered within the pale turquoise backdrop

image

To work on this you need to check out the gh-pages branch. You can test any changes you did locally with jekyll s (see README.md in gh-pages branch). The subtlecircle is probably here ;-)

image

The other circles behave different, but maybe more gracefull, when the widths gets small, for example

image

image

@Finii
Copy link
Collaborator

Finii commented Oct 4, 2023

I assume you want to work on this for Hacktoberfest? We do not do 'assignments' usually, is that needed for the fest?

@Finii
Copy link
Collaborator

Finii commented Oct 4, 2023

Apart from the circle, there is more breaking when the width decreases, see #1366 (comment)

@VitthalGund
Copy link
Contributor Author

VitthalGund commented Oct 4, 2023

Hi @Finii ,

Thank you for your swift response and for bringing up issue #1366. I greatly value your attention to this matter.

I've taken a look at issue #1366 and agree that ensuring a consistent user experience, especially on smaller screens, is crucial. Based on my observations, for screen sizes less than 800px, we could consider relocating the "Fork us on GitHub" label to the top of the navbar since space is limited. For screen sizes greater than 800px, we can use rem units to adjust the font size and reposition the label closer to the corner.

Before proceeding, I would appreciate your thoughts on this approach. I plan to explore potential solutions further, and I'll share my findings, possibly later today.

If you have any specific guidelines or additional information you'd like me to consider while working on this, please don't hesitate to share. My goal is to make a positive contribution to the "nerd-fonts" project while adhering to best practices.

Once again, thank you for your guidance, and I eagerly anticipate the opportunity to contribute meaningfully to the project.

Best regards,
Vitthal Gund

@Finii
Copy link
Collaborator

Finii commented Oct 5, 2023

Thank you @VitthalGund !

I guess we could also just drop the fork-me banner when it starts to be a problem. On very small sizes it is not shown anyhow.
And I believe serious work (i.e. forking) is not to be expected with small screen devices.

🤔

I think whatever YOU feel is appropriate will be good 👍 I'm no web designer and have no strong opinions :-)

Also I always wondered if that banner is any good. Does not anybody know you can fork? And if not, why does it lead to the gh-pages branch instead to the default (master) branch?

@VitthalGund
Copy link
Contributor Author

Hi @Finii,

Thank you for your feedback and your trust in my approach. I appreciate your insights, and I'm glad to have your support in addressing this issue.

Based on our discussion, here's the plan moving forward:

  1. We will consider dropping the "Fork-me" banner and, in its place, add a 'Visit GitHub Repository' button just below the 'number of stars' and 'forks.' This will provide a more user-friendly way for visitors, including those on smaller screens, to easily find and access the repository.

  2. We will ensure that this solution is implemented effectively to enhance the user experience.

  3. Additionally, we will investigate and resolve any issues with the banner's link, whether it's a typo or a misconfiguration. We want to ensure that users are directed to the correct repository branch.

Once these changes are made, we can revisit the issue and confirm that it's been successfully resolved.

Best regards,
Vitthal Gund

VitthalGund added a commit to VitthalGund/nerd-fonts that referenced this issue Oct 5, 2023
VitthalGund added a commit to VitthalGund/nerd-fonts that referenced this issue Oct 5, 2023
the subtitle-icon class is added and with some media quires to make the page responsive
@VitthalGund
Copy link
Contributor Author

Hii @Finii,

As per our discussion, we initially considered adding the button just below the 'number of stars' and 'forks,' which would redirect the user to the GitHub repository. However, after implementing this change, I realized that it may not provide the best user experience.

The issue with placing the link in that location is that if a user scrolls down to other sections of the page and then decides to visit the GitHub repository, they would have to scroll back up to access the link. This could potentially be frustrating for users. In an effort to improve user-friendliness, it might be more effective to provide the link in a location that's easily accessible no matter where the user is on the page.

Regarding the question you raised about the link leading to the gh-pages branch instead of the default (master) branch, I've addressed that as well. As I suspected, it was likely a typo in the link. I've corrected this by using the following link: https://github.com/ryanoasis/nerd-fonts/tree/master. This will ensure that users are directed to the master branch as expected.

Here are the before and after screenshots of the implementation:

Before:

image

After:

image

I hope these changes align with your expectations and improve the overall user experience. If you have any further suggestions or feedback, please feel free to share. Your input is greatly appreciated!

Best regards,
Vitthal Gund

Finii pushed a commit that referenced this issue Oct 7, 2023
the subtitle-icon class is added and with some media quires to make the page responsive
@Finii
Copy link
Collaborator

Finii commented Oct 7, 2023

Fixed by #1368

@Finii Finii closed this as completed Oct 7, 2023
Copy link
Contributor

github-actions bot commented Apr 8, 2024

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants