The 'Find' option is not working correctly while RDM is enabled

VERIFIED FIXED in Firefox 52

Status

DevTools
Responsive Design Mode
P1
normal
VERIFIED FIXED
2 years ago
a month ago

People

(Reporter: mboldan, Assigned: jryans)

Tracking

(Depends on: 1 bug)

52 Branch
Firefox 52
Dependency tree / graph

Firefox Tracking Flags

(firefox49 unaffected, firefox50 unaffected, firefox51 unaffected, firefox52 verified)

Details

(Whiteboard: [multiviewport] [reserve-rdm])

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
[Affected versions]:
- 52.0a1 (2016-09-27) 

[Affected platforms]:
- Windows 10x64, Mac OS X 10.11.6, Ubuntu 16.04x86

[Steps to reproduce]:
1. Launch Firefox.
2. From about:config, enable the devtools.responsive.html.enabled pref.
3. Enable RDM.
4. Open the Find option (Ctrl/Cmd+F) and search for an available string from the opened webpage.

[Expected result]:
- The searched string is correctly found and highlighted. 

[Actual result]:
- The searched string is not found. The search is performed only outside the RDM tool. 

[Regression range]:
- This is not a regression.
(Reporter)

Updated

2 years ago
QA Whiteboard: [qe-rdm]

Updated

2 years ago
Flags: qe-verify+
Priority: -- → P3
Whiteboard: [multiviewport] [triage] → [multiviewport] [reserve-rdm]
(Assignee)

Updated

2 years ago
Blocks: 1308297
(Assignee)

Updated

2 years ago
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)

Updated

2 years ago
Iteration: --- → 52.2 - Oct 17
Priority: P3 → P1

Updated

2 years ago
Iteration: 52.2 - Oct 17 → 52.3 - Nov 7

Comment 2

2 years ago
mozreview-review
Comment on attachment 8801845 [details]
Bug 1305945 - Support find in page with RDM.

https://reviewboard.mozilla.org/r/86470/#review85648
Attachment #8801845 - Flags: review?(poirot.alex) → review+

Comment 3

2 years ago
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f62a24bd53b5
Support find in page with RDM. r=ochameau

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f62a24bd53b5
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52

Comment 5

2 years ago
I have reproduced this bug with Nightly 52.0a1 (2016-09-28) in Elementary OS 64bit.

This bug's fix is now verified in latest Nighly 52.0a1

Build ID 	20161020030211
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0

[testday-20161021]
(Reporter)

Comment 6

2 years ago
I've also manage to test this issue on Firefox 52.0a1 (2016-10-23) and noticed 2 potential issues:
- if using Find option to search for an available string while RDM is enabled and disable RDM, the search string is wrongly positioned: see - https://i.imgur.com/sUL3PWv.png
- the search stings from the Find bar are deleted when RDM is enabled/disabled. Also the page is in search mode, even if the Find box is empty. (https://i.imgur.com/sUL3PWv.png)(not sure if this is a RDM related issue).
Ryan, what's your opinion about this?

The tests were performed under Windows 10x64, Mac OS X 10.11.6, Ubuntu 16.04 x64.
Flags: needinfo?(jryans)
I have reproduced this bug with Nightly 52.0a1 (2016-09-28) (64-bit) in Windows 7 , 64 Bit !

This bug's is now verified in Latest Nightly 52.0a1

Build ID    :  20161025030205
User Agent  :  Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:52.0) Gecko/20100101 Firefox/52.0

[bugday-20161026]
(Assignee)

Comment 8

2 years ago
(In reply to Mihai Boldan, QA [:mboldan] from comment #6)
> I've also manage to test this issue on Firefox 52.0a1 (2016-10-23) and
> noticed 2 potential issues:
> - if using Find option to search for an available string while RDM is
> enabled and disable RDM, the search string is wrongly positioned: see -
> https://i.imgur.com/sUL3PWv.png

I can't seem to reproduce this part myself, at least on macOS 10.12.  The highlight appears to reposition to match the text.

If you can still trigger it, let's file a bug to discuss further.

> - the search stings from the Find bar are deleted when RDM is
> enabled/disabled. Also the page is in search mode, even if the Find box is
> empty. (https://i.imgur.com/sUL3PWv.png)(not sure if this is a RDM related
> issue).

Yes, I can reproduce this part.  Let's file a separate bug for this.

Thanks for testing!
Flags: needinfo?(jryans)
(Reporter)

Updated

2 years ago
Depends on: 1313337
(Reporter)

Comment 9

2 years ago
See the comments inline.

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #8)
> (In reply to Mihai Boldan, QA [:mboldan] from comment #6)
> > I've also manage to test this issue on Firefox 52.0a1 (2016-10-23) and
> > noticed 2 potential issues:
> > - if using Find option to search for an available string while RDM is
> > enabled and disable RDM, the search string is wrongly positioned: see -
> > https://i.imgur.com/sUL3PWv.png
> 
> I can't seem to reproduce this part myself, at least on macOS 10.12.  The
> highlight appears to reposition to match the text.

The issue is still reproducible on my side, but I'm not quite sure that this is an RDM issue, since I noticed a couple issues related to the Find option, without enabling RDM. 
I will investigate and search for a regression asap.

> 
> If you can still trigger it, let's file a bug to discuss further.
> 
> > - the search stings from the Find bar are deleted when RDM is
> > enabled/disabled. Also the page is in search mode, even if the Find box is
> > empty. (https://i.imgur.com/sUL3PWv.png)(not sure if this is a RDM related
> > issue).
> 
> Yes, I can reproduce this part.  Let's file a separate bug for this.

I've logged Bug 1313337 for this issue.

> 
> Thanks for testing!

I am marking this issue Verified Fixed since one of the found issues was logged separately and the other one is unlikely to be related to this tool.
Status: RESOLVED → VERIFIED
status-firefox52: fixed → verified
No longer depends on: 1313337
Flags: qe-verify+
(Reporter)

Updated

2 years ago
Depends on: 1313337
(Assignee)

Updated

2 years ago
No longer blocks: 1308297

Updated

a month ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.