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

Re-aligning the main ribbon in the header to the centre #3520

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from

Conversation

divyanash911
Copy link

The ribbon is in the extreme right which can be aligned to the centre by simply changing the value of right attribute
Before:
image
After:
image

The ribbon is in the extreme right which can be aligned to the centre by simply changing the value of right attribute
@ritwik12 ritwik12 added the discussion open-ended issues that haven't yet defined what needs to be worked on label Oct 20, 2022
@divyanash911
Copy link
Author

divyanash911 commented Oct 22, 2022

@shiftkey if there are no modifications / addition requires please merge the PR

@ritwik12
Copy link
Collaborator

@shiftkey Please merge if looks good to you

@shiftbot
Copy link
Collaborator

👋 I'm a robot checking the state of this pull request to save the human reviewers time. I don't see any changes under _data/projects/ so I don't have any feedback here.

As you make changes to this pull request, I'll re-run these checks.

@@ -704,7 +704,7 @@ form {
text-align: center;
padding: .5em 1em;
margin: 0 auto 0;
right: 2em;
right: 15em;
Copy link
Member

@shiftkey shiftkey Aug 27, 2023

Choose a reason for hiding this comment

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

This breaks viewing the header when the viewport is narrow, because it's a hard-coded value:

Screenshot 2023-08-27 at 4 29 35 PM

I'd recommend looking for a solution that works better at different widths (flexbox?), and considering how the header looks when testing in responsive mode in your local browser dev tools:

Screenshot 2023-08-27 at 4 27 58 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion open-ended issues that haven't yet defined what needs to be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants