Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layout cleanups: smaller header and single line message input #449

Merged
merged 9 commits into from
Jul 31, 2023

Conversation

toolmantim
Copy link
Contributor

@toolmantim toolmantim commented Jul 31, 2023

Updates the overall chat layout to be waste less vertical space, and improves the chat input context UI:

  • Smaller header (no more logo)
  • Single line message compose box
  • Replace short version of codebase name from chat input context info, in favour of a better title on the repository icon that has the full repository
  • Chat input context shows "No file selected" when no files are open

Things that I didn't manage to fix, but would love to get some eng help on follow-ups:

  • Height of input should change when pasting long paragraphs, not just use the newline count to change [rows] ala https://gomakethings.com/autogrowing-textareas/ or similar
  • Editing a message, which uses it's own separate textarea
  • Removing the tabs: waiting on a quickpick to replace recipes (then I can add a button up the top, or button in the compose box)
Before After
Screenshot 2023-07-30 at 9 56 21 pm Screenshot 2023-07-30 at 9 54 43 pm
Screenshot 2023-07-31 at 1 28 39 pm Screenshot 2023-07-31 at 1 30 27 pm
Screenshot 2023-07-30 at 9 56 08 pm Screenshot 2023-07-30 at 9 55 03 pm

Test plan

  • Open Cody chat
  • Manual interaction

@toolmantim toolmantim changed the title Small header and compose box Layout cleanups: smaller header and single line message input Jul 31, 2023
@toolmantim toolmantim requested a review from a team July 31, 2023 04:23
@toolmantim toolmantim requested a review from a team July 31, 2023 05:40
Copy link
Member

@beyang beyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@toolmantim toolmantim merged commit 6198a15 into main Jul 31, 2023
9 checks passed
@toolmantim toolmantim deleted the tl/smaller-header-and-compose-box branch July 31, 2023 20:32
@toolmantim toolmantim mentioned this pull request Jul 31, 2023
toolmantim added a commit that referenced this pull request Aug 1, 2023
Updates changelog for #449

## Test plan

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants