Closed Bug 1693320 Opened 3 months ago Closed 3 months ago

Case of URI fragment after hash mark cannot be edited in search bar

Categories

(Firefox :: Address Bar, defect, P3)

Product:
Firefox
Component:
Address Bar
Version:
Firefox 85
Type:
defect
Points:
3

Tracking

()

Status:
VERIFIED FIXED
Milestone:
88 Branch
Tracking Flags:
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- wontfix
firefox87 --- wontfix
firefox88 --- verified

People

(Reporter: fabrizio.gotti, Assigned: daisuke)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1693320: Avoid autofilling if uri fragment does not match with case-sensitive.
48 bytes, text/x-phabricator-request
Details | Review

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:85.0) Gecko/20100101 Firefox/85.0

Steps to reproduce:

  1. Go to invalid URL https://docs.docker.com/engine/reference/builder/#COPY
  2. Edit the URL in search bar to correct the case of the fragment "COPY" to the correct one, "copy" (in lowercase) and press enter

Actual results:

Firefox insists on invalid upper case #COPY and it is impossible to manually correct the uri

Expected results:

I should be allowed to modify the case of the url fragment

The Bugbug bot thinks this bug should belong to the 'Firefox::Address Bar' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: Untriaged → Address Bar

Thanks for filing this.

We correctly preserve the case in the input, but when you hit enter, we take the URL from the heuristic, which has the original case (and that's incorrect in this scenario).

Daisuke, are you interested in taking this? No problem if not, or if you're busy.

Severity: -- → S3
Status: UNCONFIRMED → NEW
Points: --- → 3
status-firefox86: --- → affected
status-firefox87: --- → affected
status-firefox88: --- → affected
status-firefox-esr78: --- → unaffected
Ever confirmed: true
Flags: needinfo?(daisuke)
Keywords: regression
Priority: -- → P3
Regressed by: 1606231

Thank you very much for pinging me, Drew.
Yep, I will take a look at this issue.

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Flags: needinfo?(daisuke)

A non-obvious workaround is to paste the correct URL and hit enter. I'm not saying that's the final outcome of this bug, just noting it.

Pushed by dakatsuka.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/02385a038230
Avoid autofilling if uri fragment does not match with case-sensitive. r=adw

https://hg.mozilla.org/mozilla-central/rev/02385a038230

Status: ASSIGNED → RESOLVED
Closed: 3 months ago
status-firefox88: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

Hi,
I've verified that this bug has been fixed on Nightly 88.0a1 (2021-03-16) on windows 10.

Regards, Flor

Status: RESOLVED → VERIFIED
Has Regression Range: --- → no
Has STR: --- → yes
status-firefox88: fixed → verified
You need to log in before you can comment on or make changes to this bug.