Closed IanLee1521 closed 5 years ago
I like code.nasa.gov!
@hauten now has a design in OneDrive for this task.
@hauten -- We may need to pull apart the .ai
file in order to get the individual images into this repo, but I'll leave that for Angela to head up and work with you on.
In case my edits above don't make it obvious, the design has been slightly updated with descriptions of the 6 icons and a couple of other small adjustments. New file in OneDrive has "final" in file name. @IanLee1521 I'll make sure Angela has what she needs.
I've finished the layout for the new home page, making catalog it's own sub-page. Right now the repos being pulled in per category are from randomly selected topics. These topics can be edited in the /categories/category_info.json file when a decision is made as to what topic each category will be.
I have not implemented a "more" feature to handle a large amount of repos under a topic, this will need to be brainstormed more (maybe each category can lead to its own "page" just as each repo in the catalog does?). Also, it seems that repo name alone is not a unique identifier, so maybe we should display "repo name/repo owner" on the homepage?
All these changes are in the "new-home-page" branch
Looks good!
I took a quick look at this - for repos that are forks or mirrors (e.g. mfem, glvis) it probably makes sense to only display them once.
Visually, I think the category logos should be smaller, and the repository names larger.
Yes, the owner/name combo is what is needed for a unique identifier. I think the order you display that info depends on how you want to format it. (E.g. Most of the actual data uses owner/name
as GitHub itself does, but the catalog cards and repo pages show the name
more prominently with the owner
as a sort of subtitle.)
I think in most cases where name
is duplicated, we want the repo that is not LLNL/name
.
@angfl97 You may want to update the data promises in category-info.js
. I recently re-worked the other pages' data promises to make them less convoluted. They now use a promise variable directly instead of a promise inside a function inside a variable. (Yikes!)
repo-dynamic.js
should be a good example.
@davidbeckingsale I'm not sure that is always going to be true, but might be a good first cut... I think we may need to revisit why repos are forking into LLNL if they live on their own, and if they should be.
@angfl97 -- Could you start a pull request for this change? That way we can comment and make changes collaboratively as needed. You might call it something like "WIP: New Homepage" or similar, where WIP
== Work in Progress
.
@IanLee1521 got it!
Updated/revised decisions on 6/19 with @IanLee1521 & @angfl97:
Today's discussion - for portal home & /radiuss unless noted otherwise
We should improve the UI look and feel, including providing good metrics visibility
Some ideas:
Tasklist as of July 2: