`showLoading` and `showUndo` "ignore" `isHTML = true`

Đang mở
  • Có gì không ổn?
#2,203 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
55/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
typescript
Lĩnh vực
frontend

Hướng nghiên cứu

Bắt đầu trong lib/toast.ts, khoảng các dòng 213-223, để kiểm tra cách showLoading và showUndo xử lý text và isHTML. Kiểm tra styles/toast.scss, khoảng các dòng 23-28, sau đó xác minh rằng văn bản HTML được hiển thị như dự kiến mà không làm hỏng bố cục của toast.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

1. to develop bug

The showLoading and showUndo functions somewhat "ignore" isHTML = true, because they don't simply pass the text argument to showMessage, which itself just forwards it to Toastify, but they create a new element and set its innerText property to the value of text and then pass the whole element to showMessage.

https://github.com/nextcloud-libraries/nextcloud-dialogs/blob/c32717e1a0593f043a52a6b5bf9c42d427503d50/lib/toast.ts#L213-L223

I think it would be fine to do

const loaderContent = document.createElement('div')
loaderContent[options.isHTML ? 'innerHTML' : 'innerText'] = text

The containers are flex containers, so I think it won't be a problem

https://github.com/nextcloud-libraries/nextcloud-dialogs/blob/c32717e1a0593f043a52a6b5bf9c42d427503d50/styles/toast.scss#L23-L28

If you don't want to change this, I think it would be a good idea to at least remove isHTML from the options

export function showLoading(text: string, options?: Omit<ToastOptions, 'isHTML'>): Toast { }
Ngôn ngữ chính
TypeScript
Star
19
Fork
9
Merge trung bình
1 ngày 5 giờ
Pull request đã merge (30 ngày)
70

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của nextcloud-libraries/nextcloud-dialogs

Tất cả issue của nextcloud-libraries/nextcloud-dialogs

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.