qBittorrent icon indicating copy to clipboard operation
qBittorrent copied to clipboard

Fix login script for firefox reload

Open JayRet opened this issue 4 months ago • 2 comments

Fixes reload bug in login.js for webgui on librewolf by adding location.assign(location); before location.reload(true);. left the latter for when location.assign(location); fails. Closes #20441.

JayRet avatar Feb 18 '24 18:02 JayRet

I think Location.replace() is more appropriate here. However, wait for other web devs to weigh in on this. I am not a very good web dev.

sledgehammer999 avatar Feb 18 '24 18:02 sledgehammer999

Yep, I think you're right. I'll commit with location.replace instead.

JayRet avatar Feb 18 '24 18:02 JayRet

I realized that i missed to add { } because of the multiline if and that's probably why the checks are failing. I'll commit with brackets.

JayRet avatar Feb 20 '24 12:02 JayRet

@JayRet if you know how, please "fixup" the commits into one. Something like this:

git rebase -i HEAD~4
// choose 'fixup' in the opened file in your text editor
// save file and exit
git push --force origin fix-firefox-login
//adjust 'origin' the correct remote name if you have changed it

sledgehammer999 avatar Feb 25 '24 22:02 sledgehammer999

@JayRet Thank you!

Chocobo1 avatar Mar 02 '24 06:03 Chocobo1

@sledgehammer999 I've just squash and merged it. IMO it is fine to merge it this way and isn't worth to keep waiting.

Chocobo1 avatar Mar 02 '24 06:03 Chocobo1

@Chocobo1 is it ok if this is backported?

sledgehammer999 avatar Mar 03 '24 03:03 sledgehammer999

@Chocobo1 is it ok if this is backported?

The new behavior might depends on each browser implementation so I'm not sure. And I'm not eager to present it to users.

Chocobo1 avatar Mar 03 '24 05:03 Chocobo1