OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

During startup, enable text field, but disable submit

Open 0xSaurabhx opened this issue 1 year ago • 1 comments

The disabled attribute is removed from the input element. This allows users to type their prompt even while OpenDevin is initializing. The disabled attribute is added to the button element and set to !initialized. This ensures that the user cannot submit the prompt until OpenDevin is ready.

0xSaurabhx avatar Mar 26 '24 16:03 0xSaurabhx

@0xSaurabhx looks like a lint issue. Otherwise LGTM!

rbren avatar Mar 26 '24 20:03 rbren

Looks like You have already added changes main branch

0xSaurabhx avatar Mar 27 '24 01:03 0xSaurabhx

Not sure what happened here--looks like the changed files are unrelated?

The original problem is still there: https://github.com/OpenDevin/OpenDevin/blob/95c128c3d69a4cb3c8b13d1d4aadcbc44d8fdff5/frontend/src/components/ChatInterface.tsx#L66

rbren avatar Mar 27 '24 03:03 rbren

@0xSaurabhx this branch now contains changes unrelated to the PR above. I'm going to close it, but feel free to open a new PR with the proposed changes.

In the future, I'd suggest creating a new branch with git checkout -b every time you want to work on a new feature--it keeps the PRs from getting cluttered with unrelated changes

rbren avatar Mar 27 '24 14:03 rbren