search

MaaAssistantArknights / maa-copilot-frontend

MAA Copilot Frontend
https://prts.plus
68 stars 48 forks source link

style: remove unnecessary darkmode overwriting #311

Closed aojunhao123 closed 2 months ago

aojunhao123 commented 2 months ago

image image

fix #289

martinwang2002 commented 2 months ago

这更应该是一个blueprintjs的bug。 See https://github.com/palantir/blueprint/issues/6003

aojunhao123 commented 2 months ago

这更应该是一个blueprintjs的bug。 See https://github.com/palantir/blueprint/issues/6003

好的我再去看看👀

aojunhao123 commented 2 months ago

这更应该是一个blueprintjs的bug。 See palantir/blueprint#6003

我看了您贴出的blueprintjs的相关issue，issue中主要提到了对于tag的interactive和active样式共同作用时的bug。但是作业站的暗黑模式是直接覆写了.bp4-dark样式，这与您贴出的issue中讨论的问题其实不太一致。而我认为最后的解决方案就是覆写不同类型的标签背景色，或是使用blueprintjs提供的主题色变量。 PS：若您认为我对该问题的解决方案有偏差，希望能得到进一步的解答（感谢大佬

guansss commented 2 months ago

@aojunhao123 确实，这个修复很合理

感谢 PR！