Skip to content
This repository was archived by the owner on Jul 22, 2022. It is now read-only.

Update alignment.js#317

Open
phibersoft wants to merge 1 commit intonatergj:masterfrom
phibersoft:patch-1
Open

Update alignment.js#317
phibersoft wants to merge 1 commit intonatergj:masterfrom
phibersoft:patch-1

Conversation

@phibersoft
Copy link

Even in a very simple code, these parts fail. It is also very misleading to throw an error about the "justifyLastLine" object for all processes. Please remove these sections, "justifyLastLine" is defined and throws error even if its type is boolean.

Code

wb.createStyle({ alignment: { horizontal: "center" }});

Output :
TypeError : justifyLastLine aligment option must be of type boolean

Even in a very simple code, these parts fail. It is also very misleading to throw an error about the "justifyLastLine" object for all processes. Please remove these sections, "justifyLastLine" is defined and throws error even if its type is boolean.

Code 
```
wb.createStyle({ alignment: { horizontal: "center" }});
```
Output : 
  TypeError : justifyLastLine aligment option must be of type boolean
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant