Skip to content

Conversation

@lewisgoddard
Copy link
Member

@lewisgoddard lewisgoddard commented Dec 14, 2025

Fixes #3845

Changes Summary

  • Update config to list multiple current architectures.
  • Duplicate download sections and restyle.

This pull reques is not ready for review.

@lewisgoddard
Copy link
Member Author

lewisgoddard commented Dec 14, 2025

This still needs a close buttom (clicking outside the dialog doesn't work).

@lewisgoddard
Copy link
Member Author

image

@danirabbit danirabbit moved this to Needs review in OS 8.1.0 Dec 14, 2025
@lewisgoddard
Copy link
Member Author

image

@lewisgoddard lewisgoddard linked an issue Dec 15, 2025 that may be closed by this pull request
@milroneth
Copy link

I feel like a text "Close" button, like in #3920, would be better since clicking outside of the modal doesn't close it and the x on top left is a really small target and is almost missable for someone coming from Windows and not yet used to the close on the left design. (Also I don't like how it looks above the download icon, but that might be just me 😅)

Aesthetics wise, I think I can see it either aligned with the Download buttons only, full width below the magnet buttons too, or same width as Download but aligned right. The last one might actually differentiate it the best, if it doesn't look too weird.

@RMcNeely
Copy link
Contributor

RMcNeely commented Dec 15, 2025

How wide does the page have to be in order for the buttons to spread out to be inline? I'm looking on my laptop (16 inch MBP) and they keep coming up below the text (which could have more space between them)
2025-12-15 07 26 29 localhost 741c21dde24f

Don't mind the broken link button thats my setup 🙈

Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm having trouble building the website but approving anyways since this just merges into the 8.1 branch so we can always iterate on it before the big merge

@danirabbit danirabbit merged commit 0ead0b0 into os-8.1 Dec 16, 2025
5 checks passed
@danirabbit danirabbit deleted the os-8.1_multiarch branch December 16, 2025 17:23
@github-project-automation github-project-automation bot moved this from Needs review to Done in OS 8.1.0 Dec 16, 2025
lewisgoddard added a commit that referenced this pull request Dec 19, 2025
* Update for 8.1

* Update release details for version 8.1

Updated release filename, magnet link, and SHA256 hash for the new version.

* Add support for multiarch downloads in 8.1 (#3922)

* Add download options for Intel, AMD, and ARM 64-bit

* Refactor dialog styles for better option layout

* Reorder release information in config example

* Refactor dialog option styles for better specificity

* Adjust dialog option margins and styles

Updated margin styles for dialog options and titles.

* Revise download modal content for clarity

Updated the download modal text to clarify the choice of OS variant.

* Increase max-width of dialog paragraph

* Update download modal title for clarity

* Add close button to download modal in index.php

* Remove padding-top from main.css

Removed extra padding-top from the element.

* Update links, press page, screenshots

* Actually do screenshots

* fix missing icons

* Update dialog css

* Invert options: best option for most people in bottom right

* Better close button

* Update copy

* Add alt and set autofocus

* Use HTML for close button

* Delete _images/thirdparty-icons/apps/48/installer.svg

---------

Co-authored-by: Lewis Goddard <lewisgoddard@eustasy.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Architecture selector

5 participants