Skip to content

fix: don't crop the OpenJS trademark on desktop #2183

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.

Sign up for GitHub

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

Closed
notsecret808 wants to merge 7 commits into from
+5 −7
Closed

fix: don't crop the OpenJS trademark on desktop #2183

Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
05f2653
fix: don't crop OpenJS trademark on desktop
notsecret808 Feb 12, 2026
2c35136
chore: revert style.css to crlf
notsecret808 Feb 14, 2026
9740200
Merge branch 'gh-pages' into gh-pages
notsecret808 Feb 23, 2026
6798e10
style: make icon responsive, remove unuses cursor
notsecret808 Mar 4, 2026
4cd9d0d
Merge branch 'gh-pages' into gh-pages
notsecret808 Mar 4, 2026
4c76f4f
fix foundation logo clickable area on small screens
ShubhamOulkar Mar 29, 2026
cb7829f
Merge branch 'gh-pages' into gh-pages
ShubhamOulkar Mar 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion css/style.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -782,7 +782,7 @@ footer {

/* icons */
a.openjs-logo {
display: inline-block;
display: flex;
Copy link
Copy Markdown
Member

@ShubhamOulkar ShubhamOulkar Mar 3, 2026

Choose a reason for hiding this comment

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

Suggested change
display: flex;
display: inline-block;

This is only child component so it is not ideal here.

Copy link
Copy Markdown
Author

@notsecret808 notsecret808 Mar 4, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Agreed. Now it's working as exprected. Sorry, I think that the problem was on my side during the building of the app.

width: 125px;
height: 39px;
cursor: pointer;
Copy link
Copy Markdown
Member

@ShubhamOulkar ShubhamOulkar Mar 3, 2026

Choose a reason for hiding this comment

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

Suggested change
cursor: pointer;

Anchors already have pointer cursor by default.

Expand Down