表格渲染出现问题

[ZnPdCo]

具体参看这里的第一个表格，边框全部消失了。

[Enter-tainer]

这个是feedback sys导致的吗

[ZnPdCo]

这个是feedback sys导致的吗

是的，我在一个新的浏览器不进入内侧显示正常，进入就会出错

[Enter-tainer]

有点神奇。。可能是什么css打架了

[shaokeyibb]

神奇，按理来说不会，我 CSS 都是有设置前缀的

[ZnPdCo]

表格样式全部忽略掉了

[shaokeyibb]

初步感觉确实可能是 feedback-sys 的问题，关了以后就正常了

[ZnPdCo]

发现开启了之后 .md-typeset table:not([class]) td 就找不到了

[shaokeyibb]

发现开启了之后 .md-typeset table:not([class]) td 就找不到了

这个样式不是由 feedback-sys 提供的才对

[ZnPdCo]

发现开启了之后 .md-typeset table:not([class]) td 就找不到了

这个样式不是由 feedback-sys 提供的才对

我知道了，原本主题的样式就有问题

[ZnPdCo]

因为开启了 feedback-sys 后会给 table 加一个 class="review_enabled"，表格样式就匹配不到了

[ZnPdCo]

删去之后正常显示

[ZnPdCo]

我觉得可以给原本的样式进行修改

[ZnPdCo]

我不太理解为什么 table 要加上 :not([class])，感觉可以直接删掉：

table:not([class]) {

把这一行改掉

[ZnPdCo]

或者说我有一个更好的解决方法，就是 feedback-sys 不要添加 class 而是添加其他属性，例如 review_enabled=true，然后修改一下css就行了

[shaokeyibb]

或者说我有一个更好的解决方法，就是 feedback-sys 不要添加 class 而是添加其他属性，例如 review_enabled=true，然后修改一下css就行了

也可以，比如换成 data-review-enabled 这种

[Enter-tainer]

看了下现在feedback sys的css，抽象做的还挺好的 如果换成 data attr 的话改起来似乎也不算麻烦

[Enter-tainer]

还有个思路就是禁止所有表格的评论

[ZnPdCo]

还有个思路就是禁止所有表格的评论

直接换成 data attr 吧，这样兼容性比较高

[Enter-tainer]

我发现 除了review enabled 还有 review focused

[Enter-tainer]

有个问题，这个和 on hover 有什么区别

[Enter-tainer]

额看了下好像操作classname的方法还挺多的，有 review focused/selected/has_comments

[ZnPdCo]

额看了下好像操作classname的方法还挺多的，有 review focused/selected/has_comments

感觉操作classname都不太好

[Enter-tainer]

额看了下好像操作classname的方法还挺多的，有 review focused/selected/has_comments

感觉操作classname都不太好

感觉本身没啥问题，只是在mkdocs这个场景，因为mkdocs-material里面会写一些傻逼的css，导致操作classname会出问题。。

[Enter-tainer]

[ZnPdCo]

我刚刚找到了，鼠标移动上去时是focused

[shaokeyibb]

好消息：表格好了 坏消息：悬浮在上面现在会多一个滚动条...

[Enter-tainer]

看起来似乎是因为mkdocs会给表格外面套上几层div？这图看起来似乎悬浮按钮是在滚动条里面的😂

[Enter-tainer]

不知道这个还能不能修。。感觉如果要修的话可能就是给table写个特判？

您现在是咋改的呀，如果您已经改了attr，要不就变成 不给table注入 + 改成 data attr😂

感觉可能都行，如果能特判一下干掉好像也行。

[shaokeyibb]

不知道这个还能不能修。。感觉如果要修的话可能就是给table写个特判？

您现在是咋改的呀，如果您已经改了attr，要不就变成 不给table注入 + 改成 data attr😂

感觉可能都行，如果能特判一下干掉好像也行。

现在就是改成 data-attr，如果要做特判的话我个人觉得可能得在 content-script 那边做而不是在这边做？

[Enter-tainer]

我脑补的特判可能是在这里改的，大概是给每一个带attr的东西插入按钮的时候（或者是附近的代码）查一下它是不是table，有没有被div套起来，如果被套了就做一些事情。。。感觉丑丑的而且好像也有点脆弱。。。你是咋想的呀

[shaokeyibb]

不知道这个还能不能修。。感觉如果要修的话可能就是给table写个特判？ 您现在是咋改的呀，如果您已经改了attr，要不就变成 不给table注入 + 改成 data attr😂 感觉可能都行，如果能特判一下干掉好像也行。

现在就是改成 data-attr，如果要做特判的话我个人觉得可能得在 content-script 那边做而不是在这边做？

我的想法是，如果要把这个库当作一个“通用”的解决方案，那就应该尽量少一些这种奇奇怪怪的特判？不然后人看起来还是屎山。

其实我们只要在 content-script 上面给 table 的父 div 上一个 overflow: hidden 其实就可以一定程度上修复这个问题，只不过会让按钮显示不全而已

[Enter-tainer]

okk，应该也可以。看看效果咋样吧

[Enter-tainer]

我自己觉得table 不能评论/评论起来有点bug 应该都可以接受，因为我们本身也没有很多的table

[shaokeyibb]

okk，应该也可以。看看效果咋样吧

[Enter-tainer]

问题不大，冲！

[Enter-tainer]

其实我们只要在 content-script 上面给 table 的父 div 上一个 overflow: hidden 其实就可以一定程度上修复这个问题，只不过会让按钮显示不全而已

我在想是不是其实可以用 overflow: visible，这样好像就完全正常了？ also see #86

[Enter-tainer]

另外我在想这个可不可以直接改 css，而不用 js

[shaokeyibb]

直接改 CSS 感觉有点侵入？目前尚不清楚这样做可能对表格带来其他的副作用

overflow: visible 好像确实可以？我一会儿做实验看看