Code review comment for ~sowasred2012/cloud-images/+git/aws-eks-website:redesign

Revision history for this message
Cody Shepherd (codyshepherd) wrote :

Thanks Scott, this is great! A couple of requests:

1. Could we make the toggle between 1.11 and 1.12 (and future versions) more obvious in terms of styling?

2. Could we consolidate the date column to a single field, perhaps a subtitle or something similar at the top of the table? We do not expect to encounter circumstances where the AMIs listed in a table have different dates.

3. This is not strictly within the scope of our original proposal, but we are beginning to use some automated scripts to update the AMI listings on this page. Is there a way to add some type of tag in the html to each line that lists an AMI link, making it easier to identify those lines via a script? This is not a blocker, just thought I'd ask :)

review: Needs Fixing

« Back to merge proposal