Use the deployed BriefLoop mark and match the landing page's type - #607
Merged
Conversation
The workbench shipped the older loop-and-arrow glyph; briefloop.ai uses assets/briefloop-mark.svg (document loop with a bubble tail). Point the sidebar mark, the empty-state mark and the favicon at that geometry. The deployed landing page has no serif voice: headings are system sans at weight 640 with -0.02em tracking, and monospace is kept for data. Drop the serif headings the previous commit added and use the site's sans scale instead. Also fix the favicon link the previous change left malformed: the replacement regex stopped at the first ">" inside the data URI, so the tail of the old value leaked into the page as a stray `">`.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
1 participant
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.
上一轮我把
/website的旧检出当成了基准，用错了标志：那是"回环+箭头"的旧图，线上 briefloop.ai 现在用的是assets/briefloop-mark.svg（文档 + 回环箭头 + 气泡尾，本地检出落后一个提交才没看到）。这次按线上资产重做。briefloop-mark.svg的几何；favicon 用briefloop-icon.svg的纸底圆角版本。>，旧值的尾巴留在了 head 里，页面上漏出一个">（截图里左上角可见）。现在 head 里只有一条 favicon link，已用无头 Chrome 复验干净。验证：无头 Chrome 渲染本机服务确认侧栏标志与
">已消失；python -m pytest -q180 passed。