Conversation
|
🚀 Preview your site is ready: |
rosecers
force-pushed
the
rosecers-patch-3
branch
4 times, most recently
from
c31434a to
d3bb04f
Compare
rosecers
force-pushed
the
rosecers-patch-3
branch
from
d684991 to
c1b42b0
Compare
rosecers
force-pushed
the
rosecers-patch-3
branch
from
dc60365 to
940ac31
Compare
rosecers
force-pushed
the
rosecers-patch-3
branch
from
1bd18fd to
55fa773
Compare
rosecers
force-pushed
the
rosecers-patch-3
branch
from
732c18b to
36da759
Compare
arthur-lin1027
left a comment
arthur-lin1027
left a comment
There was a problem hiding this comment.
I think this can be somewhat improved by changing the styling and reducing the bolding. Maybe we can follow the ACS style guide? https://pubs.acs.org/doi/full/10.1021/acsguide.40303
Some examples:
Shuwen: https://yuegroup.cbe.cornell.edu/papers/
Michele: https://www.epfl.ch/labs/cosmo/index-html/publications/
Voth: https://voices.uchicago.edu/vothgroup/publications/
.github/workflows/publications.yml
Outdated
| python-version: '3.x' | ||
|
|
||
| - name: Install dependencies | ||
| run: pip install requests |
There was a problem hiding this comment.
Do we need to also pip install pyzotero?
There was a problem hiding this comment.
I think Charles and I both think there's too much bolded text in each entry, it's a bit visually striking.
Maybe consider italicizing the journal name rather than bolding it? Or, we could follow the formatting of a specific style guide like ACS (https://pubs.acs.org/doi/full/10.1021/acsguide.40303)
|
@arthur-lin1027 @cbefan, check the new preview (https://cersonsky-lab.github.io/website-previews/pr-73/publications/) and see if it's to your liking. |
|
I like it! |
|
Looks good to me, although I agree bolding the titles might be a bit much. Maybe just bold group members' names? I think it's good regardless. |
3 participants
No description provided.