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

feat: configurable panel for console + ai assistant #1267

Merged
merged 22 commits into from
Apr 16, 2024
Merged

Conversation

jimniels
Copy link
Collaborator

@jimniels jimniels commented Apr 8, 2024

  • Allows the positioning of the console + ai assistant to be configurable, either underneath the code editor, or to the left of it.
  • Updates some mui components to use shadcn

CleanShot 2024-04-10 at 12 59 28@2x

@cla-bot cla-bot bot added the cla-signed label Apr 8, 2024
Copy link

vercel bot commented Apr 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
quadratic ✅ Ready (Inspect) Visit Preview Apr 10, 2024 10:03pm

@@ -10,7 +10,7 @@ class HTMLCellsHandler {
private div?: HTMLDivElement;

attach(parent: HTMLDivElement) {
if (this.div) {
if (this.div && parent) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is breaking HMR, so I added this and now it works.

@davidkircos davidkircos temporarily deployed to quadratic-api-dev-pr-1267 April 8, 2024 16:31 Inactive
Copy link

codecov bot commented Apr 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.00%. Comparing base (a4e6a9d) to head (6afa6e0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1267   +/-   ##
=======================================
  Coverage   89.00%   89.00%           
=======================================
  Files         156      156           
  Lines       25865    25865           
=======================================
  Hits        23021    23021           
  Misses       2844     2844           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 9, 2024 22:16 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 16:46 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 16:48 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:11 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:13 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:13 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:14 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:17 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 20:19 Inactive
@ddimaria
Copy link
Collaborator

This branch needs to merge in main. I went through the code and the UI and everything acts as expected.

I don't have great vision, so the copy was hard to read, but it's the same as before (CMD + only zooms in the grid).

I wasn't familiar with the intent of the asterisk next to Console * in the bottom mode, seems like it was a reference to something more descriptive below rather than an indicator of content in the console. I don't have any great suggestions to make this more clear though.

Copy link
Collaborator

@ddimaria ddimaria left a comment

Choose a reason for hiding this comment

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

Approved, but @davidkircos will review/approve before merging.

@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 21:44 Inactive
@jimniels jimniels temporarily deployed to quadratic-api-dev-pr-1267 April 10, 2024 21:45 Inactive
@jimniels
Copy link
Collaborator Author

FWIW: I filed a separate issue for the font size increase/decrease discrepancies. These are bugs that exist today and aren't necessarily introduced by this PR.

#1277

@jimniels jimniels merged commit 188ea40 into main Apr 16, 2024
13 checks passed
@jimniels jimniels deleted the code-editor-panels branch April 16, 2024 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants