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

[Task]: Split pane debugger usability bugs #33333

Closed
2 tasks done
hetunandu opened this issue May 9, 2024 · 0 comments · Fixed by #33441
Closed
2 tasks done

[Task]: Split pane debugger usability bugs #33333

hetunandu opened this issue May 9, 2024 · 0 comments · Fixed by #33441
Assignees
Labels
IDE Navigation Issues/feature requests related to IDE navigation, and context switching New Developers Pod Issues that new developers face while exploring the IDE Task A simple Todo

Comments

@hetunandu
Copy link
Member

hetunandu commented May 9, 2024

SubTasks

Tasks

  1. IDE Navigation New Developers Pod Task
    hetunandu
  2. Bug High IDE Navigation Needs Triaging New Developers Pod Production QA QA Pod
    rohan-arthur
@hetunandu hetunandu added the Task A simple Todo label May 9, 2024
@hetunandu hetunandu self-assigned this May 9, 2024
@hetunandu hetunandu added the IDE Navigation Issues/feature requests related to IDE navigation, and context switching label May 9, 2024
@github-actions github-actions bot added the New Developers Pod Issues that new developers face while exploring the IDE label May 9, 2024
@hetunandu hetunandu changed the title [Task]: Debugger in split screen should not be closable [Task]: Split pane debugger usability bugs May 9, 2024
hetunandu added a commit that referenced this issue May 22, 2024
## Description

- Creates a new IDE component for `BottomView` that can be either
collapsed or closed.
- Creates a different view for `EntityTabs` when it is collapsed
- Start using the collapsed version for Entity Responses (Query / API /
JS)


Fixes #33333 
Fixes #33336 
Fixes #31402

## Automation

/ok-to-test tags="@tag.All"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!CAUTION]
> 🔴 🔴 🔴 Some tests have failed.
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/9175565108>
> Commit: 8f3b96e
> Cypress dashboard: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=9175565108&attempt=1&selectiontype=test&testsstatus=failed&specsstatus=fail"
target="_blank"> Click here!</a>
> The following are new failures, please fix them before merging the PR:
<ol>
>
<li>cypress/e2e/Regression/ClientSide/Binding/Button_Text_WithRecaptcha_spec.js
>
<li>cypress/e2e/Regression/ClientSide/Widgets/Others/IconButton_2_spec.ts
</ol>
> To know the list of identified flaky tests - <a
href="https://internal.appsmith.com/app/cypress-dashboard/identified-flaky-tests-65890b3c81d7400d08fa9ee3?branch=master"
target="_blank">Refer here</a>

<!-- end of auto-generated comment: Cypress test results  -->

















## Communication
Should the DevRel and Marketing teams inform users about this change?
- [x] Yes
- [ ] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IDE Navigation Issues/feature requests related to IDE navigation, and context switching New Developers Pod Issues that new developers face while exploring the IDE Task A simple Todo
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant