Bentheburrito / sacastats

A stat site for Connery's SaCa outfit members. Built with Elixir and Phoenix
MIT License
2 stars 0 forks source link

Weapons table v3 #135

Closed dm0896665 closed 1 year ago

dm0896665 commented 1 year ago

Lots of code changes, but most of it was removing the old hacky way of listening for specific clicks and waiting random amounts of time to tap into BootstrapTable's custom events to make everything a lot more robust. Other than that, there are a few key updates:

All of this appears to work properly in my testing on both firefox and chrome. There is only one thing that happens very randomly. The table data will not match the column header (the table data goes back to the default column order and only the desktop table headers are sorted properly) If you can find a way to replicate this so I can fix it that would be great. It has only happened when the column order is something other than default and I sort a column (asc or desc).

This will close #60 and will close #125.

dm0896665 commented 1 year ago

Looks like the issue I was talking about was a Bootstrap Table issue. I went ahead and submitted an issue for that. We may have to wait for a patch