GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||||
GeezerVison is showing github.com — Fix Linux and Raspberry Pi variants displayed "Intel" in /downloads by ogoregen · Pull Request #695 · processing/processing-website Conversation
|
✅ Deploy Preview for java-processing-faf822 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Flagging for review: should check how the new links get picked up by Fathom analytics. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks for calling that out! I was about to tag you @SableRaf in regards to the visual elements. But sounds like it's good. I'll check the analytics |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
inside of src/templates/download.js there is tracking logic. We need to make sure we're correctly tracking. I will make a PR for this @ogoregen since I need to create some tracking codes.
After that gets merged into this branch, we can merge into main if all looks well.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Another thing is the info tooltip for each distribution could include an explanation so ppl can tell the difference between each build.
I've been thinking about this too. I'd suggest putting the Snap button at the top since that's the recommended install for beginners, and the other two buttons could use a secondary color. People who want the .deb or flatpak versions likely know what they need, but a beginner downloading the wrong package is likely to get stuck. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Got it, @catilac. I think Snap isn't the one we should highlight as it serves only Ubuntu users. Flatpak is the option that is both easy and that serves the most number of distros @SableRaf. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Does Flatpak take care of MIME file association, setting up the JAVA environment variable that the VS Code extension uses to locate Processing, and configure support for the I know that some distributions (like Mint) choose to disable snapd by default, but you can still re-enable it. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@ogoregen @SableRaf -- would it make sense to move this discussion into another issue? it sounds less urgent than fixing these links. let's move this discussion into discord and we can distill it into a task we can assign to @ogoregen? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Came here to file an issue about this. Tool tip would be great but right now this is very confusing, I had to go into devtools to figure out which one I wanted to download. I think it would be great to merge this fix. @SableRaf think you would have the ability to do that. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@duskvirkus This should be addressed by the new download flow in #683, however it is currently blocked by processing/processing4#1425 @catilac If the download flow is likely to take longer, it may be worth patching the buttons separately in the meantime. What do you think? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
@SableRaf Yeah I think patching makes sense for now, I'm currently blocked by other tasks on the way to ars |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I was about to comment on processing/processing4#1425, saying it would be better to fix the links on the website first but now checking the discussion here, it seems we have a circular dependency. Any chance updating the links without #683 and processing/processing4#1425 ? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I just checked the deployment preview and everything looks fine. I think this can be merged without any blockers. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.