-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
General site updates #40
base: master
Are you sure you want to change the base?
Conversation
fixed a typo in the About blurb
slight update to About blurb, removing text saying that people can access our query database and pop synth web app from the top tabs.
On the new "Papers" tab, how should we handle the ACM Digital Library links (Teng et al. papers)? The ACM Digital Library does not have a publicly accessible library like NASA ADS does. Should we highlight the instrument papers there? |
Rather than adding a new tab for Data, I've collated the code, docs and data into one page accessed via the Code tab. We should discuss whether this is preferable. |
@sgossage what do you need to finish the last two items on your checklist above? |
…the font size on image credits in binary star images of the carousel too
I got the funding info and I've made updates. This is ready for review now. I've put in a placeholder image for the carousel item advertising the v2 paper, which is the grid plot, as discussed. Maybe we'll want to tweak it a bit, but I made some simple edits to get it to fit on the page and not disrupt the site much. The NU Viz credits (which are links) have simply had their font size slightly reduced to draw a bit less attention to them, but we were also considering removing the links. |
Also to note: I've updated the code/docs available for v1 to v1.0.4 (from v1.0.0) Should we have older releases (v1.0.0) recorded too? Maybe it doesn't matter much in this case. The original v1 data set is linked alongside this, but should we have the re-postprocessed data instead or in addition to this? (https://zenodo.org/records/14205146). I suppose this data set is meant for v2 more, so it should be placed alongside v2, rather than v1. For now I've added a link to the re-postprocessed data alongside the v2 code/docs/data |
…nd support and changed navbar code title to get started
Nice improvements! I'd like to point out two things, which still struggle me:
|
Hey Seth, Thank you for the updates! I just had a browse around and I noticed the following few things. I am using Firefox on MacOS:
|
…s to allow responsive scaling
</a> | ||
</div> | ||
|
||
<div class="container-fluid p-4" style="margin: 50px 5px;"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This horizontal margin here causes the horizontal scroll bar, which @maxbriel pointed out as 2.
<p></p> | ||
<h5><a href="https://www.linkedin.com/in/elizabeth-teng/" target="_blank" style="color: rgb(0,0,0)">Elizabeth Teng</a></h5> | ||
<p class="card-text">Northwestern University</p> | ||
</div> | ||
</div> | ||
|
||
</div> | ||
|
||
<br> | ||
|
||
<h2 style="text-align:center">PAST CORE DEVELOPERS</h2> | ||
<hr/> | ||
|
||
<div class="row"> | ||
<div class="col-md align-top col-half-offset"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to set the columns manually? Wouldn't it be better to put all into a multicolumn list? This would avoid misplacements in the future as we currently have: having in the last row only one person in the third column.
<ul style="columns: 4;list-style-type: none;">
<li><a href="https://ch.linkedin.com/in/ssbvr" target="_blank" style="color: rgb(0,0,0)">Simone Bavera </a><\li>
<li><a href="https://www.it.northwestern.edu/departments/it-services-support/research/staff/coughlin.html" target="_blank" style="color: rgb(0,0,0)">Scott Coughlin </a><\li>
...
<li><a href="https://www.ia.forth.gr/people/zapartas" target="_blank" style="color: rgb(0,0,0)">Manos Zapartas</a><\li>
<\ul>
The same for the past collaboration members.
Footnotes
Code, docs, and data have been collated into a card deck on the home page ↩ ↩2 ↩3
Removed "News" tab from the navigator up top ↩
This has been added to the v1.0.4 docs here, still to be added to v2.0.0-pre ↩