Modify nodebb-plugin-ws-dice

DownPW

Hi all

I have see this plugin: nodebb-plugin-ws-dice

nodebb-plugin-ws-dice/static/lib/main.js at main · Wieloswiat/nodebb-plugin-ws-dice

Dice throws integrated with timeline. Contribute to Wieloswiat/nodebb-plugin-ws-dice development by creating an account on GitHub.

GitHub (github.com)

It's a dice game but only works in posts and topics.

I would like this plugin to work in the chat because I find it strange to play this game in a topic.

I wanted the same in chats with the result, for example: /roll 4d6

... in a simple chat text feedback.

If any kind soul would like to help me do this, I would be eternally grateful.

phenomlab

@DownPW when you mention chat, do you mean one to one, or one to many - for example, a shout box ?

I don't think this needs a plugin. It could be executed using simple jQuery. I can it together a proof of concept if you like.

Probably better at Sudonix for the discussion, code, testing etc, and the final version can be added here afterwards.

DownPW

ha ha @phenomlab

I didn't want to bother you with this
you make my day, go to sudonix then

DownPW

The plugin work now on messages/chat/global chat

There is just a bug in the display of the result in the topics only when the dice are rolled too long and the result is displayed on 2 lines. it's Ok in 1 line

1 line = OK

2 lines = KO

@baris or other people:

Any idea how to fix this little bug?

Why would the display be correct on one line and not on 2 or 3 or 4 ...?

Otherwise, it will be very good to add or fork this plugin on NodeBB communauty, it's a very good plugin.

<baris>

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

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