Skip to content

Conversation

@opott
Copy link
Member

@opott opott commented Jul 19, 2025

Eliminate the loading overlay from the layout files and include SonarCloud quality badges for better visibility of project health.

Copilot AI review requested due to automatic review settings July 19, 2025 13:23
@vercel
Copy link

vercel bot commented Jul 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
coursenotes-jekyll ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 19, 2025 1:23pm

Copy link

Copilot AI left a 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 removes the loading overlay functionality from the website layouts and adds SonarCloud quality badges to the README for better project health visibility.

  • Removes the loading overlay component and its associated JavaScript from all layout files
  • Adds comprehensive SonarCloud badges to display code quality metrics in the README
  • Consolidates footer includes on a single line in the page layout

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
_layouts/page.html Removes loading overlay include and consolidates footer includes
_layouts/index.html Removes loading overlay include from header section
_layouts/course_home.html Removes loading overlay include from header section
_includes/loading.html Completely removes the loading overlay HTML component
_includes/head.html Removes JavaScript that controlled loading overlay visibility
README.MD Adds seven SonarCloud quality badges for project health monitoring

@sonarqubecloud
Copy link

@opott opott merged commit 9ab008d into main Jul 19, 2025
4 checks passed
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.

2 participants