-
Notifications
You must be signed in to change notification settings - Fork 0
Replace homepage text content with large welcome banner image #17
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
Replace homepage text content with large welcome banner image #17
Conversation
Co-authored-by: JacobKMcPherson <[email protected]>
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.
Pull Request Overview
This PR replaces the text-heavy McPherson Group homepage with a large welcome banner image, transforming it from detailed content sections to a single visual element that welcomes visitors.
- Removed all existing text content including research focus, news updates, and contact information
- Added a custom welcome banner image with responsive CSS styling
- Simplified the homepage to focus on visual impact rather than informational content
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
index.qmd | Replaced 20+ lines of text content with a single welcome banner image reference |
styles.css | Added responsive styling for the welcome banner with rounded corners and shadow effects |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
} | ||
|
||
/* Welcome banner styling for homepage */ | ||
.quarto-title + p img { |
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.
The CSS selector .quarto-title + p img
is fragile and relies on specific HTML structure generated by Quarto. Consider using a more semantic class-based selector like .welcome-banner
on the image element to make the styling more maintainable and less dependent on the generated DOM structure.
.quarto-title + p img { | |
.welcome-banner { |
Copilot uses AI. Check for mistakes.
## Contact | ||
|
||
For inquiries about our research or potential collaborations, please visit our [About](about.qmd) page for contact information. | ||
 |
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.
The alt text 'Welcome to the McPherson Group' is too generic for a welcome banner image. Consider more descriptive alt text that conveys the image content and context, such as 'McPherson Group welcome banner with gradient background and research mission statement'.
 | |
 |
Copilot uses AI. Check for mistakes.
This PR transforms the McPherson Group homepage from text-heavy content to a visually appealing welcome banner, addressing the request to have a large placeholder welcoming image instead of words.
Changes Made
Visual Transformation
welcome-banner.png
) featuring:Technical Implementation
index.qmd
: Simplified from 24 lines of content to a single image referencestyles.css
: Added responsive styling for the welcome banner including:Result
The homepage now provides an immediate visual impact with a clean, professional appearance that effectively communicates the group's identity and mission. The banner serves as an engaging entry point while maintaining the site's existing design language.
The new design creates a strong first impression for visitors while keeping the implementation lightweight (22KB image) for fast loading times.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.