microsoft / microsoft/TypeScript
Give 'this' keyword suggestion the same `sortText` as class properties if inside a class
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Go
- Star
- 111k
- Fork
- 14.3k
- Merge trung bình
- 2 ngày 4 giờ
- Pull request đã merge (30 ngày)
- 132
Mô tả
From https://github.com/Microsoft/vscode/issues/66868
TypeScript Version: 3.4.0-dev.20190220
Search Terms:
- suggestions
- completions
- sortText
Code
For the javascript:
class Foo {
constructor() {
this.prop = 1;
}
render() {
th
}
}
- Trigger intellisense after
thinrender
Expected behavior:
this should be the first suggestion in VS Code
Actual behavior:
render and then prop are the first suggestions
The root cause of this is that the suggestion for this has a sortText of "0" while the one for prop and render have a sortText of "1". This causes VS Code to sort prop and render before this
@amcasey @minestarks I'm not sure if the current sorting issue would also effect VS
Playground Link:
Related Issues:
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Tái hiện thứ tự hoàn thành trong TypeScript language service bằng ví dụ lớp và kiểm tra các mục hoàn thành cho this, prop và render. Cập nhật hành vi hoàn thành liên quan để this sử dụng cùng sortText với các thuộc tính lớp, sau đó thêm hoặc cập nhật một bài kiểm thử hồi quy bao quát ví dụ này và xác minh thứ tự kết quả.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript, typescript
- Lĩnh vực
- compilers, tooling
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100