-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
💅🏽 461 [New UX] Trace View # 3 (Test Results) #477
Conversation
@@ -21,7 +21,6 @@ describe('Show Trace', () => { | |||
cy.get('[data-cy=span-details-attributes]').should('be.visible'); | |||
cy.get('[data-cy=empty-assertion-table]').should('be.visible'); | |||
|
|||
cy.get('[id*=tab-test-results').click(); | |||
cy.get('[data-cy=test-results]').should('be.visible'); | |||
cy.get('[data-cy=test-results]').should('exist'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test results its only partially displayed when the trace view loads
@@ -24,12 +24,6 @@ body { | |||
background: #fbfbff; | |||
} | |||
|
|||
.ant-typography { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a wrong move, it was overwriting the color for the whole app 🥴
@@ -0,0 +1,94 @@ | |||
import {Badge, Typography} from 'antd'; | |||
import {useSelector} from 'react-redux'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moving it to its own component folder
onSelectSpan={onSelectSpan} | ||
selectedSpan={selectedSpan} | ||
/> | ||
<TraceDrawer visiblePortion={visiblePortion} testResultDetails={testResultDetails} onSelectSpan={onSelectSpan} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trace drawer is whatever we show at the bottom of the page
return ( | ||
<ResizableDrawer open min={visiblePortion} max={max}> | ||
<TraceTimeline | ||
<TestResults |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now we show the test results instead of the timeline chart.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome! If we can, lets push all the way through to the demo site - it is looking good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the current component of TraceResult expands bigger the actual screen.
We need to take a look at this @xoscar
Note: this is the first pass for the test results drawer. We'll be waiting for the final designs to fully implement the changes.
This PR moves the test results from the span detail/tab section to the bottom drawer.
Changes
Fixes
Checklist
https://www.loom.com/share/432e43f773d64935b8d7e67de18445fb