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

feat(sponsors): add CodSpeed as Head Partner #538

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

oscard0m
Copy link
Collaborator

@oscard0m oscard0m commented Aug 6, 2024

No description provided.

@SaeLLo
Copy link
Contributor

SaeLLo commented Aug 6, 2024

Can we maybe put the logos in one line? Otherwise, the importance of sentry looks less.

@oscard0m
Copy link
Collaborator Author

oscard0m commented Aug 6, 2024

Can we maybe put the logos in one line? Otherwise, the importance of sentry looks less.

This is how it looks to me in Mobile and Desktop:

mobile Desktop
CleanShot 2024-08-06 at 15 28 55@2x CleanShot 2024-08-06 at 15 29 43@2x

What device are you using to preview this so it is more than one line?

As a reference, this is how we were doing this in Eurorust 2023:

CleanShot 2024-08-06 at 15 31 54@2x

@SaeLLo
Copy link
Contributor

SaeLLo commented Aug 6, 2024

What device are you using to preview this so it is more than one line?

I used the computer - so I checked the desktop version. in your screenshots it looks good.

@SaeLLo
Copy link
Contributor

SaeLLo commented Aug 6, 2024

image

Codspeed looks pretty big.

@oscard0m
Copy link
Collaborator Author

oscard0m commented Aug 6, 2024

image Codspeed looks pretty big.

Let me check. Thanks for sharing the screenshot.

@oscard0m oscard0m force-pushed the add-codspeed-as-head-partner branch from 62bec6e to c3995b9 Compare August 6, 2024 13:49
@oscard0m oscard0m requested review from SaeLLo and removed request for SaeLLo August 6, 2024 13:50
@oscard0m
Copy link
Collaborator Author

oscard0m commented Aug 6, 2024

How does it look now @SaeLLo ?

@SaeLLo
Copy link
Contributor

SaeLLo commented Aug 6, 2024

Thank you. Looks good now!

@IdeliaRisella IdeliaRisella merged commit 911ba3a into main Aug 7, 2024
6 checks passed
@IdeliaRisella IdeliaRisella deleted the add-codspeed-as-head-partner branch August 7, 2024 11:59
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.

3 participants