Site dark mode: IMPORTANT callouts render as light text on a white box
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
- 88/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- css
- Lĩnh vực
- frontend
Hướng nghiên cứu
Bắt đầu với src/site/resources/css/site.css và so sánh các biến màu thay thế của tệp này với quy tắc cho callout IMPORTANT ở chế độ tối được hiển thị trong issue. Thêm phần ghi đè nhắm mục tiêu body.dark .callout.callout-important được mô tả ở đó, sau đó xác minh rằng callout IMPORTANT có màu chữ và màu nền dễ đọc ở chế độ tối.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Where: src/site/resources/css/site.css (MetricsHub colors), with Sentry Maven Skin 8.
What happens: in dark mode, an > [!IMPORTANT] callout renders as a white box with light text, which is unreadable.
The skin styles IMPORTANT callouts with the alternate colors, and swaps them in dark mode (published css/main-combined.css):
.callout.callout-important { --callout-bg: var(--alternate-bgcolor); --callout-fg: var(--alternate-fgcolor); }
body.dark .callout.callout-important { --callout-bg: var(--alternate-fgcolor); --callout-fg: var(--alternate-bgcolor); }
Our site.css sets --alternate-bgcolor: #266fd0 and --alternate-fgcolor: #fff and does not override them in body.dark, so in dark mode the callout background becomes #fff. The callout's body text keeps the dark-mode foreground (#e9ecef), so it is light text on white.
Impact today: none of the current pages uses an IMPORTANT callout, so the bug is latent. It shows up as soon as a page adds one. It was found in ipmi-java, which copied this site.css (MetricsHub/ipmi-java#113).
Suggested fix: the fix applied in ipmi-java, appended to site.css:
/* The skin inverts the alternate colors for IMPORTANT callouts in dark mode, which turns our white-on-blue
palette into a white box with light text */
body.dark .callout.callout-important {
--callout-bg: var(--main-bgcolor);
--callout-fg: var(--link-color);
}
Overriding --alternate-fgcolor in body.dark instead would also darken the text that the skin draws on the blue alternate background (active menu item, ...), so the targeted rule is safer. The other MetricsHub projects that share this site.css probably have the same latent bug. The skin's dark rule could arguably handle a light --alternate-fgcolor itself.
- Ngôn ngữ chính
- Java
- Star
- 13
- Fork
- 4
- Merge trung bình
- 7 giờ 38 phút
- Pull request đã merge (30 ngày)
- 14
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của MetricsHub/winrm-java
-
enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
MetricsHub/winrm-java#201 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 78/100
MetricsHub/winrm-java#199 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Streaming command fails with a spurious timeout when reconnecting after an idle pauseCó thể đã có người làm @NassimBtk đã nhận 2 ngày trước. Đang mở
MetricsHub/winrm-java#198 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Long-lived `WinRMClient` keeps failing with WSManFault 2150859174 after a terminate Signal failsCó thể đã có người làm @NassimBtk đã nhận 1 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
MetricsHub/winrm-java#196 · 4 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
MetricsHub/winrm-java#194 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của MetricsHub/winrm-java
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Netcracker/qubership-integration-platform#1046 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
`check_java_version()` fails when Java path contains spaces (Windows / Git Bash, `C:\Program Files`)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Fix Math.ceilDiv wrong result for exact positive divisionsCó thể đã có người làm @pamod-madubashana đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
scala-native/scala-native#5094 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] AI unread message badge counts a batch of new bubbles as one messageCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
apache/rocketmq-dashboard#5784 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
[i18n] 安装实例完成后的成功提示未正确本地化Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
PCL-Community/PCL-CE#3658 ·
Maintainer thường phản hồi trong vòng 1 ngày