`showLoading` and `showUndo` "ignore" `isHTML = true`
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- frontend
調査の方向性
lib/toast.ts の 213～223 行付近から始めて、showLoading と showUndo が text と isHTML をどのように処理しているかを確認します。次に styles/toast.scss の 23～28 行付近を確認し、HTML テキストがトーストのレイアウトを崩さず意図したとおりにレンダリングされることを検証します。
索引モデルが issue の本文から書いたものです。
説明
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.
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
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 { }
- 主要言語
- TypeScript
- スター
- 19
- フォーク
- 8
- 平均マージ
- 1日 21時間
- マージ済み PR（30日）
- 46
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nextcloud-libraries/nextcloud-dialogs のほかの issue
-
Actions on Toastsオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
nextcloud-libraries/nextcloud-dialogs#2669 ·
メンテナーはふだん 1 日以内に返信
-
ConflictPicker non-scoped styles leed to literal ":deep()" in css assets.対応中かも @rotdrop が 17 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
nextcloud-libraries/nextcloud-dialogs#2648 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
nextcloud-libraries/nextcloud-dialogs#2510 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
nextcloud-libraries/nextcloud-dialogs#2491 · コメント 3 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 55/100
nextcloud-libraries/nextcloud-dialogs#2299 ·
メンテナーはふだん 1 日以内に返信
nextcloud-libraries/nextcloud-dialogs の issue をすべて見る
似ている issue
-
bnv-manager-v2: accept_invite creates the Mailcow domain admin before the commit対応中かも @myOmikron が今日担当しました。 オープンbug ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
platform:windows tool triaged wontfix
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
nicolettas-muggelbude/RechnungsFee#448 ·
メンテナーはふだん 1 日以内に返信
-
Android bug feature/error-tracking react-native team/error-tracking
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
PostHog/posthog-js#5310 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信