Closed
Bug 1355073
Opened 7 years ago
Closed 5 years ago
Accidentally starting a drag on the identity block reloads the page and discards pending permission prompts
Categories
(Firefox :: Address Bar, enhancement, P5)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
Firefox 68
Tracking | Status | |
---|---|---|
firefox68 | --- | fixed |
People
(Reporter: florian, Assigned: phoenixgyaan, Mentored)
References
Details
Attachments
(1 file)
Noticed this while trying to reproduce bug 1345667. Things we could do: - ignore a drop when the dropped URL is the same as the current one - ignore URLs dropped on the identity part of the urlbar.
Comment 1•7 years ago
|
||
Marking this as P5, I think it's a good idea but we won't pursue it. If anyone wants to pick it up I'm happy to offer advice. I personally think we should do the former and ignore same URLs.
Mentor: jhofmann
Priority: -- → P5
Updated•7 years ago
|
Assignee: nobody → prathikshaprasadsuman
Updated•7 years ago
|
Assignee: prathikshaprasadsuman → nobody
Comment 2•5 years ago
|
||
The drop in the url bar is received here: https://searchfox.org/mozilla-central/rev/486b64e4bff86b7988af8c8b80845404ad197533/browser/base/content/urlbarBindings.xml#1051
We should add some code there that does one of the options mentioned in comment 0.
Assignee | ||
Comment 3•5 years ago
|
||
I would like to take this up please :)
Comment 4•5 years ago
|
||
You could assign yourself now :)
Assignee: nobody → phoenixgyaan
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•5 years ago
|
||
Changed the function to now check if the dropped and current URL are not same.
Assignee | ||
Comment 6•5 years ago
|
||
Treeherder for the change:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fdd11eec56f7760648eb1d1b9658c283345cbce8
Assignee | ||
Updated•5 years ago
|
Flags: needinfo?(jhofmann)
Assignee | ||
Comment 7•5 years ago
|
||
Hi Johann, how do I test this change locally..???
Comment 8•5 years ago
|
||
Hey, you can just drag the (i) icon in the url bar a few inches and then drop it again, with your new patch it shouldn't reload the page.
Flags: needinfo?(jhofmann)
Pushed by jhofmann@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/768adc665e9b Accidentally starting a drag on the identity block reloads the page and discards pending permission prompts. r=johannh
Comment 10•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox68:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Updated•5 years ago
|
Component: Site Identity and Permission Panels → Address Bar
You need to log in
before you can comment on or make changes to this bug.
Description
•