Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: [WebUI] フルダークテーマの修正 #708

Merged
merged 3 commits into from
Apr 9, 2024
Merged

Fix: [WebUI] フルダークテーマの修正 #708

merged 3 commits into from
Apr 9, 2024

Conversation

yuicho
Copy link

@yuicho yuicho commented Apr 9, 2024

フルダークテーマの下記箇所を修正しています。

  • textarea[id='upload-modal__description']

    • 投稿に添付したメディアの編集モーダルウインドウに存在する、「視覚的に閲覧が難しいユーザーへの説明」の入力エリア
    • textareaの背景色および文字色が白色のままだったため、視認性が悪かったものを修正しました。
    • 修正前:image
    • 修正後:image
  • .compose-form__warning

    • 投稿画面にて一部の公開範囲などを選んだ際に、入力欄の上部に表示される、注意書き等の表示エリア
    • 明るい背景色のため視認性が悪かったものを修正しました
    • 修正前:image
    • 修正後:image

yuicho added 2 commits April 10, 2024 05:20
投稿に添付したメディアの編集モーダルウインドウに存在する「視覚的に閲覧が難しいユーザーへの説明」を入力するテキストエリアの色を修正
投稿画面にて一部の公開範囲などを選んだ際に、入力欄の上部に表示される .compose-form__warning の背景色を修正(文字の視認性向上のため)
@yuicho yuicho marked this pull request as ready for review April 9, 2024 21:02
idの指定を`#~~~` から `[id=~~~]` に変更
@kmycode kmycode added this to the kb11.4 milestone Apr 9, 2024
@kmycode kmycode mentioned this pull request Apr 9, 2024
4 tasks
@kmycode kmycode added the bug Something isn't working label Apr 9, 2024
@kmycode kmycode merged commit 207fff0 into kmycode:kb_development Apr 9, 2024
22 checks passed
kmycode pushed a commit that referenced this pull request Apr 9, 2024
* Fix media alt textarea color

投稿に添付したメディアの編集モーダルウインドウに存在する「視覚的に閲覧が難しいユーザーへの説明」を入力するテキストエリアの色を修正

* Fix compose-form__warning color

投稿画面にて一部の公開範囲などを選んだ際に、入力欄の上部に表示される .compose-form__warning の背景色を修正(文字の視認性向上のため)

* fix CSS Selector

idの指定を`#~~~` から `[id=~~~]` に変更
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants