[dpdk-web] [PATCH] release 16.07.1
Van Haaren, Harry
harry.van.haaren at intel.com
Wed Oct 26 10:58:11 CEST 2016
Hi,
Looks ok, I have a few comments:
> -----Original Message-----
> Another note: there is a <p> decorator after the "latest stalbe
> release" icon, because in my local testing, it can't show 4
> icons in one row. Thus I splitted it them in two rows, each with
> 2 icons.
I'm ok with the split into 2x2 grid, centering the buttons makes the page look better IMO.
I can patch after this is applied.
> - <td><a href="http://fast.dpdk.org/rel/dpdk-16.07.tar.xz">DPDK 16.07</a></td>
> - <td><a href="/doc/guides/rel_notes/release_16_07.html">2016 July 28</a></td>
Looks like release notes date is being updated, but it still points to the same link?
> - <td><pre>690a2bb570103e58d12f9806e8bf21be</pre></td>
> + <td><a href="http://fast.dpdk.org/rel/dpdk-16.07.1.tar.xz">DPDK 16.07.1</a></td>
It is not clear that this is the stable branch. Would it make more sense to rename to "DPDK Stable 16.07.1" or similar to make it more obvious this release has backports/fixes?
If there's no concerns about naming or release-notes link, please apply patch as is and I'll push a new patch to center the buttons on the download page.
Regards, -Harry
More information about the web
mailing list