Closed Bug 901019 Opened 9 years ago Closed 9 years ago

[MP] Defect - monocles do not adjust to pasted selection on the navigation bar

Categories

(Firefox for Metro Graveyard :: Input, defect, P2)

x86
Windows 8.1
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: jbecerra, Assigned: jimm)

References

Details

(Whiteboard: [preview] [selection] feature=defect u=metro_firefox_user c=content_features p=1)

Attachments

(1 file)

Tested on 2013-08-02 using try build from https://tbpl.mozilla.org/?tree=Try&showall=0&rev=12e7404f9314

I was using the software keyboard to copy paste addresses in the navigation bar when I noticed the monocles do not adjust to a new, smaller address pasted on the navigation app bar.

Steps:
1. Open Firefox Metro and go to a page like macrumors.com (or any address several characters long.
2. Open a new tab and go to io9.com
3. Using the skb select io9.com from the navigation app bar.
4. Go to the first tab and select swipe up to bring up the navigation bar.
5. Select the address there, and long-tap to get the context menu and Paste

Expected: The string you just pasted, io9.com, should be highlighted with the monocles at each end of the string.

Actual: io9.com is not highlighted (selected) and the right monocle is all by its lonesome self all the way to the right, where the previous string ended.
Whiteboard: feature=defect c=tbd u=tbd p=0
OS: Mac OS X → Windows 8 Metro
Whiteboard: feature=defect c=tbd u=tbd p=0 → [selection] feature=defect c=tbd u=tbd p=0
Priority: -- → P2
Blocks: caret-sel
Whiteboard: [selection] feature=defect c=tbd u=tbd p=0 → [selection] feature=defect u=metro_firefox_user c=content_features p=0
Whiteboard: [selection] feature=defect u=metro_firefox_user c=content_features p=0 → [preview-triage] [selection] feature=defect u=metro_firefox_user c=content_features p=0
Blocks: metrov1backlog
No longer blocks: metrov1defect&change
Summary: defect - monocles do not adjust to pasted selection on the navigation bar → [MP] Defect - monocles do not adjust to pasted selection on the navigation bar
Whiteboard: [preview-triage] [selection] feature=defect u=metro_firefox_user c=content_features p=0 → [preview] [selection] feature=defect u=metro_firefox_user c=content_features p=0
Component: General → Input
Assignee: nobody → jmathies
Blocks: metrov1it13
Whiteboard: [preview] [selection] feature=defect u=metro_firefox_user c=content_features p=0 → [preview] [selection] feature=defect u=metro_firefox_user c=content_features p=1
No longer blocks: metrov1backlog
Status: NEW → ASSIGNED
QA Contact: jbecerra
> Expected: The string you just pasted, io9.com, should be highlighted with
> the monocles at each end of the string.

This part doesn't appear to be desktop behavior so I'm going to fixi the monocles but leave this part out.
Attached patch fixSplinter Review
easy peasy one line fix.
Attachment #790380 - Flags: review?(netzen)
Attachment #790380 - Flags: review?(netzen) → review+
https://hg.mozilla.org/mozilla-central/rev/4662d40dca32
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.