feat/redesign-and-improvements#20
Conversation
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Moved i18n access to function call time instead of store definition time - Added error handling with a fallback message - Called getInitialMessage() when actually needed in initiateConversation()
a-klos
left a comment
There was a problem hiding this comment.
friendly reminder^^:
- indentation
- dark mode default
- something else that i most probably forgot :D
file and sitemap loader still work. confluence not tested... its not configured in the uni deployment ^^
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Added features.useMockData to support mock API configuration - Properly handles VITE_USE_MOCK_DATA environment variable - Fixes TypeScript error in document.api.factory.ts
|
@a-klos FIY: just fixed pipeline and add your suggestions. Not tested it yet on the system. |
There was a problem hiding this comment.
Pull Request Overview
This PR introduces environment-based configuration, theming improvements, and enhanced document management for the frontend application. The changes enable better customization through environment variables and provide a more polished user experience.
- Add environment-based settings system with support for bot name, logo path, and theme configuration
- Implement comprehensive light/dark theme system with user preference persistence
- Enhance document management UI with improved interactions and mock API support
Reviewed Changes
Copilot reviewed 22 out of 24 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.base.json | Add path aliases for new shared settings and theme store modules |
| tailwind.config.js | Expand theme configuration with comprehensive light/dark color schemes |
| ThemeToggle.vue | New component for theme switching with sun/moon icons |
| NavigationContainer.vue | Updated to use configurable logo path and include theme toggle |
| theme.store.ts | New Pinia store for theme management with localStorage persistence |
| settings.ts | New configuration system with environment variable support and fallbacks |
| global.css | Add theme-specific chat text styles and avatar improvements |
| README.md | Document new environment variables and customization options |
| DocumentItem.vue | Enhanced document item UI with confirmation modal and improved status display |
| document.api.mock.ts | New mock implementation for document API with realistic data |
| document.api.factory.ts | Factory pattern for selecting real vs mock API implementation |
| documents.store.ts | Code formatting improvements and better error handling |
| App.vue files | Theme store initialization and code formatting updates |
| UI_Customization.md | Comprehensive documentation for UI customization features |
- Introduced `navigation-logo-dark.svg` with detailed path definitions for dark theme. - Introduced `navigation-logo-light.svg` with detailed path definitions for light theme.
fix(tailwind): swap secondary color values for better contrast
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 33 out of 42 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
improve bot name placeholder handling in chat messages for better reliability
Overview
This PR introduces several improvements to the frontend application, focusing on customization options and document handling. The main changes include environment-based configuration, enhanced document interactions, and visual refinements.
Changes
Environment-Based Settings Configuration
.envfiles using the VITE_ prefixDocument Experience Improvements
Visual Updates
How to Test
.envfile with custom settings:Screenshots
tbd