ScottyLabs/cmugpt-surface
Watch
7
0
Fork
You've already forked cmugpt-surface
 0
Code Issues 12 Pull requests 2 Projects Releases Packages Wiki Activity Actions

fix: reach and leave chat search on mobile #45

Open
Jc-965 wants to merge 1 commit from chat-ui-overhaul into staging
pull from: chat-ui-overhaul
merge into: ScottyLabs:staging
Conversation 1 Commits 1 Files changed 2 +29 -4
Jc-965 commented 2026-08-16 00:50:52 +00:00
Member
Copy link

Summary

Makes chat search usable on a phone.

  • Tapping search in the sidebar now closes the sidebar and lands on the search screen.
  • The search screen now has a menu button to the left of the search bar that reopens the sidebar. Before, there was no way to leave search on mobile
  • Desktop behavior is unchanged
## Summary Makes chat search usable on a phone. - Tapping search in the sidebar now closes the sidebar and lands on the search screen. - The search screen now has a menu button to the left of the search bar that reopens the sidebar. Before, there was no way to leave search on mobile - Desktop behavior is unchanged
fix: reach and leave chat search on mobile
All checks were successful
CI / build (pull_request) Has been skipped
Details
kennel/build build succeeded
Details
kennel/deploy deployment healthy
Details
CI / check-1 (pull_request) Successful in 2m7s
Details
CI / check (pull_request) Successful in 0s
Details
f2a49bb744
scottylabs-bot commented 2026-08-16 00:51:36 +00:00
Owner
Copy link

Kennel Deployments

Service URL
api https://cmugpt-surface-api-pr-45.scottylabs.net
web https://cmugpt-surface-web-pr-45.scottylabs.net

Last updated for commit f2a49bb.

<!-- kennel-deployment --> ### Kennel Deployments | Service | URL | | --- | --- | | api | https://cmugpt-surface-api-pr-45.scottylabs.net | | web | https://cmugpt-surface-web-pr-45.scottylabs.net | Last updated for commit `f2a49bb`.
All checks were successful
CI / build (pull_request) Has been skipped
Details
kennel/build build succeeded
Details
kennel/deploy deployment healthy
Details
CI / check-1 (pull_request) Successful in 2m7s
Details
CI / check (pull_request) Successful in 0s
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin chat-ui-overhaul:chat-ui-overhaul
git switch chat-ui-overhaul

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch staging
git merge --no-ff chat-ui-overhaul
git switch chat-ui-overhaul
git rebase staging
git switch staging
git merge --ff-only chat-ui-overhaul
git switch chat-ui-overhaul
git rebase staging
git switch staging
git merge --no-ff chat-ui-overhaul
git switch staging
git merge --squash chat-ui-overhaul
git switch staging
git merge --ff-only chat-ui-overhaul
git switch staging
git merge chat-ui-overhaul
git push origin staging
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
  
needs/docs
Must have documentation written

  
needs/research
Discuss the ideal implementation with the tech lead

  
needs/upstream
Needs a contribution to an upstream project

  
type/bug
Something not working as intended

  
type/external
Lives outside of code

  
type/feature
New functionality

  
type/refactor
Improving existing code

  
urgency
high
 This is currently causing issues

  
urgency
immediate
 This is making a project or feature unusable

  
urgency
low
 This can happen whenever

  
urgency
medium
 This is expected to cause issues soon

No labels
needs/docs
needs/research
needs/upstream
type/bug
type/external
type/feature
type/refactor
urgency
high
urgency
immediate
urgency
low
urgency
medium
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
ScottyLabs/cmugpt-surface!45
No description provided.