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

NF: patch2self2 banner #72

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Conversation

iprajwalreddy
Copy link
Contributor

Added Patch2self2 images for the carousel on the main dipy page

Copy link
Contributor

github-actions bot commented Dec 20, 2024

🪓 PR closed, deleted preview at https://github.com/dipy/preview-html/tree/main/dipy.org/pull/72/

@maharshi-gor
Copy link
Member

LGTM, Merging!

@maharshi-gor maharshi-gor merged commit 1b5836d into dipy:master Dec 20, 2024
3 checks passed
github-actions bot added a commit that referenced this pull request Dec 20, 2024
@skoudoro
Copy link
Member

this PR has been merged too fast!

the quality/resolution of the banner is very bad (blurry text mainly). look at the other banners and you can see the quality difference for the text and image. this is the first think we see when we go to the website see below

image

compare to

image

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