Algorithm to pad string is not Unicode aware
- 主要语言
- JavaScript
- 星标
- 6k
- 派生
- 1.3k
- 平均合并
- 1 天 3 小时
- 30 天内合并 PR
- 611
描述
## Checklist
> Please ensure the following tasks are completed before filing a bug report.
- [x] Read and understood the [Code of Conduct][code-of-conduct].
- [x] Searched for existing issues and pull requests.
## Description
> Description of the issue.
Encountered an error when attempting to pad a string based on the number of visual characters. Currently, we rely on `String#length` to determine the number of characters to pad; however, this returns the number of code points, which may not be same as the number of graphemes.
We should consider how to properly handle Unicode characters comprised of multiple code points.
## Related Issues
> Does this issue have any related issues?
No.
## Questions
> Any questions for reviewers?
No.
## Other
> Any other information relevant to this issue? This may include screenshots, references, stack traces, sample output, and/or implementation notes.
This affects the following packages:
- [`@stdlib/string/left-pad`](https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/left-pad)
- [`@stdlib/string/right-pad`](https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/right-pad)
- [`@stdlib/string/pad`](https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/pad)
#### Demo
> If relevant, provide a link to a live demo.
N/A
#### Reproduction
> What steps are required to reproduce the unexpected output?
In order to reproduce this bug, do the following:
- attempt to pad using an input string containing surrogate pairs or by providing a string with which to pad containing surrogate pairs.
#### Expected Results
> What are the expected results?
I might expect that I would see padding based on the number of visual characters. Currently, the implementation runs the risk of unpaired surrogates.
#### Actual Results
> What are the actual results?
N/A
#### Environments
> What environments are affected (e.g., `Node v0.4.x`, `Chrome`, `IE 11`)? If Node.js, include the `npm` version, operating system, and any other potentially relevant platform information.
The following environments are affected:
- all
[code-of-conduct]: https://github.com/stdlib-js/stdlib/blob/develop/CODE_OF_CONDUCT.md
贡献指南
调研方向
Start by inspecting the implementations of @stdlib/string/left-pad, @stdlib/string/right-pad, and @stdlib/string/pad, focusing on their use of String#length. Reproduce the behavior with strings containing surrogate pairs, then verify that padding is based on visual characters and does not produce unpaired surrogates.
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript
- 领域
- tooling
- Issue 类型
- 缺陷
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100