Modify nodebb-plugin-ws-dice

DownPW

@baris said in Modify nodebb-plugin-ws-dice:

Are the dice results displayed as topic events? There might be an issue with the way the html is rendered and how it behaves when it is not one line. https://github.com/NodeBB/NodeBB/blob/master/public/src/modules/helpers.common.js#L199-L221

Yes @baris , the dev of this plugins displayed the dice results as topic events
is it possible to improve this?

phenomlab

@DownPW https://sudonix.com/topic/309/nodebb-chat-roll-dice-game/15?_=1660553010524

oplik0

Hmm... Now that I'm thinking about it again, I think NodeBB could make it easy for a plugin to modify whole topic event styles, without waiting for browsers to implement some way to select elements by their children (:has is one of few useful things Safari has that other browsers haven't yet enabled by default)

Currently, the event is a <li component="topic/event" class="timeline-event" data-topic-event-id="${event.id}"> element. It has a data-topic-event-id already, so there is precedent for adding some information about the event there, but it's not useful for styling. However, if it also had a data-topic-event-type attribute, it'd be trivial to select that element and its children.

I'll make a PR then I guess

EDIT: https://github.com/NodeBB/NodeBB/pull/10837

DownPW

@oplik0 said in Modify nodebb-plugin-ws-dice:

Hmm... Now that I'm thinking about it again, I think NodeBB could make it easy for a plugin to modify whole topic event styles, without waiting for browsers to implement some way to select elements by their children (:has is one of few useful things Safari has that other browsers haven't yet enabled by default)
Currently, the event is a <li component="topic/event" class="timeline-event" data-topic-event-id="${event.id}"> element. It has a data-topic-event-id already, so there is precedent for adding some information about the event there, but it's not useful for styling. However, if it also had a data-topic-event-type attribute, it'd be trivial to select that element and its children.
I'll make a PR then I guess

Hello @oplik0

Glad to see you here. Your plugin is already great and it deserves to be in the official nodeBB community