danreeves / dt-exchange

39 stars 18 forks source link

Filter rules gui #58

Closed CautemocSg closed 1 year ago

CautemocSg commented 1 year ago

Adds an in-extension user interface for editing and adding filter rules previously handled by the JSON editor

errnoh commented 1 year ago

Doesn't seem to open in Firefox (probably good idea to check other browsers as well, for example there are bunch of users using some android browsers)

CautemocSg commented 1 year ago

Doesn't seem to open in Firefox (probably good idea to check other browsers as well, for example there are bunch of users using some android browsers)

It was broken in Firefox, latest should be working now. Tested on my desktop is working as intended. Could use a check on phone browsers.

errnoh commented 1 year ago

Yup, seems to be working on Firefox now.

I initially had some problems understanding the save button is at top. If you have bunch of rules you'll need to scroll upwards to find the save button. Should it be at the bottom?

positive:

negative:

errnoh commented 1 year ago

And a feature wish: allow dragging rules to reorder them. Wasn't relevant previously, but with the color option it's a proper QoL improvement :)

CautemocSg commented 1 year ago

Addressed these points with recent update:

  • JSON is a bit verbose with all fields being visible with default zero values
  • Not yet on feature parity with the current available rbf rules
  • The rule string doesn't seem to show everything, I have some rules that show as empty even when they have some rules defined. (e.g. rule with only blessing rarity defined)
errnoh commented 1 year ago

Great changes! Hopefully I wasn't too blunt with my feedback :) Just wanting to be sure that such a major UI overhaul feels intuitive to the users.

errnoh commented 1 year ago

Couple additional things that might be good features. Maybe not worth scoping in on this PR, but related:

CautemocSg commented 1 year ago

And a feature wish: allow dragging rules to reorder them. Wasn't relevant previously, but with the color option it's a proper QoL improvement :)

Have the highlight color visible in the rules listing somehow (not just edit window). Small box with that color?

@errnoh Thanks for the feedback and suggestions.

I implemented these 2 requests in the latest update.

errnoh commented 1 year ago

Replied on Discord but I'll add it to discussion here as well:

👍 from me functionality-wise. It's in a state that I have no need to manually edit JSON other than to test things and share configs between browsers & machines. I think if we can get someone to thumbs up the code changes this should be ok to merge?

Btw I've tested this on Firefox and Chrome. If someone has a setup where they can test that it works fine with the android browser bunch of people are using that would be a good thing to check.