Refresh Gridsuite homepage with 2026 data#4
Open
antoinebhs wants to merge 6 commits into
Open
Conversation
📝 WalkthroughWalkthroughUpdates README to point to a separate documentation repository, adds three new site configuration URLs (GitHub organization, demo, release), restructures the header layout to a CSS grid using these new URLs, and revises homepage news, project status, and links accordingly. ChangesSite links and content update
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (3)
_layouts/default.html (2)
19-25: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winAdd
alttext to the favicon image.The
<img>at Line 20 has noaltattribute, so screen readers will announce the raw file path instead of meaningful content.♿ Proposed fix
- <img src="/favicon.ico" style="height:125px;"> + <img src="/favicon.ico" alt="GridSuite logo" style="height:125px;">🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@_layouts/default.html` around lines 19 - 25, The favicon image in the default layout is missing an alt attribute, so update the <img> element in the header block to include meaningful alternative text that describes the site/logo rather than the file path. Keep the change localized to the image inside the main title/header markup in the default layout.
19-36: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueInline styles reduce maintainability; verify mobile layout.
The header grid is defined entirely via inline
styleattributes with fixed pixel widths (80px,900px). Consider moving this to the site stylesheet for reuse/theming. Also worth confirming this fixed 3-column grid degrades gracefully on narrow viewports, since there's no accompanying media query.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@_layouts/default.html` around lines 19 - 36, The header markup in the default layout is using inline styles with fixed widths, which hurts maintainability and may not adapt well on small screens. Move the grid and image sizing from the template in the default layout into the shared stylesheet, using a reusable class for this header block, and add responsive rules (media query or equivalent) so the three-column layout collapses or stacks gracefully on narrow viewports.index.md (1)
34-43: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winUse
site.demo_urlandsite.github_organization_urlinstead of hardcoding.Line 35 hardcodes the demo URL, and Line 43 hardcodes the GitHub org URL, even though
_config.ymlnow definessite.demo_urlandsite.github_organization_urlfor exactly this purpose (Releases already uses{{ site.release_url }}on Line 41). This duplication makes future URL changes error-prone.♻️ Proposed fix
## Online Demo -Please visit [https://demo.gridsuite.org/](https://demo.gridsuite.org/) to get your own impressions of using GridSuite apps! You can use any Microsoft account (such as private Outlook accounts, GitHub accounts, or anything Microsoft). +Please visit [{{ site.demo_url }}]({{ site.demo_url }}) to get your own impressions of using GridSuite apps! You can use any Microsoft account (such as private Outlook accounts, GitHub accounts, or anything Microsoft). ## Stay up to date * Releases: [GridSuite deployment releases]({{ site.release_url }}) * Documentation: [GridSuite documentation](https://www.gridsuite.org/documentation/) -* Development: [GitHub organization](https://github.com/gridsuite/) +* Development: [GitHub organization]({{ site.github_organization_url }})🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@index.md` around lines 34 - 43, The links in the “Online Demo” and “Stay up to date” sections are hardcoded instead of using the site config values. Update the Markdown in index.md to reference the existing config symbols site.demo_url and site.github_organization_url, matching the pattern already used for site.release_url, so URL changes stay centralized and consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@_layouts/default.html`:
- Around line 19-25: The favicon image in the default layout is missing an alt
attribute, so update the <img> element in the header block to include meaningful
alternative text that describes the site/logo rather than the file path. Keep
the change localized to the image inside the main title/header markup in the
default layout.
- Around line 19-36: The header markup in the default layout is using inline
styles with fixed widths, which hurts maintainability and may not adapt well on
small screens. Move the grid and image sizing from the template in the default
layout into the shared stylesheet, using a reusable class for this header block,
and add responsive rules (media query or equivalent) so the three-column layout
collapses or stacks gracefully on narrow viewports.
In `@index.md`:
- Around line 34-43: The links in the “Online Demo” and “Stay up to date”
sections are hardcoded instead of using the site config values. Update the
Markdown in index.md to reference the existing config symbols site.demo_url and
site.github_organization_url, matching the pattern already used for
site.release_url, so URL changes stay centralized and consistent.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: bade5d6f-04db-4091-b6b3-c56744839c7a
⛔ Files ignored due to path filters (14)
demo/v0.2.0/add-button.gifis excluded by!**/*.gifdemo/v0.2.0/all-none.gifis excluded by!**/*.gifdemo/v0.2.0/apps-menu.gifis excluded by!**/*.gifdemo/v0.2.0/center-map.gifis excluded by!**/*.gifdemo/v0.2.0/country-info.gifis excluded by!**/*.gifdemo/v0.2.0/fix-refresh.gifis excluded by!**/*.gifdemo/v0.2.0/ieee14.gifis excluded by!**/*.gifdemo/v0.2.0/pan-substation-view.gifis excluded by!**/*.gifdemo/v0.3.0/fix-center-on-substation.gifis excluded by!**/*.gifdemo/v0.3.0/fullscreen.gifis excluded by!**/*.gifdemo/v0.3.0/rename.gifis excluded by!**/*.gifdemo/v0.3.0/settings-window.gifis excluded by!**/*.gifdemo/v0.3.0/transformers.gifis excluded by!**/*.gifimages/architecture.svgis excluded by!**/*.svg
📒 Files selected for processing (4)
README.md_config.yml_layouts/default.htmlindex.md
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
PR Summary