Repository navigation
Add files via upload - #4
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new HTML landing page for SecureArchive documentation, providing a modern, styled interface to showcase the encryption tool's features and capabilities.
Key Changes:
- Addition of a new standalone HTML documentation page with embedded CSS styling
- Integration of branding elements including banner graphics and gradient effects
- Feature cards highlighting AES-256-GCM encryption, secure containers, GUI/CLI support, and zero-knowledge architecture
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| <div class="buttons"> | ||
| <a href="https://github.com/bylickilabs/SecureArchive/releases" class="btn download">⬇ Download SecureArchive</a> | ||
| <a href="docs/index.html" class="btn docs">📘 Documentation</a> |
There was a problem hiding this comment.
The link references 'docs/index.html' which does not exist in the repository. Since this HTML file is already located in the 'docs/' directory, the path should be either 'index.html' (if an index.html file exists) or updated to point to an existing documentation file.
| <a href="docs/index.html" class="btn docs">📘 Documentation</a> | |
| <a href="index.html" class="btn docs">📘 Documentation</a> |
| <li><a href="SECURITY.md">Security Architecture</a></li> | ||
| <li><a href="docs/ARCHITECTURE.md">System Architecture</a></li> | ||
| <li><a href="docs/CONTAINER_FORMAT.md">Container Specification</a></li> | ||
| <li><a href="docs/INSTALLATION.md">Installation Guide</a></li> | ||
| <li><a href="docs/CHANGELOG.md">Changelog</a></li> |
There was a problem hiding this comment.
The relative paths in these links are inconsistent. Since this HTML file is in the 'docs/' directory, line 170 should use '../SECURITY.md' to reference the root-level file, and lines 171-174 should use relative paths without the 'docs/' prefix (e.g., 'ARCHITECTURE.md', 'CONTAINER_FORMAT.md', etc.).
| <li><a href="SECURITY.md">Security Architecture</a></li> | |
| <li><a href="docs/ARCHITECTURE.md">System Architecture</a></li> | |
| <li><a href="docs/CONTAINER_FORMAT.md">Container Specification</a></li> | |
| <li><a href="docs/INSTALLATION.md">Installation Guide</a></li> | |
| <li><a href="docs/CHANGELOG.md">Changelog</a></li> | |
| <li><a href="../SECURITY.md">Security Architecture</a></li> | |
| <li><a href="ARCHITECTURE.md">System Architecture</a></li> | |
| <li><a href="CONTAINER_FORMAT.md">Container Specification</a></li> | |
| <li><a href="INSTALLATION.md">Installation Guide</a></li> | |
| <li><a href="CHANGELOG.md">Changelog</a></li> |
| font-weight: 600; | ||
| border-radius: 10px; | ||
| text-decoration: none; | ||
| color: white; |
There was a problem hiding this comment.
[nitpick] Using 'white' as a hardcoded color value is less maintainable. Consider using '#ffffff' or a CSS variable for consistency with the rest of the stylesheet where hex colors are used (e.g., #e5e7eb, #9ca3af).
| color: white; | |
| color: #ffffff; |
No description provided.