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

doc: fix to show sponsors in narrow view #4793

Merged
merged 2 commits into from Feb 7, 2022

Conversation

outsideris
Copy link
Member

@outsideris outsideris commented Nov 30, 2021

In the narrow width screen, our sponsor logos disapear except the wallaby logo.

스크린샷 2021-12-01 오전 1 09 10

Now, all sponsor logos are displayed.

스크린샷 2021-12-01 오전 1 10 52

Under 320px width, logo size is not good, but it's fine over 320px width.

@coveralls
Copy link

coveralls commented Nov 30, 2021

Coverage Status

Coverage remained the same at 94.433% when pulling bddc02e on outsideris:fix-sponsors-log into cc51b8f on mochajs:master.

@outsideris outsideris marked this pull request as ready for review November 30, 2021 16:14
@juergba
Copy link
Member

juergba commented Dec 2, 2021

For my taste these logos are way too large. And the one of OpenJS Foundation belongs into the gray part, where the netlify logo shows up.

@outsideris
Copy link
Member Author

Could you show a screenshot?

@juergba
Copy link
Member

juergba commented Dec 2, 2021

I mean the current docs, even before this PR. Those logos are larger than our own Mocha logo.

@outsideris
Copy link
Member Author

I moved all sponsorship logos into the footer because I agreed they are too big.

스크린샷 2021-12-31 오후 7 49 15

under 400px width:

스크린샷 2021-12-31 오후 7 49 26

@juergba
Copy link
Member

juergba commented Jan 7, 2022

OpenJS Foundation is not a sponsor, but one of Mocha's owners. That's why I proposed to move its logo to the footer where OpenJS Foundation is mentioned several times.

Imo the remaining true sponsors don't belong into the footer. I don't really like it, but it's upto you.
The size of the images seems ok to me.

@outsideris
Copy link
Member Author

Yes, I also don't like sponsors images bottom. It is a design issue. I will try another approach.

@outsideris
Copy link
Member Author

outsideris commented Jan 26, 2022

How about we remove matomo and wallaby logos on our homepage?

@juergba
Copy link
Member

juergba commented Jan 27, 2022

There are sponsor logos missing, at least the one of Saucelabs. There is a pending issue about that.
So removing the two logos mentioned doesn't solve anything.

Signed-off-by: Outsider <outsideris@gmail.com>
Signed-off-by: Outsider <outsideris@gmail.com>
@outsideris
Copy link
Member Author

I fixed them.

wide width:

스크린샷 2022-02-04 오후 9 08 42

narrow width:
스크린샷 2022-02-04 오후 9 09 38

alt="OpenJS Foundation Logo"
/>
<a href="https://wallabyjs.com" rel="external noopener" title="Mocha is sponsored by Wallaby" id="wallaby-logo">
<img src="https://wallabyjs.com/assets/img/logoWithText.svg" loading="lazy" width="140" alt="Wallaby logo">
Copy link
Member

Choose a reason for hiding this comment

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

Some logos are loaded out of the images folder, some by external links.
What's the difference? Performance?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think there is no reason.
Whether we manage all assets on our side or not is only difference. Performance won't make much difference on mocha site.

@outsideris outsideris merged commit 509938d into mochajs:master Feb 7, 2022
KuznetsovRoman pushed a commit to gemini-testing/mocha that referenced this pull request Sep 6, 2022
* doc: move sponsorship logos into footer
* doc: move sponsorship logos into bottom of content

Signed-off-by: Outsider <outsideris@gmail.com>
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.

None yet

3 participants