Closed Bug 1517596 Opened 6 years ago Closed 6 years ago

Add enable toggle and cache/endpoint info to DiscoveryStream devtools

Categories

(Firefox :: New Tab Page, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 66
Iteration:
66.3 - Jan 7 - 20
Tracking Status
firefox66 --- fixed

People

(Reporter: k88hudson, Assigned: k88hudson)

References

Details

Attachments

(2 files)

In this bug we'll add some basic devtools for turning DiscoveryStream on/off and looking at cache info.
Iteration: --- → 66.2 - Dec 24 - Jan 6
Priority: -- → P1
Severity: normal → enhancement
Blocks: 1517867
Iteration: 66.2 - Dec 24 - Jan 6 → 66.3 - Jan 7 - 20

Can you verify devtools are working as expected?

Steps are:

  1. Set browser.newtabpage.activity-stream.asrouter.devtoolsEnabled to true in about:config
  2. Visit about:newtab#devtools-ds. Ensure you see devtools
  3. Try turning the new tab experiment off/on via the "enabled" checkbox, you should existing and newly opened tabs show the default experience / the experiment according to this setting.
Flags: needinfo?(bnagabandi)

Sure, will do as soon as build is ready tomorrow.

Flags: needinfo?(bnagabandi)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66

As of step3 :
Can you please clarify :
What does "existing and newly opened tabs show the default experience / the experiment according to this setting." mean?

Tested on both Mac and Windows (Version : 66.0a1 (2019-01-08))

MAC OS :

I see the difference for enabled vs disabled :

Enabled :
--> On New Tab Standard experience is missing, and seeing as in attachment, or is the new tab view as expected?
--> Rest works fine : Search Adding articles via Pocket tile

Disabled :
Standard experience and works as expected

WINDOWS :

After config setup , "about:newtab#devtools-ds" page has

  1. Text misalignment.
  2. Not able to turn on/off "enabled" switch. (Nothing happens)
Flags: needinfo?(khudson)

The screenshot you posted for enabled state is expected for now, as the design hasn't been fully implemented

For windows, I'm interested that you're not able to turn the on/off enabled switch – that definitely doesn't sound right!

Flags: needinfo?(khudson)
Depends on: 1518858

(In reply to Brahmini Nagabandi from comment #5)

Created attachment 9035127 [details]
Screen Shot 2019-01-08 at 12.00.33 PM.png

As of step3 :
Can you please clarify :
What does "existing and newly opened tabs show the default experience / the experiment according to this setting." mean?

Tested on both Mac and Windows (Version : 66.0a1 (2019-01-08))

MAC OS :

I see the difference for enabled vs disabled :

Enabled :
--> On New Tab Standard experience is missing, and seeing as in attachment, or is the new tab view as expected?
--> Rest works fine : Search Adding articles via Pocket tile

Disabled :
Standard experience and works as expected

WINDOWS :

After config setup , "about:newtab#devtools-ds" page has

  1. Text misalignment.
  2. Not able to turn on/off "enabled" switch. (Nothing happens)
Blocks: 1518858
No longer depends on: 1518858

After testing on windows on latest nightly, it seems that the enabled switch is working. Can you confirm this is still an issue, if so, maybe we can try to verify over Vidyo?

Flags: needinfo?(bnagabandi)

Will check when the build is ready.

Test Results:

Verified on :

FF Nightly Version - 66.0a1 (2019-01-17)

OS - Mac / Windows 10 Pro N

Looks good to me.

Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: