Skip to content

second try for commit of BIDS diagram#47

Open
gbezold1 wants to merge 2 commits intomainfrom
BIDS-mindmap
Open

second try for commit of BIDS diagram#47
gbezold1 wants to merge 2 commits intomainfrom
BIDS-mindmap

Conversation

@gbezold1
Copy link

@gbezold1 gbezold1 commented May 8, 2025

--> new PR from branch rather than forked repo
--> from @NEStock on last PR

  • Suggestion: Add link via tabs

Currently this page is only available the direct URL, but I think it would be great to make it more accessible to users. Any thoughts on where a good place would be? Perhaps a dropdown one of the existing tabs?

in my opinion, a dropdown tab under "metadata" feels like the closest fit but not quite right. one idea might be changing metadata -> "Data Model" and then adding a tab underneath for "BIDS" or something.

Definitely open to other suggestions!

@guittari curious if you have thoughts on this as well

-it might also make sense to add some details and/or links about:

what is BIDS & why is it import for EMBER
link/reference to the original diagram

  • added new yarn lock for this PR!
  • fixed height
  • renamed direct URL to BIDS-mindmap

@gbezold1 gbezold1 requested review from NEStock and guittari May 8, 2025 18:54
@netlify
Copy link

netlify bot commented May 8, 2025

Deploy Preview for emberarchive ready!

Name Link
🔨 Latest commit e82f4b0
🔍 Latest deploy log https://app.netlify.com/sites/emberarchive/deploys/681cfe8d6b798100082f29c3
😎 Deploy Preview https://deploy-preview-47--emberarchive.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant