Brand: integrate the approved WVO monogram - #35
Merged
Conversation
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
apps/web/public/brand/using optimized transparent WebP assets.Asset Validation
WVO-logo-transparent.png: 2,008,932 bytes, PNG,1536x1024, transparent corners.WVO-logo-white-transparent.png: 2,115,578 bytes, PNG,1536x1024, transparent corners.WVO-logo-transparent.webp: 26,718 bytes, WebP,768x512,argbalpha.WVO-logo-white-transparent.webp: 43,058 bytes, WebP,768x512,argbalpha.WVO-favicon.png: 7,701 bytes, PNG,256x256, derived from the approved white monogram on the app slate background.The app logo pair went from 4,124,510 bytes to 69,776 bytes, about a 98.3% reduction. The supplied logo artwork was not redrawn or recolored; the app-facing assets are lossless WebP conversions scaled to a practical 768px source width for UI and README use. Only the favicon was derived as a small browser-chrome crop/scale.
Scope Boundaries
.tmp/,data/, coverage output, build output, personal files, or private media committed.Visual Review
npm run review:ui-screensafter both the branding change and the WebP optimization.Verification
npm ci: passed, added 548 packages.npm run format: passed.npm run format:check: passed.npm run lint: passed.npm run typecheck: passed.npm run test:run: passed, 388 tests passed, 1 skipped.npm run build: passed.npm run check: passed after WebP optimization.npm run test:coverage: passed.npm audit --omit=dev: passed, 0 vulnerabilities.npm run test:e2e: passed, 10 Chromium tests.npm run test:integration:media: passed, 5 real-media integration tests.npm run review:ui-screens: passed, 1 screenshot review test.git diff --check: passed.npm run test:run --workspace apps/web -- src/app/App.test.tsxpassed, 21 tests;npm run test:runpassed, 388 tests, 1 skipped.Notes
A stale ignored local
.tmp/dev-smokescratch folder caused one initial lint attempt to inspect an old generated script; it was removed..tmp/ui-review/and local application data were preserved.