OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix(frontend): fix mobile view of SettingsModal

Open KianoshArian opened this issue 7 months ago • 0 comments

  • [ ] This change is worth documenting at https://docs.all-hands.dev/
  • [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Summarize what the PR does, explaining any non-trivial design decisions.

I tweaked some tailwind styles in order to fix the SettingsModal mobile view. The desktop view is unchanged. Please let me know if you have any suggestions on this matter.

Before After
Before After

How to run the changes locally Go to OpenHands/frontend directory Use npm run dev:mock to run the frontend project You should see the modal open in the home page. If not, go to Line 30 of /frontend/src/components/features/sidebar/sidebar.tsx and set the initial state value of settingsModalIsOpen to true:

const [settingsModalIsOpen, setSettingsModalIsOpen] = React.useState(false);

Change to:

const [settingsModalIsOpen, setSettingsModalIsOpen] = React.useState(true);

KianoshArian avatar May 26 '25 16:05 KianoshArian