Retort - a reaction-style plugin for Discourse

(Angus McLeod) #269

Yeah fair.

Useful feedback, thanks.

0 Likes

(Danny Goodall) #270

Something that I would like to see, especially if you’re using the Like system as the model for much of the functionality being added, is a notification feature.

I would see this operating in the same way as notifications currently work for Likes. i.e. it would be a good idea to implement the logic that stops you being notified about every Like unless time has passed between Likes.

If my post attracts a retort I should receive a message telling me who reacted and with which emoji. I wouldn’t expect a notification for different retorts from the same user - unless significant time had passed between each different retort.

1 Like

(DiscourseMetrics.com) #271

Badges based on retorts would also be neat :slight_smile:

3 Likes

(Ryan Erwin) #272

I’ve found that lots of folks can be a troll once in a while… Maybe they’re just having a rough day. Discourse has tons of tools (flags, re-categorization, move to new topic, editing, hiding, convert to PM) to manage posts and replies… Interesting that folks using all of these tools would feel easily editing retorts is unnecessary.

1 Like

(Tom) #273

That’d be great. That way we can remove the “bugged/translated” reactions that some people add as a result of translating the page using builtin translation tools in the browser (as mentioned in a previous report).

2 Likes

(Wolftune) #274

After recent Discourse upgrade, I’m seeing only undefined in text instead of emoji in all retort reactions, both new and historic.

We’ve adjusted color and position CSS for retorts but no other customizing that would relate to this glitch. I’m guessing something just broke with the upgrade to Discourse and needs to be fixed.

0 Likes

(James Kiesel) #275

Hmm the demo site is running the latest Discourse / Retort and doesn’t seem to be having issues; could you share your customizations, and link me to the site in question?

0 Likes

(Boost) #276

I just got the same error after updating to v2.2.0.beta8 +63 and the latest retort version (b65cac1).

Here’s an example post showing UNDEFINED instead of the emoji: Possibilities Considering Symbiosis - Future Game - Thrive Community Forum

I think the test site you linked uses a different version as on my site clicking the UNDEFINED button opens a list of people who added reactions (when you are logged in) and on that test side the list of people who reacted is shown on mouse hover.

Edit: I looked through my theme and tried commenting out a few retort related rules but that did nothing.

0 Likes

(Ixeft) #277

Same problem for us on a different instance!

0 Likes

#278

yup. :point_up:

0 Likes

(James Kiesel) #279

@8BIT Can you link me to that post on your instance please? It looks like you have the new ‘retort_standard_count’ setting enabled; is that the case?

1 Like

(James Kiesel) #280

For now, I’ve reverted a recent change which I believe caused this, and will investigate further. Rebuilding should fix this for now; thanks for the reports everyone. :slight_smile:

1 Like

#281

Can I use this for android?

0 Likes

#282

that’s right, i turned it on just for shits… but, it didn’t do anything.

a bunch of examples in this large thread.

0 Likes

(Boost) #283

Updating to the latest commit of retort seems to have fixed it.

0 Likes

(Danny Goodall) #284

I’ve got a couple of issues to report - I’ll keep them in separate posts.

Firstly, there seems to be an issue between the Retort plugin and the panel that is displayed by the Insert Date widget.

When the date panel is displayed it will appear to go ‘behind’ posts beneath it.

Before clicking on the date widget

After clicking on the date widget

in case it is not clear from the screen-grab, the orange rectangle is the panel containing the list of timezones for that local time. It clearly goes ‘beneath’ the post below.

To Reproduce

I’m able to reproduce this on our live site under the latest v2.3.0.beta1 +8 with the Retort plugin enabled.

Simply insert a date into a post, then post a reply to the topic so there is at least one post below the post with the date widget on it.

Cause?

It looks like the issue is down to the Retort plugin setting the z-index of the div.topic-body to 3 in retort.scss.

If I use dev tools so that the style is not applied to .topic-post .topic-body, then the date panel behaves as I’d expect.

Disabling Retorts was not enough as the CSS remains active, so I had to rebuild without the Retorts plugin in my app.yml.

Having rebuilt without the Retorts plugin the date panel is displayed as I would expect.

0 Likes

(Hanzo) #285

@gdpelican

Any chance at a fix for when you react, it updates position randomly? I’d like to have them act just like Discord reactions; it seems kind of strange they change order constantly.

Can provide screencast if this is confusing.

0 Likes