-
Notifications
You must be signed in to change notification settings - Fork 116
New download and releases page #412
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
Conversation
|
Looks great! Just a quick note: It looks like the tooltips are missing and the button design doesn't match the mockup (by the way, the tooltip should be on the Apple Silicon button instead of where it's placed in the mockup). I'll put the rest of the feedback in the code review. |
SableRaf
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Fernando! Here are some small changes needed. Also see the comment on the PR for some design specific feedback. The icon choices for the bottom links are good!
i18n/react-intl/en.json
Outdated
| { | ||
| "home": "Home", | ||
| "download": "Download", | ||
| "downloadTitle": "Create with code", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Create with code, everywhere"
i18n/react-intl/en.json
Outdated
| "earlierReleases": "Earlier releases have been removed because we can only support the current versions of the software. Read about <a href='https://github.com/processing/processing4/blob/main/build/shared/revisions.md'>revisions between individual releases</a>.", | ||
| "preReleases": "Pre-Releases", | ||
| "downloadOutro": "<a href='https://github.com/processing/processing4/wiki/Changes-in-4.0'>Read about changes in 4.0 since 3.x</a>.", | ||
| "downloadIntro": "With its vibrant community and endless resources, Processing is the perfect free and open-source tool for artists, designers, educators, and anyone looking to explore the creative side of coding. Download Processing and start creating today!", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"downloadIntro": "Processing is open source and is available for macOS, Windows, and Linux. Projects created with Processing are also cross-platform, and can be used on macOS, Windows, Android, Raspberry Pi, and many other Linux platforms."
| else if (name.includes('macos-aarch64')) return '(Apple Silicon)'; | ||
| else if (name.includes('linux-arm32')) return '(Raspberry Pi 32-bit)'; | ||
| else if (name.includes('linux-arm64')) return '(Raspberry Pi 64-bit)'; | ||
| if (name.includes('x64')) return 'Intel 64-bit'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is the place to change it but the order of the buttons for Linux should be as follow: Intel 64 at the top, then Rpi 32, then Rpi 64
|
Oh and one thing I forgot to mention: the latest release of Processing is now 4.1.2 (and no, it hasn't been updated on the website either 😅) |
|
Hey @SableRaf ! I just pushes changes based on your comments:
I have some notes and questions for the couple pending comments:
It doesn't! This was a design update we did while I was implementing the mockup. We updated it cause the previous design for different download assets didn't visually suggest they are clickable or actionable. The OS images had a similar look and weren't clickable, so we went with a different and more obviously clickable look for them.
Yes! I left it for later while developing and then forgot to ask about it. I'm not sure whats expected of the tooltip:
|
Thanks for the clarification! That brings up a tricky conceptual issue with the design. Ok, let's work through it :) The OS images where meant to be clickable, similar to the 1Password download page. The thing is they can get away with it because they only have one possible download per OS, but in our case it complicates the UX flow. The version text below is similar to what you can find on the Blender download page where it is not clickable or actionable as you correctly pointed out. It's also meant to act as additional information for the download button itself so it is better suited as a sort of caption for it. Alright, here's a proposed design that solves those issues by reorganizing the information and using radio buttons for selecting the versions: And with the tooltip: The idea is to show the versions only for the currently selected OS. Note that I separated the Raspberry Pi releases for clarity and to emphasize the "runs on any platform" angle:
Yes, all assets should have a tooltip.
Supported OS versions + release date.
The timestamp for the release can be found in the JSON file for the release (e.g.: |
|
Thanks for the text and comments! Will work on them during the week, and probably will have a talk with Rune and Dana about the designs. Just to clarify about your proposal, what's the interaction flow you expect? What's clickable? What happens when clicked? Which clicks actually trigger a download? |
Thanks for your question! There are two scenarios: A) The OS detection picked the right version for the user: they click on the big blue button to trigger the download. Done. B) The OS detection wasn't successful or the user needs the Apple Silicon, or RPi versions:
Note: The OS detection doesn't have to be perfect since the OS selector allows users to quickly switch to their desired system. |
|
Just reading up on this discussion! I don't feel too strongly about the current layout, although I do think it's a pretty simple UX flow where you click the big blue button or click one of the more specific download links below. Everything is one click. If we do want to change it to multiple clicks with the proposed radio boxes, I think that could work too, but I'm a bit concerned that we're adding options below the download button that affects the state of the download button. That's a bit unexpected, and I would be in favor of moving the download button below the OS selector if we go that direction. @fdoflorenzano @SableRaf Thoughts? |
The concern I have over the current layout is that it brings back the "too many choices" issue the new design was meant to solve. I agree that the "surprise radio buttons" isn't the best. We could show the radio buttons at all time and make it also possible to click directly on the radio button to select the OS instead of the extra step of clicking the OS button then radio button, at the cost of more visual clutter. Putting the download button below would improve the B flow but add friction to the A flow. Another tradeoff 🤔 Also it might make the placement of the "caption" a little more awkward but that's not so bad. I can try an mock this up. |
|
Ok I tried to reorganize things. It brings a bunch of issues however:
I would suggest keeping the button above but making the radio buttons visible even when the OS isn't selected. It is a little more cluttered but that's a compromise I think we can live with 🙃 If you have other suggestions or ideas, let me know! |
|
Thanks for the explanations! Going through the rationale behind the decisions made me think of another alternative. This is trying to:
|
I like that! I'll go forward with this approach :) |
|
Thanks @fdoflorenzano :) I explored the design a little further and I feel like there is a risk that many Apple Silicon users would download the Intel 64-bit version by mistake. I propose the following small tweak: Current design: the Button says "Download Processing 4.1.2 for macOS". If you have an Apple Silicon Mac, you have to notice the small text that says "Intel 64-bit" and know what it means, or click the info button to maybe notice this is not the right version for you. Proposed tweak: Add a line of text that says "Got an M1 or M2 CPU? Download the Apple Silicon version instead." with a link that downloads the Apple Silicon version. |
|
@SableRaf I just pushed the latest tweaks including the warning for Apple Intel and mobile layouts! |
|
Thanks @fdoflorenzano! It looks amazing. Here are a few items of feedback: Main download button
Tooltips
OS detection Thanks again for all your work on this update. I'm really excited to launch the new page and it couldn't have happened without everyone at DSI! |
|
I just got some updates for the tooltip text from Ben. I can make those changes myself later if you prefer. Just putting this here for reference. Windows: macOS Intel 64-bit: macOS Apple Silicon: Linux 64-bit Intel: Raspberry Pi (32-bit ARM): Raspberry Pi (64-bit ARM): |
|
Thanks for the feedback @SableRaf! Just pushed another round of updates.
All considered and tweaked as proposed.
Yeah that;s true, I added a light border so there's more of a visual separation to the background.
I added a little bit more to the horizontal padding. I also tweaked a little z-index issue I found the tooltip buttons had with the donate banner; and fixed the positioning of the tooltip when too close to the right of the screen.
I looked a bit into it and I couldn't find a well established and reliable way of doing it. Some approaches don't work in all browsers and even may break soon because the base API is subjected to change. There are some visual indicators for the Apple Silicon case to realize what they are downloading, and I'm also under the impression they usually have to check compatibility for their software.
Also updated these and their translation! |
SableRaf
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and everything looks good to me! I recommend that we merge and include the new download page in the next release of the website. Thanks to Dana Elkis-Blanks, @fdoflorenzano, @runemadsen and the whole DSI team! ✨
|
@SableRaf amazing! Thanks so much for the feedback! it made for a better result at the end. Do you want me to merge or should I leave it to you to correctly time it? I'm not sure how you do it over here haha |
|
Amazing! Can't wait to see this in the real world :) |
|
@fdoflorenzano I will merge it myself, thanks for asking! I'd like to ask for input on timing the release. |















This PR adds the re-design of the download page, and adds a separate releases page for older releases as proposed by Raphaël.
Changes:
i18n/react-int(both English and Spanish) using Raphaël reference (and translated to Spanish myself).