feat: Comprehensive UI/UX overhaul with INI configuration system, monitoring endpoints, and unified light button design - #5
Merged
Conversation
- Implement comprehensive INI parser supporting file sizes, booleans, and lists - Add shared base.css design system for consistent template styling - Update template variables for consistency (STATUS_CODE → ERROR_CODE) - Add comprehensive test coverage for configuration file functionality - Enable flexible configuration through CLI args > INI file > defaults hierarchy IronDrop can now be configured via INI files in addition to CLI arguments.
• Add file type-specific Bootstrap Icons (file, folder, zip, image, video)
• Implement smart file type detection based on extension
• Replace logo text with logo image across all pages
• Move logo styling to base CSS for consistency
• Add conditional template support ({{#if}}/{{/if}})
• Create new template handler system (handlers.rs, router.rs, middleware.rs)
• Update static asset paths to /_irondrop/static/ namespace
• Fix icon backgrounds and improve visual consistency
• Add comprehensive template system documentation
• Enhance upload functionality with proper path handling
…s and config changes
- Remove trailing slashes from directory titles while preserving leading slashes - Update upload integration tests to use correct /_irondrop/upload endpoint - Fix upload directory configuration in test helpers - Update test expectations to match new clean title format
• Convert dark buttons to light buttons across all templates • Implement unified .card class for consistent styling • Reduce shadow depth for minimal modern appearance • Fix clippy warnings and update documentation
- Update comprehensive_test.rs: Change button class from 'error-button' to 'btn btn-light' - Update monitor_test.rs: Fix endpoint URLs from '/monitor' to '/_irondrop/monitor' - Update template_embedding_test.rs: Change assertion from 'back-link' to 'Go Back' text - All tests now pass with the new light button system and unified card architecture - Tests verify proper integration of monitor endpoints and error page styling
dev-saw99
changed the title
dev-harsh1998
force-pushed
the
ss-upload
branch
2 times, most recently
from
August 9, 2025 11:23
ffdfdc0 to
f5ecd0e
Compare
Signed-off-by: Harshit Jain <reach@harsh1998.dev>
Signed-off-by: Harshit Jain <reach@harsh1998.dev>
…ase warnings - Added std::mem import for Windows target to fix unresolved module errors - Removed unused std::ptr import to fix warning - Added #[allow(non_snake_case)] to PROCESS_MEMORY_COUNTERS struct The PROCESS_MEMORY_COUNTERS struct must maintain Windows API field naming convention (PascalCase) for FFI compatibility. Converting to snake_case would break the foreign function interface with Windows system APIs. The struct fields must exactly match the Windows header definitions.
dev-harsh1998
force-pushed
the
ss-upload
branch
from
August 9, 2025 11:28
f5ecd0e to
df8cb45
Compare
dev-harsh1998
approved these changes
Aug 9, 2025
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
2 participants
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.
Uh oh!
There was an error while loading. Please reload this page.