Skip to content

misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions#31119

Merged
jennifer-shehane merged 10 commits into
developfrom
viewport-dimensions-ux-update
Feb 21, 2025
Merged

misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions#31119
jennifer-shehane merged 10 commits into
developfrom
viewport-dimensions-ux-update

Conversation

@jennifer-shehane
Copy link
Copy Markdown
Member

@jennifer-shehane jennifer-shehane commented Feb 17, 2025

Additional details

Steps to test

  • I expanded some of the tests - didn't seem we had anything testing the scale display.
  • I'm leaving the Percy snapshots not approved so that they can be easily reviewed.

How has the user experience changed?

Before

End-to-end tests: with viewport size and scale

Screenshot 2025-02-18 at 1 53 27 PM

Component tests: just didn't render an input element (which design and I agreed is weird)

Screenshot 2025-02-17 at 1 40 54 PM

Dropdown on click of viewport

Screenshot 2025-02-18 at 1 53 54 PM

After

End-to-end tests:

Screenshot 2025-02-18 at 1 55 03 PM

Component tests:

Screenshot 2025-02-18 at 4 10 19 PM

PR Tasks

@jennifer-shehane jennifer-shehane self-assigned this Feb 17, 2025
@jennifer-shehane jennifer-shehane changed the title misc: Begin code for moving viewport dimensions and removing dropdown misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions Feb 17, 2025
@jennifer-shehane jennifer-shehane marked this pull request as ready for review February 18, 2025 18:35
@cypress
Copy link
Copy Markdown

cypress Bot commented Feb 18, 2025

cypress    Run #60483

Run Properties:  status check passed Passed #60483  •  git commit 3a0d38f1bc: Fix incorrect selector
Project cypress
Branch Review viewport-dimensions-ux-update
Run status status check passed Passed #60483
Run duration 09m 32s
Commit git commit 3a0d38f1bc: Fix incorrect selector
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 260
View all changes introduced in this branch ↗︎
UI Coverage  44.19%
  Untested elements 72  
  Tested elements 57  
Accessibility  94.9%
  Failed rules  0 critical   6 serious   0 moderate   1 minor
  Failed elements 132  

Copy link
Copy Markdown
Collaborator

@AtofStryker AtofStryker left a comment

Choose a reason for hiding this comment

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

I really like this change!

@jennifer-shehane jennifer-shehane merged commit 7012f11 into develop Feb 21, 2025
@jennifer-shehane jennifer-shehane deleted the viewport-dimensions-ux-update branch February 21, 2025 20:42
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 23, 2025
…dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…e dropdown explanation of viewport dimensions (#31119)
cypress-app-bot pushed a commit that referenced this pull request Feb 24, 2025
…dropdown explanation of viewport dimensions (#31119)
jennifer-shehane pushed a commit that referenced this pull request Feb 24, 2025
* chore: updating v8 snapshot cache

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
jennifer-shehane pushed a commit that referenced this pull request Feb 24, 2025
* chore: updating v8 snapshot cache

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
jennifer-shehane pushed a commit that referenced this pull request Feb 24, 2025
* chore: updating v8 snapshot cache

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

* index on develop: 7012f11 misc: Display viewport info in badge, remove dropdown explanation of viewport dimensions (#31119)

---------

Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
@cypress-bot
Copy link
Copy Markdown
Contributor

cypress-bot Bot commented Feb 25, 2025

Released in 14.1.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v14.1.0, please open a new issue.

@cypress-bot cypress-bot Bot locked as resolved and limited conversation to collaborators Feb 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Studio: Move viewport info into badge within URL bar

2 participants