Mintplex-Labs/anything-llm
1
0
Fork 0
mirror of https://github.com/Mintplex-Labs/anything-llm.git synced 2026-07-19 14:13:35 -04:00
Code Issues 328 Packages Projects Releases 10 Wiki Activity

[PR #2072] [CLOSED] Patch reliance on isMobile for mobile device UI detection #3876

New Issue
Closed
opened 2026-02-22 18:34:41 -05:00 by yindo · 0 comments
yindo commented 2026-02-22 18:34:41 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/2072
Author: @timothycarambat
Created: 8/8/2024
Status: ❌ Closed

Base: master ← Head: resize-mobile-ui

📝 Commits (5)

  • 2a0828f Patch reliance on isMobile for mobile device UI detection
  • b16114f implement useIsMobile hook
  • ab4be4d Merge branch 'master' into resize-mobile-ui
  • 21b3b8f fix resizing of text input
  • b7222ec Merge branch 'master' into resize-mobile-ui

📊 Changes

36 files changed (+121 additions, -39 deletions)

View changed files

📝 frontend/src/App.jsx (+2 -0)
📝 frontend/src/components/DefaultChat/index.jsx (+3 -1)
📝 frontend/src/components/Footer/index.jsx (+2 -1)
📝 frontend/src/components/Modals/ManageWorkspace/index.jsx (+2 -1)
📝 frontend/src/components/SettingsSidebar/index.jsx (+2 -1)
📝 frontend/src/components/WorkspaceChat/ChatContainer/PromptInput/index.jsx (+2 -5)
📝 frontend/src/components/WorkspaceChat/ChatContainer/index.jsx (+2 -1)
📝 frontend/src/components/WorkspaceChat/LoadingChat/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/Agents/index.jsx (+4 -1)
📝 frontend/src/pages/Admin/ExperimentalFeatures/Features/LiveSync/manage/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/ExperimentalFeatures/index.jsx (+4 -1)
📝 frontend/src/pages/Admin/Invitations/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/Logging/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/System/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/Users/index.jsx (+2 -1)
📝 frontend/src/pages/Admin/Workspaces/index.jsx (+2 -2)
📝 frontend/src/pages/FineTuning/Steps/index.jsx (+2 -1)
📝 frontend/src/pages/FineTuning/index.jsx (+2 -1)
📝 frontend/src/pages/GeneralSettings/ApiKeys/index.jsx (+2 -1)
📝 frontend/src/pages/GeneralSettings/Appearance/index.jsx (+2 -1)

...and 16 more files

📄 Description

Pull Request Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #xxx

What is in this change?

  • Implement useIsMobile hook to replace all instances of isMobile from react-device-detect
  • The useIsMobile hook allows us to control the mobile breakpoint easier and also makes all UI that uses this hook reactive to screen size changes to switch between mobile and desktop views without refreshing the page
  • Fix a bug where the text input box for chatting was a fixed size and would go off the screen on mobile views (bug from implementing the AttachmentManager feature)

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/2072 **Author:** [@timothycarambat](https://github.com/timothycarambat) **Created:** 8/8/2024 **Status:** ❌ Closed **Base:** `master` ← **Head:** `resize-mobile-ui` --- ### 📝 Commits (5) - [`2a0828f`](https://github.com/Mintplex-Labs/anything-llm/commit/2a0828f4e6462c8a803abd58654ab87ec7f250f7) Patch reliance on `isMobile` for mobile device UI detection - [`b16114f`](https://github.com/Mintplex-Labs/anything-llm/commit/b16114f400affb1885e7ecc301b3fae98818b3ee) implement useIsMobile hook - [`ab4be4d`](https://github.com/Mintplex-Labs/anything-llm/commit/ab4be4d64633f27b0e691f1f2b1f6f1c8f44260d) Merge branch 'master' into resize-mobile-ui - [`21b3b8f`](https://github.com/Mintplex-Labs/anything-llm/commit/21b3b8fa539ecc9f86d9d6cea237672ad8ebac90) fix resizing of text input - [`b7222ec`](https://github.com/Mintplex-Labs/anything-llm/commit/b7222ec6922528c9b9c984c72c97e5db7b11ad3d) Merge branch 'master' into resize-mobile-ui ### 📊 Changes **36 files changed** (+121 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `frontend/src/App.jsx` (+2 -0) 📝 `frontend/src/components/DefaultChat/index.jsx` (+3 -1) 📝 `frontend/src/components/Footer/index.jsx` (+2 -1) 📝 `frontend/src/components/Modals/ManageWorkspace/index.jsx` (+2 -1) 📝 `frontend/src/components/SettingsSidebar/index.jsx` (+2 -1) 📝 `frontend/src/components/WorkspaceChat/ChatContainer/PromptInput/index.jsx` (+2 -5) 📝 `frontend/src/components/WorkspaceChat/ChatContainer/index.jsx` (+2 -1) 📝 `frontend/src/components/WorkspaceChat/LoadingChat/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/Agents/index.jsx` (+4 -1) 📝 `frontend/src/pages/Admin/ExperimentalFeatures/Features/LiveSync/manage/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/ExperimentalFeatures/index.jsx` (+4 -1) 📝 `frontend/src/pages/Admin/Invitations/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/Logging/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/System/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/Users/index.jsx` (+2 -1) 📝 `frontend/src/pages/Admin/Workspaces/index.jsx` (+2 -2) 📝 `frontend/src/pages/FineTuning/Steps/index.jsx` (+2 -1) 📝 `frontend/src/pages/FineTuning/index.jsx` (+2 -1) 📝 `frontend/src/pages/GeneralSettings/ApiKeys/index.jsx` (+2 -1) 📝 `frontend/src/pages/GeneralSettings/Appearance/index.jsx` (+2 -1) _...and 16 more files_ </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [ ] 🐛 fix - [x] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #xxx ### What is in this change? <!-- Describe the changes in this PR that are impactful to the repo. --> - Implement `useIsMobile` hook to replace all instances of `isMobile` from `react-device-detect` - The `useIsMobile` hook allows us to control the mobile breakpoint easier and also makes all UI that uses this hook reactive to screen size changes to switch between mobile and desktop views without refreshing the page - Fix a bug where the text input box for chatting was a fixed size and would go off the screen on mobile views (bug from implementing the AttachmentManager feature) ### Additional Information <!-- Add any other context about the Pull Request here that was not captured above. --> ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:34:41 -05:00
yindo closed this issue 2026-02-22 18:34:41 -05:00
yindo changed title from [PR #2072] Patch reliance on `isMobile` for mobile device UI detection to [PR #2072] [CLOSED] Patch reliance on `isMobile` for mobile device UI detection 2026-06-05 15:15:41 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Desktop
Docker
Integration Request
Integration Request
OS: Linux
OS: Mobile
OS: Windows
UI/UX
blocked
bug
bug
core-team-only
documentation
duplicate
embed-widget
enhancement
feature request
github_actions
good first issue
investigating
needs info / can't replicate
possible bug
pull-request
Mirrored from GitHub Pull Request
 question
stage: specifications
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3876
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?