scottjehl
Repos
55
Followers
2357
Following
36

Load CSS asynchronously

6760
532

A simple function for asynchronously loading JavaScript files

897
97

A modal web component

74
2

A minimal container query web component

38
1

An accessible carousel web component

18
0

An accessible collapsible web component

Events

Load the details page with display_errors 1

one more

ditch idx(), ?? coalece instead

revert vendor changes

CRUX presentation tweaks:

  • same 12 point font for all %
  • WPT metrics in blue shades
  • add units of measure
  • WPT values in the title for mouseover
  • key with fewer :s

fix(test runs): don't count against exempt host

We like people to be able to test features against things like The Metric Times. This helps!

Merge pull request #2686 from WPO-Foundation/do-not-count-exempt-hosts

fix(test runs): don't count against exempt host

Merge pull request #2669 from WPO-Foundation/thecrux

CRUX presentation tweaks:

Merge pull request #2653 from WPO-Foundation/display_errors

Load the details page with display_errors 1

Anon experiments on the metric times

Merge pull request #2687 from WPO-Foundation/metrictimes-anon

Anon experiments on the metric times

Put the free experiments we offer on the anon plan

Merge pull request #2689 from WPO-Foundation/free-exp-anon

Put the free experiments we offer on the anon plan

Merge pull request #2688 from WPO-Foundation/master

update prod

Created at 1 day ago
pull request closed
update prod
Created at 1 day ago

Put the free experiments we offer on the anon plan

Merge pull request #2689 from WPO-Foundation/free-exp-anon

Put the free experiments we offer on the anon plan

Created at 2 days ago
pull request closed
Put the free experiments we offer on the anon plan
Created at 2 days ago
pull request opened
Put the free experiments we offer on the anon plan
Created at 2 days ago
scottjehl create branch free-exp-anon
Created at 2 days ago
pull request opened
update prod
Created at 2 days ago
scottjehl delete branch metrictimes-anon
Created at 2 days ago

Anon experiments on the metric times

Merge pull request #2687 from WPO-Foundation/metrictimes-anon

Anon experiments on the metric times

Created at 2 days ago
pull request closed
Anon experiments on the metric times
Created at 2 days ago
pull request opened
Anon experiments on the metric times
Created at 2 days ago
scottjehl create branch metrictimes-anon
Created at 2 days ago
Test syntax highlighting and tab control in the edit response html experiment

it currently does not use syntax highlighting and tab leaves the field.

Trapping tab may be a bad idea... This will need some testing and planning.

Created at 5 days ago
Always save first response body by default

Several experiments use the initial HTML and it'd be nice to not have to remember to save response bodies for those to work

Created at 5 days ago
Add Experiment: Edit response body

This experiment allows you to edit the response of any same-domain file requested in this test. The text in this field will replace the text of the response before it is sent to the browser.

As an initial release, allowing editing the initial request response body is likely the most important one (the HTML). Ideally, we'd allow one or many of these fields and offer all same domain requests in the select menu.

Proposed Design

Created at 5 days ago
Add Experiment: Edit response body

Merged!

Created at 5 days ago

styles for #2471

custom experiment for #2471

experiments page edits to support #2471

test runner updates to handle post for #2471

special task to return submitted experiment HTML for #2471

Merge branch 'master' into 2471-htmledit

var for local host url

Typo fix common.inc

secondary anon setting for experiments

move var outside if

Merge pull request #2681 from WPO-Foundation/staging-exps

secondary anon setting for experiments

disable dependent fields when their initial check is unchecked

This will be helpful in avoiding sending large payloads from populated fields that are not part of an experiment run

css to pair with hiding fields that are unchecked

Merge pull request #2682 from WPO-Foundation/dep-fields

disable dependent fields when their initial check is unchecked

Merge pull request #2659 from WPO-Foundation/typoOppsExp

Typo fix common.inc

merge

fix presentation of textarea

Merge pull request #2643 from WPO-Foundation/2471-htmledit

Edit Initial Response HTML experiment. #2471

Merge pull request #2683 from WPO-Foundation/master

update prod

Created at 5 days ago
pull request closed
update prod
Created at 5 days ago
pull request opened
update prod
Created at 5 days ago
scottjehl delete branch 2471-htmledit
Created at 5 days ago

