-
-
Notifications
You must be signed in to change notification settings - Fork 79
Feature/landing page news section #619
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
Closed
BytePaul
wants to merge
22
commits into
precice:master
from
BytePaul:feature/landing-page-news-section
Closed
Feature/landing page news section #619
BytePaul
wants to merge
22
commits into
precice:master
from
BytePaul:feature/landing-page-news-section
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…cements - Updated news-collect.js to support both local and production environments (replaced hardcoded 127.0.0.1 references with dynamic endpoint detection) - Added short excerpts from the body of each Discourse post - Filtered out pinned and “Welcome to the category” topics - Removed external link marker from “Read more” links for cleaner UI - Resolved CSS conflicts and unified .news-card styles in customstyles-precice.css
… same color and arrow, and removing external link icon
- Added package.json and package-lock.json for Node.js dependencies - Updated 'More news' link to point to the correct Discourse category
- Added .github/workflows/fetch-news.yml to run daily at 6 AM UTC - Updated news-collect.js to display 3 news cards from static JSON - Added tools/fetch-news.js for CI-based Discourse data generation
…s irrelvant for static fetching
…m/BytePaul/precice.github.io into feature/landing-page-news-section
|
Superceded by #625 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR addresses the issue reported in #402 where the Latest News section did not stack correctly on mobile devices and cards remained in a horizontal layout.
Changes Included
Ensured .news-card entries stack vertically on small screens
Adjusted flex behaviour in #news-container to allow wrapping
Increased summary visibility by allowing 4 text lines on mobile
Improved spacing between cards for better readability
Kept desktop layout unchanged
Tested On
Responsive mode in Chrome DevTools