Created System Monitor pages#23
Conversation
✅ Deploy Preview for daniel-herr-software ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for daniel-herr-software-forms ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Visit the preview URL for this PR (updated for commit 8e0a477): https://daniel-herr-software--pr23-dev-jyii4z7n.web.app (expires Fri, 15 May 2026 22:33:49 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 8c152685cc1a60c2381e223a321e638cf5e1d307 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces the "System Monitor" Chrome App by adding a product page, an uninstallation feedback form, and updating the main index. Review feedback identifies a redundant closing tag in the feedback form and recommends correcting the heading hierarchy and indentation in the changelog to improve accessibility and maintain code consistency.
| </label><br> | ||
| </label> |
There was a problem hiding this comment.
There is a redundant closing </label> tag on line 53. Removing it ensures the HTML is well-formed and prevents potential rendering issues.
</label><br>| <ol reversed> | ||
| <li> | ||
| <h2>4.0 - May 8, 2026</h2> | ||
| <ul> | ||
| <li>Rename from Monitor Panel</li> | ||
| <li>Added full view UI surface with detailed CPU, RAM, battery, display, and network information</li> | ||
| <li>Moved settings directly into app instead of relying on Memory/Processor Monitor extensions settings</li> | ||
| <li>Combined CPU and RAM floating panels and added battery</li> | ||
| <li>Licensed app as GPLv3</li> | ||
| </ul> | ||
| </li> | ||
| </ol> |
There was a problem hiding this comment.
The indentation of the changelog section is inconsistent with the rest of the file. Additionally, the version header should be changed from <h2> to <h4> to maintain a correct heading hierarchy under the <h3>Changelog</h3> section, which improves accessibility and SEO.
| <ol reversed> | |
| <li> | |
| <h2>4.0 - May 8, 2026</h2> | |
| <ul> | |
| <li>Rename from Monitor Panel</li> | |
| <li>Added full view UI surface with detailed CPU, RAM, battery, display, and network information</li> | |
| <li>Moved settings directly into app instead of relying on Memory/Processor Monitor extensions settings</li> | |
| <li>Combined CPU and RAM floating panels and added battery</li> | |
| <li>Licensed app as GPLv3</li> | |
| </ul> | |
| </li> | |
| </ol> | |
| <ol reversed> | |
| <li> | |
| <h4>4.0 - May 8, 2026</h4> | |
| <ul> | |
| <li>Rename from Monitor Panel</li> | |
| <li>Added full view UI surface with detailed CPU, RAM, battery, display, and network information</li> | |
| <li>Moved settings directly into app instead of relying on Memory/Processor Monitor extensions settings</li> | |
| <li>Combined CPU and RAM floating panels and added battery</li> | |
| <li>Licensed app as GPLv3</li> | |
| </ul> | |
| </li> | |
| </ol> |
1 participant
No description provided.