styles for #2471

custom experiment for #2471

experiments page edits to support #2471

test runner updates to handle post for #2471

special task to return submitted experiment HTML for #2471

Merge branch 'master' into 2471-htmledit

var for local host url

move var outside if

merge

fix presentation of textarea

Merge pull request #2643 from WPO-Foundation/2471-htmledit

Edit Initial Response HTML experiment. #2471

Created at 5 days ago
pull request closed
Edit Initial Response HTML experiment. #2471

A proof of concept for review.

This is the experiments page UI to pair with a new Edit HTML Response body experiment. It allows WebPageTest Pro users to freely edit the response HTML of the site being tested, and the experiment runner will deliver that edited HTML directly to the browser as if it was server-generated.

The big advantage here is control over complicated changes to HTML, which is going to be preferable in many cases to running a find and replace experiment.

Here's how it looks in the Opportunities page on any test that has a response body for the initial HTML:

Currently at least, on focus, the textarea fills much of the window. We'll need to think about this toggle from an accessibility and usability standpoint, but it's nice for big text edits I think.

This experiment is a bit different than others in that the experiment post data can be quite large, so the experiment runner does not expect to find that posted HTML in the cookie we usually send for experiments, but instead in the test's metadata. There's an endpoint set up in the files of this PR that gets that HTML from the metadata. The experiment proxy fetches that HTML using the test ID that is submitted as a cookie and uses that response as the HTML response.

Here's a result of an experiment that used an completely different HTML source with just an H1 in it:

Still to-do:

  • review experiment post/retrieve workflow to see if it's a good approach for experiments that rely on heavier post data
  • limit the experiment proxy to the specific request url instead of any HTML-type request as it currently replaces.
  • consider accessibility of textarea expand
  • Note: This experiment relies upon a pull request in the experiments repo, so that'll need to merge first.
Created at 5 days ago

fix presentation of textarea

Created at 5 days ago

Dead code: delete or fix

  • srcMatches is not declared (though srcmatches is)
  • srcAttrVal is not used

A better fix?

Counting fix in the insecure requests opp

Breakpoint fix for footer ul

Signup FAQ width adjustment

Merge pull request #2649 from WPO-Foundation/signupFaqWidth

Signup FAQ line width adjustment

Merge pull request #2648 from WPO-Foundation/footerBreakFix

Breakpoint fix for footer ul

Merge pull request #2647 from WPO-Foundation/insecureRequests

Counting fix in the insecure requests opp

remove overflow entirely on results_body, restoring sticky filmstrips fixes #2650

this pseudo lets the results body padding kick in, as overflow: hidden had done, which we do not want to use

Merge pull request #2651 from WPO-Foundation/2650

Restore filmstrip sticky on compare view

Traceroute home page tweaks addressing #2658

  • Remove browser select
  • Number of tests a
  • formatting

Traceroute results tweaks, prep to blade-ify

  • error notice
  • typo
  • formatting

Lightouse start page notice cleanup

Replace the common pattern of hunting for .ini

  • It's a little much to type and fix
  • Moved to a static helper
  • Start with the most likely to overwrite otherwise we were sometimes parsing three files

format

composer format

Close request details on hitting Escape

formatting

Fix border in results summary table

Created at 5 days ago
scottjehl delete branch dep-fields
Created at 5 days ago

disable dependent fields when their initial check is unchecked

This will be helpful in avoiding sending large payloads from populated fields that are not part of an experiment run

css to pair with hiding fields that are unchecked

Merge pull request #2682 from WPO-Foundation/dep-fields

disable dependent fields when their initial check is unchecked

Created at 5 days ago
pull request closed
disable dependent fields when their initial check is unchecked

This will be helpful in avoiding sending large payloads from populated fields that are not part of an experiment run

Created at 5 days ago

css to pair with hiding fields that are unchecked

Created at 5 days ago
pull request opened
disable dependent fields when their initial check is unchecked

This will be helpful in avoiding sending large payloads from populated fields that are not part of an experiment run

Created at 5 days ago
scottjehl create branch dep-fields
Created at 5 days ago