Skip to content

Conversation

lmiller1990
Copy link
Contributor

@lmiller1990 lmiller1990 commented Nov 15, 2021

Targeting: feat-spec-list-branch (has PR: #18901)

Uses the spec tree code from the ^ branch to also show a tree view on the main spec page.

Note: some tiny imperfections, like the control to toggle between tree and flat view is black/white - it should be inverted.

image

@lmiller1990 lmiller1990 marked this pull request as ready for review November 15, 2021 12:56
@lmiller1990 lmiller1990 changed the title add tree list to spec page feat(app): add tree list to spec page Nov 15, 2021
@cypress
Copy link

cypress bot commented Nov 15, 2021



Test summary

18648 1 216 7Flakiness 4


Run details

Project cypress
Status Failed
Commit 8507184
Started Nov 15, 2021 4:02 PM
Ended Nov 15, 2021 4:14 PM
Duration 11:30 💡
OS Linux Debian - 10.9
Browser Multiple

View run in Cypress Dashboard ➡️


Failures

cypress/integration/commands/navigation_spec.js Failed
1 ... > throws on: '[object Object]'

Flakiness

commands/navigation_spec.js Flakiness
1 src/cy/commands/navigation > #visit > window immediately resolves and doesn't reload when visiting the same URL with hashes
cypress/proxy-logging-spec.ts Flakiness
1 Proxy Logging > request logging > xhr log has response body/status code
commands/net_stubbing_spec.ts Flakiness
1 network stubbing > intercepting request > can modify the request body
2 network stubbing > waiting and aliasing > can timeout waiting on a single request using "alias.request"

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@ZachJW34
Copy link
Contributor

I tweaked a few styles, mostly for the icon alignment. Rest looks good, LGTM!

@ZachJW34 ZachJW34 merged commit 7df0921 into feat-spec-list-branch Nov 15, 2021
@ZachJW34 ZachJW34 deleted the lmiller1990/feat-spec-list-branch-updates branch November 15, 2021 15:58
lmiller1990 added a commit that referenced this pull request Nov 16, 2021
* feat: create spec tree

* improve types

* feat(app): add tree list to spec page (#18913)

Co-authored-by: Zachary Williams <[email protected]>

* address comments

* update style

* better types

* remove unused code

* remove old code

Co-authored-by: Lachlan Miller <[email protected]>
Co-authored-by: Zachary Williams <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants