Skip to content

Update template site homepage#52

Merged
egrace479 merged 7 commits intodevfrom
dev-45
Jul 25, 2025
Merged

Update template site homepage#52
egrace479 merged 7 commits intodevfrom
dev-45

Conversation

@egrace479
Copy link
Copy Markdown
Member

@egrace479 egrace479 commented Jun 30, 2025

  • Adds the updated Collaborative Infrastructure Diagram (Update Collaborative Infrastructure Diagram #49).
  • Refocuses the Imageomics-specific parts to be Imageomics and ABC.
  • Adds link to About the Guide Section of the README for personalization and interaction instructions.

Closes #45 and #49.

egrace479 added 3 commits June 13, 2025 17:46
@egrace479
Rename diagram image to be descriptive
31653dc 
it will be updated
@egrace479
Ref #49: redesign collab infrastructure diagram
d8365c2
@egrace479
Add joint components to template site homepage
c0dfbbf 
* collaborative infrastructure from both

* intro linking to 'About this Guide' for more context and personalization instructions

* shared logo
@egrace479 egrace479 added this to the Generalize Guide milestone Jun 30, 2025
@egrace479 egrace479 requested review from gwtaylor and hlapp June 30, 2025 23:05
@egrace479 egrace479 added design UX or presentation needs attention documentation Improvements or additions to documentation labels Jun 30, 2025
gwtaylor added 3 commits July 23, 2025 18:11
@gwtaylor
style: fix typo and markdownlint issues in homepage
c0f93c2 
- Fix typo: '<Ogranization>' → '<Organization>' in branding section
- Remove trailing punctuation from main heading (MD026)
- Add blank lines around headings (MD022)
- Add blank line around lists (MD032)
- Remove inline HTML <br> tags (MD033)
- Remove trailing whitespace (MD009)

All markdownlint rules now pass for docs/index.md
@gwtaylor
docs: improve homepage content flow and clarity
7576249 
- Split dense introductory paragraph into focused sections
- Clarify scope as 'imageomics and related interdisciplinary fields'
- Move template customization info to separate, less prominent paragraph
- Change 'imageomics-related terms' to 'scientific terms' for broader consistency
- Improve readability and reduce user confusion about template vs guide usage
@gwtaylor
chore: add site/ to .gitignore
e22b0a0 
MkDocs build output directory should not be tracked in version control
gwtaylor
gwtaylor approved these changes Jul 23, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@gwtaylor gwtaylor left a comment

Choose a reason for hiding this comment

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

Hi @egrace479,
This looks great! I've reviewed the homepage updates for the template transition and made some edits as additional commits.

Changes Made

Style & Quality Improvements

  • Fixed typo: <Ogranization> → <Organization> in the branding section
  • Applied markdownlint fixes: Resolved all formatting violations (heading punctuation, blank lines, inline HTML)
  • Improved content flow: Split the dense introductory paragraph for better readability
  • Enhanced clarity: Positioned scope as "imageomics and related interdisciplinary fields" and moved template meta-info to separate paragraph

Repository Maintenance

  • Added site/ to .gitignore: MkDocs build output should not be tracked

Outstanding Issue (Not Blocking)

⚠️ Missing logo file: The referenced logos/Imageomics_ABC.png doesn't exist yet. Based on issues #45 and #17, this is an ongoing design decision rather than a blocker. I actually see a version of this logo added to a comment in #17 so perhaps it is just a matter of adding it to the proper location in the repo.

The template placeholder syntax is appropriate until the logo approach is finalized.

Technical Verification

✅ All markdownlint rules pass
✅ MkDocs builds successfully
✅ Links functional
✅ No new rendering issues

The homepage updates successfully implement the joint Imageomics/ABC approach from issue #45 while maintaining the template structure for future forking.

gwtaylor
gwtaylor reviewed Jul 23, 2025
View reviewed changes
gwtaylor
gwtaylor reviewed Jul 23, 2025
View reviewed changes
@egrace479
Copy link
Copy Markdown
Member Author

egrace479 commented Jul 24, 2025

Outstanding Issue (Not Blocking)

⚠️ Missing logo file: The referenced logos/Imageomics_ABC.png doesn't exist yet. Based on issues #45 and #17, this is an ongoing design decision rather than a blocker. I actually see a version of this logo added to a comment in #17 so perhaps it is just a matter of adding it to the proper location in the repo.

The template placeholder syntax is appropriate until the logo approach is finalized.

The updated logo was added in #50, but it seems I didn't merge dev into this branch after that update.

@egrace479
Re-add '!' and allow in header
5b7f909 
allowing generally simplifies updates for downstream users
hlapp
hlapp approved these changes Jul 25, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@hlapp hlapp left a comment

Choose a reason for hiding this comment

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

👍🎉 and as per online discussion.

@egrace479 egrace479 merged commit a162243 into dev Jul 25, 2025
@egrace479 egrace479 deleted the dev-45 branch July 25, 2025 21:59
This was referenced Jul 25, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gwtaylor gwtaylor gwtaylor approved these changes

@hlapp hlapp hlapp approved these changes

Assignees

No one assigned

Labels

design UX or presentation needs attention documentation Improvements or additions to documentation

Projects

None yet

Milestone

Generalize Guide

Development

Successfully merging this pull request may close these issues.

3 participants

@egrace479 @hlapp @gwtaylor