Skip to content

Conversation

@dididy
Copy link
Contributor

@dididy dididy commented Oct 15, 2024

What is this PR for?

Screen_Recording_2024-09-08_at_23.44.12.mov

Steps to reproduce:

  1. Make a new Paragraph.
  2. Refresh the page.
  3. Click on the paragraph to make the cursor blink.

[How to solve]

Set default value of editor's height

What type of PR is it?

Bug Fix

Todos

What is the Jira issue?

How should this be tested?

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? N
  • Is there breaking changes for older versions? N
  • Does this needs documentation? N

@pan3793 pan3793 merged commit cd31c31 into apache:master Oct 16, 2024
pan3793 pushed a commit that referenced this pull request Oct 16, 2024
### What is this PR for?

https://github.com/user-attachments/assets/6204b6f5-409e-4cd1-8901-eb09114e9ded

Steps to reproduce:
1. Make a new Paragraph.
2. Refresh the page.
3. Click on the paragraph to make the cursor blink.

[How to solve]
https://github.com/apache/zeppelin/blob/bdf5b067b6bdde2614b98a6a3f6a7b5d6637e57c/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/code-editor/code-editor.component.ts#L62

Set default value of editor's height

### What type of PR is it?
Bug Fix

### Todos

### What is the Jira issue?
* [[ZEPPELIN-6096](https://issues.apache.org/jira/browse/ZEPPELIN-6096)]

### How should this be tested?

### Screenshots (if appropriate)

### Questions:
* Does the license files need to update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? N

Closes #4873 from dididy/fix/ZEPPELIN-6096.

Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit cd31c31)
Signed-off-by: Cheng Pan <chengpan@apache.org>
@pan3793
Copy link
Member

pan3793 commented Oct 16, 2024

Thanks, merged to master/0.12

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.

2 participants