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

chore: apply ux feedback #19286

Merged
merged 12 commits into from
Dec 22, 2021
Merged

chore: apply ux feedback #19286

merged 12 commits into from
Dec 22, 2021

Conversation

JessicaSachs
Copy link
Contributor

  • Closes

User facing changelog

Additional details

How has the user experience changed?

PR Tasks

  • Have tests been added/updated?
  • Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
  • Has a PR for user-facing changes been opened in cypress-documentation?
  • Have API changes been updated in the type definitions?
  • Have new configuration options been added to the cypress.schema.json?

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Dec 7, 2021

Thanks for taking the time to open a PR!

@cypress
Copy link

cypress bot commented Dec 7, 2021



Test summary

18355 0 218 0Flakiness 3


Run details

Project cypress
Status Passed
Commit 00e8eea
Started Dec 22, 2021 12:07 AM
Ended Dec 22, 2021 12:19 AM
Duration 11:53 💡
OS Linux Debian - 10.10
Browser Multiple

View run in Cypress Dashboard ➡️


Flakiness

net_stubbing_spec.ts Flakiness
1 network stubbing > waiting and aliasing > can timeout waiting on a single request using "alias.request"
2 network stubbing > waiting and aliasing > can timeout waiting on a single request using "alias.request"
xhr_spec.js Flakiness
1 ... > no status when request isnt forced 404

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

@@ -174,8 +174,8 @@
"bodySuccess": "You are now logged in as {0}.",
"bodyError": "An error occurred while attempting to log in:",
"bodyBrowserError": "Cypress was unable to open your installed browser. To continue logging in, please paste this URL into your web browser:",
"dashboard": "Cypress Dashboard Service",
"titleInitial": "Log In To Cypress",
"cloud": "Cypress Cloud Service",
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we using "Cloud" in this release? I think this is still very much up in the air and may cause confusion if we haven't changed this verbiage throughout services before releasing this.

Copy link
Contributor

@marktnoonan marktnoonan left a comment

Choose a reason for hiding this comment

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

Pulled this branch and the changes all seem to work just fine.

Copy link
Contributor

@marktnoonan marktnoonan left a comment

Choose a reason for hiding this comment

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

Actually noticed some inconsistent font weights in the header, not sure why we are bold for Docs when focused (and Upgrade always) but not for Browser and Login, but it looks off
Screen Shot 2021-12-07 at 2 46 00 PM
Screen Shot 2021-12-07 at 2 45 54 PM
Screen Shot 2021-12-07 at 2 45 49 PM

marktnoonan
marktnoonan previously approved these changes Dec 21, 2021
@marktnoonan
Copy link
Contributor

I made minor changes to have consistent fonts in the header and resolve some conflicts where @ZachJW34 had made changes that overlapped.

@marktnoonan marktnoonan merged commit 67c42fc into 10.0-release Dec 22, 2021
@marktnoonan marktnoonan deleted the chore/apply-ux-feedback branch December 22, 2021 01:12
tgriesser added a commit that referenced this pull request Dec 22, 2021
…cycle-ui

* 10.0-release:
  chore: apply ux feedback (#19286)
  feat: add favicon to the app to avoid 404 (#19416)
tgriesser added a commit that referenced this pull request Dec 22, 2021
…cycle

* 10.0-release:
  chore: apply ux feedback (#19286)
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.

3 participants