once the new release system goes live, and releases are real nodes (http://drupal.org/node/83339), it'd be nice to clean up the downloads listed on the project node itself. instead of a single "download default release" link (which will be a pain for maintainers to keep updated in its current form), we should provide some kind of table of download from various branches of interest.

i'm imagining rows for each version of core compatibility, and columns with specific downloads.
either the table should have separate columns for real releases (from tags) and snapshots (from branches), or those should be in separate tables entirely. we don't want this table to get too confusing, so it'd be nice to keep it limited. perhaps tying into the notion of the "default" branch for each compatibility term (http://drupal.org/node/89538) is necessary.

it might also be nice to have a notion of each release "series" having its own description (both a really short thing to live in this table, like "development" or "stable", and a longer blob of text that could go on the all-releases page as a header when browsing by series (see http://drupal.org/node/89537)

however, there are a few ideas floating around about what it should look like. mockups, screenshots and/or patches are most welcome. ;)

thanks,
-derek

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww’s picture

Version: 4.7.x-1.x-dev » 4.7.x-2.x-dev
Assigned: Unassigned » dww
Status: Active » Needs review
FileSize
6.47 KB

the project_release_table() in here is originally by merlinofchaos (thanks!), but has undergone a *lot* of modifications by me. in particular, it now takes advantage of the DB table from http://drupal.org/node/89538 (so you must install at least http://drupal.org/files/issues/project_release_default_version_patch_1.txt for this patch to work). not all of the functionality in the table is being used by this patch yet, but it will be soon via http://drupal.org/node/96971, and it's easier to just include all the code to generate the tables here, and then i can change the call sites via other issues where appropriate.

dww’s picture

FileSize
187.6 KB

screenshot attached. also, i just installed this on s.d.o for folks to play with there, too. ;)

merlinofchaos’s picture

My notes:

Remove

+/********************
+* CSS possibilities
+*/

From the CSS. Or make it more appropriate.

Add 'Release notes' as a link to the release node because while the link already exists, it is not completely obvious it actually gives you release notes and we want to encourage people to provide release notes by pointing them out a lot.

Change the title from 'Releases' to 'Current releases'.

And since it's so easy to do, I think add another table of dev releases; I think there are those who like to stay on the bleeding edge but not necessarily using CVS, and the table is not that big; it is worth putting there.

merlinofchaos’s picture

Also perhaps change 'Operations' to 'Links' which is more appropriate in this context.

merlinofchaos’s picture

If putting dev releases in a table beneath, make it 'official' for the top and 'snapshot' for the bottom to avoid duplicates which are somewhat disconcerting.

dww’s picture

incorporating all of merlin's suggestions.

dww’s picture

more visual nitpicks based on feedback from merlin and webchick. this is pretty close to RTBC, if you ask me. ;)

webchick’s picture

Status: Needs review » Reviewed & tested by the community

I'll be bold. Checked over the code and it looks fine to me, and already helped nitpick the visuals, as stated. :)

dww’s picture

we decided to shrink the size of the date column to give more space for the links. links are back to the same size as everything else (for now -- whenever we start adding more links, we might have to shrink it back down again).

dww’s picture

removing the time from the date of the tarball, since it doesn't really add much and further complicates things.

dww’s picture

Status: Reviewed & tested by the community » Fixed

committed to DRUPAL-4-7--2.

dww’s picture

Status: Fixed » Reviewed & tested by the community
FileSize
1 KB

minor fix: don't display the table (or release links) if the project has disabled releases via the edit subtab.

dww’s picture

Status: Reviewed & tested by the community » Fixed

committed to DRUPAL-4-7--2 and installed on d.o.

Anonymous’s picture

Status: Fixed » Closed (fixed)