Skip to content
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

feat: Column width updates for latest runs and average duration work #22269

Merged

Conversation

pstakoun
Copy link
Contributor

@pstakoun pstakoun commented Jun 12, 2022

  • Closes

User facing changelog

Changes the Spec Explorer column sizes and adds padding to the rightmost column for Dashboard Integration data so the data is not flush with the scrollbar.

Additional details

Steps to test

How has the user experience changed?

PR Tasks

  • [na] Have tests been added/updated?
  • [na] Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
  • [na] Has a PR for user-facing changes been opened in cypress-documentation?
  • [na] Have API changes been updated in the type definitions?

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jun 12, 2022

Thanks for taking the time to open a PR!

data-cy="specs-testing-type-header"
>
{{ props.gql.currentProject?.currentTestingType === 'component' ?
t('specPage.componentSpecsHeader') : t('specPage.e2eSpecsHeader') }}
</div>
<div class="flex col-span-2 items-center justify-between">
<div class="flex items-center justify-between whitespace-nowrap">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if there is a better way to do this - this was the only header that was wrapping on smaller screens

@pstakoun pstakoun changed the title Column width updates for latest runs and average duration work feat: Column width updates for latest runs and average duration work Jun 12, 2022
@cypress
Copy link

cypress bot commented Jun 12, 2022



Test summary

37553 0 454 0Flakiness 6


Run details

Project cypress
Status Passed
Commit 692cb69
Started Jun 14, 2022 7:28 PM
Ended Jun 14, 2022 7:46 PM
Duration 18:01 💡
OS Linux Debian - 10.11
Browser Multiple

View run in Cypress Dashboard ➡️


Flakiness

commands/actions/click.cy.js Flakiness
1 ... > scroll-behavior > can scroll to and click elements in html with scroll-behavior: smooth
next.cy.ts Flakiness
1 Working with next-11-webpack-4 > should live-reload on src changes
cypress/proxy-logging.cy.ts Flakiness
1 Proxy Logging > request logging > xhr log has response body/status code when xhr response is logged second
2 ... > intercept log has consoleProps with intercept info
3 Proxy Logging > request logging > xhr log has response body/status code when xhr response is logged second
This comment includes only the first 5 flaky tests. See all 6 flaky tests in the Cypress Dashboard.

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

packages/app/src/specs/SpecsList.vue Outdated Show resolved Hide resolved
packages/app/src/specs/SpecsList.vue Outdated Show resolved Hide resolved
pstakoun and others added 2 commits June 12, 2022 21:34
Co-authored-by: Muaz Othman <muazweb@gmail.com>
@pstakoun pstakoun requested a review from MuazOthman June 13, 2022 01:58
@pstakoun pstakoun requested a review from tbiethman June 14, 2022 14:10
Copy link
Contributor

@mike-plummer mike-plummer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Took me a minute to parse the intent of the scrollbarOffset && logic. Might be a bit cleaner to extract these styles as computed props just to make intent clearer, but this gets the job done.

@jennifer-shehane
Copy link
Member

@pstakoun Could you please fill out the PR template explaining the work in this PR?

@pstakoun pstakoun merged commit 7e44f29 into muaz/CLOUD-577-spec-list-display-latest-runs Jun 14, 2022
@pstakoun pstakoun deleted the peter/feat/aci-columns branch June 14, 2022 19:22
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.

None yet

5 participants