-
Notifications
You must be signed in to change notification settings - Fork 6
Homepage and menu design improvements #292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 32 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
1ea0757
Update design_guidelines.md
Aamirrafiqgfx c970996
Updated .gitignore
Aamirrafiqgfx a079630
Updated vite.config.ts
Aamirrafiqgfx 4b46e65
Updated vite.config.ts
Aamirrafiqgfx 50fc037
Added themeicon.switch
Aamirrafiqgfx 225da62
Added themeicon.svg
Aamirrafiqgfx 0b0c239
Added header-minimize.svg
Aamirrafiqgfx ed0b99f
Added demo-button-play.svg
Aamirrafiqgfx abece4e
Added search-icon.svg
Aamirrafiqgfx c7dbaf4
Added side-panel-switch-icon.svg
Aamirrafiqgfx 78bf2f6
Added icon.png
Aamirrafiqgfx a67d54b
Updated Header.tsx
Aamirrafiqgfx 15a0139
Updated header-minimize.svg
Aamirrafiqgfx 0d136b7
Updated themeicon.svg
Aamirrafiqgfx fd43a9f
Updated demo-button-play.svg
Aamirrafiqgfx cd22edc
Updated search-icon.svg
Aamirrafiqgfx 236b1d0
Updated side-panel-switch-icon.svg
Aamirrafiqgfx ae37c4b
Updated Header.tsx
Aamirrafiqgfx 79b2ca5
Updated Header.tsx
Aamirrafiqgfx 367f99f
Updated Header.tsx
Aamirrafiqgfx 906729e
Updated Header.tsx
Aamirrafiqgfx 71bd2cd
Updated icon.png
Aamirrafiqgfx 68bb4c3
Updated Layout.tsx
Aamirrafiqgfx 465b0b5
Updated Header.tsx
Aamirrafiqgfx 7dc501c
Updated vite.config.ts
Aamirrafiqgfx 14c4a2c
Added home-v2.tsx
Aamirrafiqgfx 859ca28
Updated vite.config.ts
Aamirrafiqgfx b7e862a
Updated App.tsx
Aamirrafiqgfx 0c088c5
Updated home-v2.tsx
Aamirrafiqgfx 49d759a
Initial commit with updated project code
Aamirrafiqgfx b3fecc8
Resolve merge conflicts by keeping local updated code
Aamirrafiqgfx a01203a
Made all client-requested updates
anasafzaal2005 91cea4c
Deleted All the V1-V2 extra files, extra logo file in the assets, and…
anasafzaal2005 1376851
Now the app inherits system theme preferances behaves according to th…
anasafzaal2005 c0d4846
Added icons for every individual tool.
anasafzaal2005 73ae23d
Refactor code structure for improved readability and maintainability
anasafzaal2005 ac6aa81
Remove unused SVG paths from svg-paths.ts to streamline the codebase
anasafzaal2005 e5eb80e
Enhance Header and Layout components for demo state & Speed managemen…
anasafzaal2005 94a700c
Enhance Header component styling and layout for demo controls; add cl…
anasafzaal2005 4ef2e4f
Refactor Sidebar and tools data to streamline icon management; remove…
anasafzaal2005 c5b9042
Add keyboard shortcuts for theme toggle and sidebar collapse; impleme…
anasafzaal2005 7e0f6a1
Add tooltips to header toggle and menu button for improved accessibility
anasafzaal2005 31bd842
Refactor imports and improve component structure in Header, Layout, S…
anasafzaal2005 536c7c3
Fix button positioning in Layout and update Sidebar icon fallback; ad…
anasafzaal2005 d8d326e
demo.spec.ts tests passed successfully.
Aamirrafiqgfx 3fc0773
menu-toggle.spec.ts & page-title.spec.ts test passed successfully
Aamirrafiqgfx afb8586
search.spec.ts test passed succesfully
Aamirrafiqgfx f7756c1
theme-toggle.spec.ts tests passed successfully
Aamirrafiqgfx b83537b
barcode-generator.spec.ts tests passed successfully.
Aamirrafiqgfx 2906f20
search-replace.spec.ts tests passed successfully
Aamirrafiqgfx ff986b3
Fixed make ci errors in Layout.tsx and textarea.tsx files
anasafzaal2005 bee3d38
removed the pnpm-lock.yaml file
anasafzaal2005 21b7da3
Apply suggestion from @Copilot
spring1843 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,3 +8,4 @@ vite.config.ts.* | |
| test-results | ||
| playwright-report | ||
| attached_assets | ||
| .env | ||
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| import React, { useState } from 'react' | ||
|
|
||
| const ERROR_IMG_SRC = | ||
| 'data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iODgiIGhlaWdodD0iODgiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgc3Ryb2tlPSIjMDAwIiBzdHJva2UtbGluZWpvaW49InJvdW5kIiBvcGFjaXR5PSIuMyIgZmlsbD0ibm9uZSIgc3Ryb2tlLXdpZHRoPSIzLjciPjxyZWN0IHg9IjE2IiB5PSIxNiIgd2lkdGg9IjU2IiBoZWlnaHQ9IjU2IiByeD0iNiIvPjxwYXRoIGQ9Im0xNiA1OCAxNi0xOCAzMiAzMiIvPjxjaXJjbGUgY3g9IjUzIiBjeT0iMzUiIHI9IjciLz48L3N2Zz4KCg==' | ||
|
|
||
| export function ImageWithFallback(props: React.ImgHTMLAttributes<HTMLImageElement>) { | ||
| const [didError, setDidError] = useState(false) | ||
|
|
||
| const handleError = () => { | ||
| setDidError(true) | ||
| } | ||
|
|
||
| const { src, alt, style, className, ...rest } = props | ||
|
|
||
| return didError ? ( | ||
| <div | ||
| className={`inline-block bg-gray-100 text-center align-middle ${className ?? ''}`} | ||
| style={style} | ||
| > | ||
| <div className="flex items-center justify-center w-full h-full"> | ||
| <img src={ERROR_IMG_SRC} alt="Error loading image" {...rest} data-original-url={src} /> | ||
| </div> | ||
| </div> | ||
| ) : ( | ||
| <img src={src} alt={alt} className={className} style={style} {...rest} onError={handleError} /> | ||
| ) | ||
| } |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| // This component wraps all pages with the home page's exact header and sidebar | ||
| import { useLocation } from "wouter"; | ||
| import Home from "@/pages/home"; | ||
|
|
||
| interface AppLayoutProps { | ||
| children: React.ReactNode; | ||
| } | ||
|
|
||
| export function AppLayout({ children }: AppLayoutProps) { | ||
| const [location] = useLocation(); | ||
| const isHomePage = location === "/"; | ||
|
|
||
| // If it's the home page, just render it directly (it has its own layout) | ||
| if (isHomePage) { | ||
| return <>{children}</>; | ||
| } | ||
|
|
||
| // For all other pages, wrap them with the home page structure | ||
| // but replace the content area with the actual tool page | ||
| return <Home isToolPage={true}>{children}</Home>; | ||
| } | ||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed default theme from system/user preference to hardcoded 'light'. This may not respect user's system dark mode preference. Consider using 'system' as default or documenting this breaking change.