Skip to content

Conversation

@julia-pfarr
Copy link
Member

@julia-pfarr julia-pfarr commented Oct 1, 2025

I made a first draft for the modified landing page. Please give input on

  • color
  • content
  • spacing and other details

Also, since I am not super skilled with html, please review the code and refactor. I guess there's also a lot in the code that is actually not needed/used or could be simplified. I didn't do a proper code review in the end because I am just happy that I got this far at all :-D

I am also imagining some nice figure between the "in a nutshell" and the "stats" section, does anyone have an idea?


📚 Documentation preview 📚: https://bids-website--725.org.readthedocs.build/en/725/

@julia-pfarr
Copy link
Member Author

I don't know why it doesn't show the background image and the images in the sponsors section, it works in my local build

@effigies
Copy link
Contributor

effigies commented Oct 1, 2025

image

https://bids-website--725.org.readthedocs.build/en/assets/img/logos/NIH.png instead of https://bids-website--725.org.readthedocs.build/en/725/assets/img/logos/NIH.png

Not sure how to fix that...

Copy link
Contributor

@effigies effigies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possibly just:

Co-authored-by: Chris Markiewicz <[email protected]>
@julia-pfarr
Copy link
Member Author

nice, this worked, thanks @effigies

Copy link
Contributor

@Remi-Gau Remi-Gau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did a quick first pass

mkdocs.yml Outdated
Comment on lines 247 to 277
- includes/abbreviations.md
- includes/mkdocs.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you sure about this one?

I don't think mkdocs.md exist and if we do not point to this:
https://github.com/bids-standard/bids-website/blob/main/includes/abbreviations.md?plain=1

We lose links and abbreviations "tool-tips" that are defined for the whole website.

Image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, not sure about this one :-D not sure about any of what I did haha

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not a big fan of ducplicating files that we can already exist from the specification sudmodule, maybe just copy it during the build process? or symlink to it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I haven't been working with submodules that much, so didn't know about that. Will change.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Remi-Gau I tried it with the submodule, but the logo doesn't show in the preview. Could you make a suggestion for how to solve this?

</div>
</section>

<!-- Stats/Social proof section -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think for those we could maybe add links to the other relevant sections of the website

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You mean that if I click on "2000+ public datasets", for example, it should direct me to the "Datasets" section of our BIDS website?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes exactly

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wonder if we could use scss to make it easier to navigate the css overall, seems that mkdocs material does it but it is not mentioned in the mkdocs so I figure we may have to generate the css at build time

https://github.com/squidfunk/mkdocs-material/blob/master/src/overrides/assets/stylesheets/custom.scss

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe make a new issue/PR for this? I've never done it so it could take some time and since the current solution works, we could go with that for now

margin-left: 0 !important;
}

/* Mobile adjustments */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there are some issues with the look on tablet and mobile but I would leave this kind of tweaking to a follow up PR.

Image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to make the review easier we could keep all the @media adjustements for a follow up PR

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, good idea. And yes, I haven't been paying attention to tablet and phone adjustments, just focused on keeping the stuff going for laptop

Copy link
Contributor

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it. 👏
I'd go with blue or green for easier on the eyes, but no objection to the red if everyone loves it.

Copy link
Member Author

@julia-pfarr julia-pfarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix navigation sidebars

@julia-pfarr
Copy link
Member Author

@ericearl @bclenet @christinerogers I think I addressed all your comments now, could you please check? If yes, @christinerogers could you approve because you clicked "request changes" so merging is blocked until you approve ;-)

@bclenet
Copy link
Contributor

bclenet commented Nov 27, 2025

Hi @julia-pfarr,
I just realized that the two links I suggested (for contributors and BIDS apps) should not be used as is because they are consistent on https://bids.neuroimaging.io/ only (not on the PR previews for example). I'll make a new review accordingly.

Copy link
Contributor

@bclenet bclenet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two fixes for the contributors and bids apps links on landing page.

@Remi-Gau
Copy link
Contributor

unresolving some discussions to create my own TODO list

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copy the logo from the spec during the build

Comment on lines 3 to 19
--mri: #a6cee3;
--pet: #1f78b4;
--micr: #33a02c;
--meeg: #6a3d9a;
--beh: #fb9a99;
--nirs: #b2df8a;
--motion: #e31a1c;
--mrs: #fdbf6f;
--unused2: #ff7f00;
--unused3: #cab2d6;
--unused4: #ffff99;
--unused5: #b15928;
--key: #e41a1c;
--label: #377eb8;
--suffix: #4daf4a;
--ext: #984ea3;
--underscore: #ff7f00;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to keep those otherwise we lose the colors on in several places in the getting starting guide:

in this PR:

image

on main:

image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should add some formating to css because it's harder to track changes otherwise: for a later PR

- content.code.copy
- content.tabs.link
- content.tooltips
- footnotes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move to the extension section (it's not a feature)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gonna move most of those changes to a separate PR because I think that even if some of them can be useful I think the vast majority are orthogonal to the goal of the current PR

@Remi-Gau
Copy link
Contributor

Remi-Gau commented Nov 29, 2025

there are a few contrast issues with the color chosen: will try to tweak to improve

checking contrast on firefox: open the developer tools -> click Accessibility in the developer tools menu -> finally select 'contrast' in the 'check issue' dropdown.

especially in dark mode

image

EDIT

after fixing

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants