ironmonk88 / monks-little-details

GNU General Public License v3.0
33 stars 36 forks source link

Monk's Little Details, Sort Statuses by Columns incorrect sort order #181

Closed GigawattWarlock closed 2 years ago

GigawattWarlock commented 2 years ago

When I use Monk's Little Details in combination with DFred's Convenient Effects, I prefer to use the "Sort Statuses by Columns" option within the Little Details module settings.

Unfortunately, the list isn't alphabetized correctly if you add anything to the list.

It is possible that to reproduce this issue you may have to follow this procedure:

  1. Enable DFred's Convenient Effects and Monk's Little Details.
  2. Set Monk's Little Details --> Sort Status Effects --> COLUMNS (save and reload)
  3. Using DFred's Convenient Effects, right-click on any effect and choose "Toggle Status Effect" (reload)

When the sorting arrangement is set to Columns, adding anything to the list causes several items to come out of order.

image
Shareenx commented 2 years ago

Same thing happens with CUB (Combat Utility Belt)

ironmonk88 commented 2 years ago

What system are you using?

Shareenx commented 2 years ago

5e =) By the way, love your modules and videos

On Wed, Jun 22, 2022 at 6:22 PM ironmonk88 @.***> wrote:

What system are you using?

— Reply to this email directly, view it on GitHub https://github.com/ironmonk88/monks-little-details/issues/181#issuecomment-1163820559, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATK6IEXHNZEOEOOGIE34NKTVQO34BANCNFSM5ZF72SSA . You are receiving this because you commented.Message ID: @.***>

ronin-seven commented 2 years ago

I can reproduce this without applying any effect (# 3 above).

Modules active: DFreds Convenient Effects, Monk's Little Details, libWrapper and socketlib System: DnD5e

Default settings with the exception of:

  1. DCE -> Modify Status Effects = Replace
  2. MLD -> Sort Statuses = Columns

image

If you notice, Exhaustion 1 - Grappled - Prone cells seem to have been shifted one position to the left with Exhaustion 1 wrapping up to the previous line.

ronin-seven commented 2 years ago

Looks like this is no longer an issue with the latest versions of DCE (2.7.0) and MLD (1.0.55)

ironmonk88 commented 2 years ago

This should be fixed in the latest release