Add animated Logo component and replace static logo images#49
Merged
Conversation
…move logo_black.png image file
990b8fa Update environment configuration, replace logo references, and enhance UI components. Added NEXT_PUBLIC_APP_NAME to .env.example and build workflow. Changed JSX setting in tsconfig.json. Removed old logo images and integrated AnimatedLogo component across various UI components, including Navbar and authentication forms. Updated layout metadata and adjusted app name handling for consistency. git-subtree-dir: src/lib/core git-subtree-split: 990b8fa7d4270d7fd261818255dc3735411dfb13
…ut and authentication components
…ng logo updates and UI enhancements.
…mproved UI consistency and animation integration.
…ks, enhancing flexibility and configuration management.
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
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.
Summary
Introduces a new
AnimatedLogocomponent (Logo.tsx) to replace static logo image files (logo.png,logo_black.png) across the application. This PR also integrates aNEXT_PUBLIC_APP_NAMEenvironment variable (viaCURRENT_APPconstant) to drive dynamic app branding throughout layout and authentication components.Changes
New Components & Assets
Logo.tsx(274 lines): New animated SVG-based logo component with spinner/animation capabilities, replacing the need for static PNG assetsui-components/index.tsfor shared usage across the core libraryStatic Asset Removal
public/images/logo.pngandpublic/images/logo_black.png— no longer needed since the logo is now rendered programmatically via theAnimatedLogocomponentEnvironment & CI Configuration
NEXT_PUBLIC_APP_NAMEto.env.exampleand.github/workflows/build.ymlto support dynamic app naming at build timeComponent Updates
<Image>(static logo) with<AnimatedLogo>for consistent animated branding in the landing headerCURRENT_APPconstant for dynamic app name rendering; refactored logo referencesCURRENT_APPfor page titles and metadataConfig & Types
auth-core/config.ts: Added/updatedCURRENT_APPconstant sourced fromNEXT_PUBLIC_APP_NAMEauth-core/types.ts: Extended types to support the new app name configurationcore/index.ts: Updated exports to include new Logo component and constantsTests
SignInForm.test.tsxandSignUpForm.test.tsxto account for the new dynamic app name and logo component changesKey UI/UX Improvements
Breaking Changes
NEXT_PUBLIC_APP_NAMEis now required: All deployment environments must set this variable. Without it, app name references may render asundefinedor fallback values. Update.envfiles and CI/CD pipelines accordingly.public/images/logo.pngorpublic/images/logo_black.pngwill break. All logo rendering should now go through theAnimatedLogo/Logocomponent.Testing Notes for Reviewers
NEXT_PUBLIC_APP_NAMEis set in your local.env— confirm the app name renders correctly on login, register, maintenance, and main layout pageslogo_black.pngvariant — verify the new SVG-based component handles both themes appropriatelyNEXT_PUBLIC_APP_NAMEenv var injectedBrowser Compatibility Considerations
Logo.tsxcomponent uses SVG animations (likely CSS keyframes or SVG<animate>elements) — these are well-supported in all modern browsers (Chrome, Firefox, Safari, Edge)requestAnimationFrameor JS-driven animation, confirm there are no jank issues on lower-powered devicesNEXT_PUBLIC_*vars are statically replaced at build time by Next.js🤖 Generated with Claude Code
Branch Info:
feature/logo-spinnermainCo-Authored-By: Claude noreply@anthropic.com