fix(app): expose context pane on mobile web #10457
Conversation
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
brittlewis12
force-pushed
the
fix/mobile-context-fresh
branch
from
f778094 to
89bb1bf
Compare
|
rebased to resolve the conflicts in session.tsx to play nicely with the latest changes to the file tree! i know you folks have a lot on your plate as it is @adamdotdevin, but being able to see this context panel was really useful in helping me track down another bug*, so i'd love to get this (or something like it) in! *- client generated user message IDs can be created out of order relative to server-generated assistant message IDs — you'll notice the assistant message sorted before the user message in the screen recording — i'm making a separate issue for this. |
brittlewis12
force-pushed
the
fix/mobile-context-fresh
branch
4 times, most recently
from
2eabe4a to
206a1ea
Compare
* workaround tooltip ignoring touch events * introduce third mobile tab state for context panel * two-tap pattern for opening context tab from tooltip
brittlewis12
force-pushed
the
fix/mobile-context-fresh
branch
from
206a1ea to
40a9485
Compare
|
rebased once again to resolve conflicts with the latest file tree handling! also noticed a broken e2e test from the change from “Review” to “File tree”; happy to contribute that separately if preferred. |
1 participant
fixes #10443
note: recommend reviewing with
?w=1github review whitespace parameterWhat does this PR do?
How did you verify your code works?
tested on device running the locally-built web app client:
before
before.mp4
after
after.mp4