Closed TeaEffTeu closed 2 years ago
I can't see any benefits to accepting this? What does this add, specifically? (Of course, apart from the new websites to target.)
Just wanted to make it look nicer
the javascript part that removes the number_of_requests bla bla after ever site is nice, haven't tested if it works though.
What does the rand stuff exactly do?
The rand thing was a leftover bit that I was testing and forgot to remove, no point to it really
This looks great! Unfortunately - another PR made, conflicts with it - if you can refactor it to resolve the new conflicts, I can merge it.
First time using bootstrap, so it took a bit
Changed
Removed
Added settings panel for:
I like how many changes you tried to add, even though non have made it so far. I don't feel like rewriting everything would be useful again, but, that last option. Put it in with the current code base, and make it behind an "advanced" button thats next to the pause/resume button. This button should open a modal that lets you tweak the settings yourself. I would recommend closing this pull request, copy our code and add the settings panel with an advanced button + popup/modal. Since ur new with bootstrap, you can use this bootstrap code to make a modal/popup work. If you need help with the bootstrap, just ask me.
`
Added:
Changed: