核心功能请求：添加 3 个类

开发
1

你好，我正在为“引用”框设置样式……是否有可能让 Discourse 核心在下箭头（参见上下文中的引用）和上箭头（跳转到被引用的消息）上添加类名？

image
image2034×456 54.2 KB

另外——更重要的是——能否在主要的 .quote aside（父 div）上也添加一个类名，以指示被引用的用户是否已被屏蔽？

我并不是要求任何视觉上的更改，也不是要求更改“忽略”功能的运作方式 :wink: 我只是想要一些类名，这样我和其他未来的主题开发者就能有更多的选项可用。

如果有人能大致指点一下我应该在哪里尝试修改代码库，我也很乐意亲自尝试并提交 PR。

2

你确定需要这样做吗？难道不能用其他方式来定位它吗？如果实在必要，你甚至可以使用 title 属性来定位它。

3

我或许可以在 CSS 中定位那两个元素（箭头），是的。这将需要一些复杂的选择器，而且显然比直接给这些元素添加类名更脆弱，但我应该能搞定。

更困难的是判断该用户是否被忽略。据我所见，没有任何父容器添加类名来表明引用的“已忽略”状态。如果没有这个类名，我就无法在 CSS 中做任何不同的处理。你怎么看？如果我错了请告诉我 :slight_smile:

4

被忽略的用户被引用，但该用户可能不存在，这确实需要核心代码的修改。

我不太确定是否应该在这里为添加这个类而工作，不过如果你提交一个 PR 且实现起来足够简单，我很乐意将其合并。

5

我看看能不能搞定 :slight_smile:

6

这出乎意料地简单！我花的时间比写那一行代码还长，主要是找在哪里添加代码。难道不总是这样吗？:wink:

嘿，伙计们，这是我第一次为 Discourse 核心提交拉取请求！:partying_face:

拉取请求已提交：FEATURE: add class to <aside> quote block when quoting an ignored user by pnoeric · Pull Request #9511 · discourse/discourse · GitHub

7

上下箭头很简单

.quote-controls .svg-icon-title
.quote-controls .back

关于被屏蔽的帖子我完全不确定，你说的有道理，你的 PR 可能是最好的解决方案。