Skip to content

Conversation

@hsuanxyz
Copy link
Member

@hsuanxyz hsuanxyz commented Jan 7, 2020

What is this PR for?

Convert code editor font size units (from pt to px).

What type of PR is it?

[Bug Fix]

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4540

How should this be tested?

N/A

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@zjffdu
Copy link
Contributor

zjffdu commented Jan 9, 2020

LGTM, thanks @hsuanxyz

asfgit pushed a commit that referenced this pull request Jan 10, 2020
### What is this PR for?

Convert code editor font size units (from pt to px).

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

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4540

### How should this be tested?

N/A

### Screenshots (if appropriate)

N/A

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

Author: Hsuan Lee <[email protected]>

Closes #3588 from hsuanxyz/fix/code-editor-font-size and squashes the following commits:

3778a12 [Hsuan Lee] fix: convert code editor font size units
@hsuanxyz hsuanxyz closed this Jan 10, 2020
asfgit pushed a commit that referenced this pull request Jan 12, 2020
### What is this PR for?

Convert code editor font size units (from pt to px).

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

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4540

### How should this be tested?

N/A

### Screenshots (if appropriate)

N/A

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

Author: Hsuan Lee <[email protected]>

Closes #3588 from hsuanxyz/fix/code-editor-font-size and squashes the following commits:

3778a12 [Hsuan Lee] fix: convert code editor font size units
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