Skip to content

DownloadCounter Moved Closer to CTA#412

Open
RounakJoshi09 wants to merge 1 commit intoadoptium:mainfrom
RounakJoshi09:rounakj/download-section
Open

DownloadCounter Moved Closer to CTA#412
RounakJoshi09 wants to merge 1 commit intoadoptium:mainfrom
RounakJoshi09:rounakj/download-section

Conversation

@RounakJoshi09
Copy link
Copy Markdown
Contributor

@RounakJoshi09 RounakJoshi09 commented Oct 4, 2025

#397

  • I've implemented the changes outlined in the issue. Additionally, I adjusted the layout so that the download counter are positioned directly beneath the download button. To achieve better alignment, I slightly reduced the size of the download component. Now, while a user initiates a Temurin download, they'll immediately see the total number of downloads, providing instant credibility and reinforcing trust in the product.
    Another implementation of this enhancement is raised under DownloadCounter Moved Below Carousel #413

@CarmenDelgadoEclipse @gdams @smlambert Please check, if this PR aligns with your expectation. Happy to make adjustments if needed!

Description of change

image
screen-capture.1.webm

Checklist

  • npm test and npm run build passes
  • documentation is changed or added (if applicable)
  • permission has been obtained to add new logo (if applicable)
  • contribution guidelines followed here

…ayout

- Adjusted styles in DownloadCounter for improved responsiveness, including font sizes and padding.
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 4, 2025

Deploy Preview for adoptium-next ready!

Name Link
🔨 Latest commit 64677cb
🔍 Latest deploy log https://app.netlify.com/projects/adoptium-next/deploys/68e0aa2926032700080dfb48
😎 Deploy Preview https://deploy-preview-412--adoptium-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@xavierfacq
Copy link
Copy Markdown
Member

it's the same content that the #413
?

@RounakJoshi09
Copy link
Copy Markdown
Contributor Author

Hi @xavierfacq , Thanks for taking the time to review this. This PR #412 and PR #413 are nearly identical, with one key difference: in PR #413, the download count appears below the adopters carousel, while in this PR, it's positioned below the download button. Both placements were suggested in issues #397 , so I’ve submitted PRs for each approach. Feel free to merge whichever looks best and close the other accordingly.

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.

2 participants