Repository navigation
Refine chat and dashboard UI - #65
Closed
offbyonebit wants to merge 1 commit into
Closed
offbyonebit wants to merge 1 commit into
offbyonebit wants to merge 1 commit into
Conversation
Chat page: - Move the history and settings buttons into the header instead of floating over the composer, and give the history button a real history icon. - Give icon-only buttons zero padding. The late global text-button padding squeezed the attach and send icons to a few pixels wide. - Style the theme toggle, which fell back to the browser default. - Let the header wrap on narrow screens and centre the logo with the title. - Open the side panels beneath the header so the header buttons stay reachable, publishing the header height as --header-h. - Remove the superseded toggle positioning rules. Dashboard: - Honour the hidden attribute on the scan status bar, which was showing as an empty bordered strip. Assets: - Resize the four 1254px logo and icon PNGs to 256px (about 3.7 MB to about 0.2 MB); they display at 42px or smaller.
Owner
Author
|
Closing as superseded by merged PR #68 (48bc7e9), which is an ancestor of current main. Verified the original text patches against that merge and the four UI image blobs by Git blob SHA. The plugin annotation from #64 is present; its surrounding fallback also received a safer getattr check. No outstanding changes from these PRs remain. |
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
Fixes found by rendering the real UI (served locally, screenshots at 1366px and 390px, dark and light).
Chat page
button { padding: 8px 13px }squeezed the attach and send icons to 4px wide, so they rendered as dots.chat.css(.ghostonly exists instyle.css) and showed as a default white browser button.chat.jspublishes the header height as--header-h.Dashboard
.scan-status[hidden]is honoured.display: flexwas overriding the attribute and left an empty bordered strip above the model list.Assets
Testing
test_semantic_search_requires_optional_dependency, which only passes whenfastembedis not installed; with all extras it tries to download a model and was blocked by the sandbox proxy.browser-tests): 10/10 passed, run against this sandbox's Chromium.Not run: real GPU or model streaming through the UI, Windows browsers.
Generated by Claude Code